Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1235530 > unrolled thread

[PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt

Started byDenys Vlasenko <dvlasenk@redhat.com>
First post2015-09-29 22:40 +0200
Last post2015-09-30 21:30 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt Denys Vlasenko <dvlasenk@redhat.com> - 2015-09-29 22:40 +0200
    Re: [PATCH] x86/apic: Deinline __x2apic_en/disable,  __smp_spurious/error_interrupt Thomas Gleixner <tglx@linutronix.de> - 2015-09-30 21:20 +0200
      Re: [PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt Denys Vlasenko <dvlasenk@redhat.com> - 2015-09-30 21:30 +0200
    [tip:x86/apic] x86/apic: Deinline various functions tip-bot for Denys Vlasenko <tipbot@zytor.com> - 2015-09-30 21:30 +0200

#1235530 — [PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt

FromDenys Vlasenko <dvlasenk@redhat.com>
Date2015-09-29 22:40 +0200
Subject[PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt
Message-ID<qe9QS-oL-13@gated-at.bofh.it>
From: root <root@lenovo-x3950-02.khw.lab.eng.bos.redhat.com>

__x2apic_disable: 178 bytes, 3 calls
__x2apic_enable: 117 bytes, 3 calls
__smp_spurious_interrupt: 110 bytes, 2 calls
__smp_error_interrupt: 208 bytes, 2 calls

Reduces code size by about 850 bytes.

Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
CC: Ingo Molnar <mingo@kernel.org>
CC: Borislav Petkov <bp@alien8.de>
CC: "H. Peter Anvin" <hpa@zytor.com>
CC: Thomas Gleixner <tglx@linutronix.de>
CC: Andy Lutomirski <luto@amacapital.net>
CC: x86@kernel.org
CC: linux-kernel@vger.kernel.org
---
 arch/x86/kernel/apic/apic.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index 24e94ce..d0c6577 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -1431,7 +1431,7 @@ enum {
 };
 static int x2apic_state;
 
-static inline void __x2apic_disable(void)
+static void __x2apic_disable(void)
 {
 	u64 msr;
 
@@ -1447,7 +1447,7 @@ static inline void __x2apic_disable(void)
 	printk_once(KERN_INFO "x2apic disabled\n");
 }
 
-static inline void __x2apic_enable(void)
+static void __x2apic_enable(void)
 {
 	u64 msr;
 
@@ -1807,7 +1807,7 @@ int apic_version[MAX_LOCAL_APIC];
 /*
  * This interrupt should _never_ happen with our APIC/SMP architecture
  */
-static inline void __smp_spurious_interrupt(u8 vector)
+static void __smp_spurious_interrupt(u8 vector)
 {
 	u32 v;
 
@@ -1848,7 +1848,7 @@ __visible void smp_trace_spurious_interrupt(struct pt_regs *regs)
 /*
  * This interrupt should never happen with our APIC/SMP architecture
  */
-static inline void __smp_error_interrupt(struct pt_regs *regs)
+static void __smp_error_interrupt(struct pt_regs *regs)
 {
 	u32 v;
 	u32 i = 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1236678 — Re: [PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt

FromThomas Gleixner <tglx@linutronix.de>
Date2015-09-30 21:20 +0200
SubjectRe: [PATCH] x86/apic: Deinline __x2apic_en/disable, __smp_spurious/error_interrupt
Message-ID<qev51-5JG-37@gated-at.bofh.it>
In reply to#1235530
On Tue, 29 Sep 2015, Denys Vlasenko wrote:

> From: root <root@lenovo-x3950-02.khw.lab.eng.bos.redhat.com>

Interesting patch author that.
 
> __x2apic_disable: 178 bytes, 3 calls

That superseeds your other patch which just touched
__x2apic_disable. Bah, I just queued that and added the
__x2apic_enable part myself. There would have been nothing wrong to
actually send a delta patch in reply to the other one...

Thanks,

	tglx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1236698

FromDenys Vlasenko <dvlasenk@redhat.com>
Date2015-09-30 21:30 +0200
Message-ID<qeveG-5V3-21@gated-at.bofh.it>
In reply to#1236678
On 09/30/2015 09:10 PM, Thomas Gleixner wrote:
> On Tue, 29 Sep 2015, Denys Vlasenko wrote:
> 
>> From: root <root@lenovo-x3950-02.khw.lab.eng.bos.redhat.com>
> 
> Interesting patch author that.

Oops. :)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1236694 — [tip:x86/apic] x86/apic: Deinline various functions

Fromtip-bot for Denys Vlasenko <tipbot@zytor.com>
Date2015-09-30 21:30 +0200
Subject[tip:x86/apic] x86/apic: Deinline various functions
Message-ID<qeveF-5V3-3@gated-at.bofh.it>
In reply to#1235530
Commit-ID:  d786ad32c305ca0f6be1924558866fe9f901e291
Gitweb:     http://git.kernel.org/tip/d786ad32c305ca0f6be1924558866fe9f901e291
Author:     Denys Vlasenko <dvlasenk@redhat.com>
AuthorDate: Tue, 29 Sep 2015 22:37:02 +0200
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Wed, 30 Sep 2015 21:15:53 +0200

x86/apic: Deinline various functions

__x2apic_disable: 178 bytes, 3 calls
__x2apic_enable: 117 bytes, 3 calls
__smp_spurious_interrupt: 110 bytes, 2 calls
__smp_error_interrupt: 208 bytes, 2 calls

Reduces code size by about 850 bytes.

Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
Cc: Borislav Petkov <bp@alien8.de>
Cc: Andy Lutomirski <luto@amacapital.net>
Link: http://lkml.kernel.org/r/1443559022-23793-1-git-send-email-dvlasenk@redhat.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---
 arch/x86/kernel/apic/apic.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/x86/kernel/apic/apic.c b/arch/x86/kernel/apic/apic.c
index 24e94ce..2f69e3b 100644
--- a/arch/x86/kernel/apic/apic.c
+++ b/arch/x86/kernel/apic/apic.c
@@ -1431,7 +1431,7 @@ enum {
 };
 static int x2apic_state;
 
-static inline void __x2apic_disable(void)
+static void __x2apic_disable(void)
 {
 	u64 msr;
 
@@ -1447,7 +1447,7 @@ static inline void __x2apic_disable(void)
 	printk_once(KERN_INFO "x2apic disabled\n");
 }
 
-static inline void __x2apic_enable(void)
+static void __x2apic_enable(void)
 {
 	u64 msr;
 
@@ -1807,7 +1807,7 @@ int apic_version[MAX_LOCAL_APIC];
 /*
  * This interrupt should _never_ happen with our APIC/SMP architecture
  */
-static inline void __smp_spurious_interrupt(u8 vector)
+static void __smp_spurious_interrupt(u8 vector)
 {
 	u32 v;
 
@@ -1848,7 +1848,7 @@ __visible void smp_trace_spurious_interrupt(struct pt_regs *regs)
 /*
  * This interrupt should never happen with our APIC/SMP architecture
  */
-static inline void __smp_error_interrupt(struct pt_regs *regs)
+static void __smp_error_interrupt(struct pt_regs *regs)
 {
 	u32 v;
 	u32 i = 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web