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

Groups > linux.kernel > #1671590 > unrolled thread

[PATCH] x86/apic: Make init_legacy_irqs() __init

Started byDou Liyang <douly.fnst@cn.fujitsu.com>
First post2017-06-21 12:20 +0200
Last post2017-06-22 13:20 +0200
Articles 4 — 2 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] x86/apic: Make init_legacy_irqs() __init Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-06-21 12:20 +0200
    [PATCH 2/2] x86/apic: Make arch_init_msi/htirq_domain __init Dou Liyang <douly.fnst@cn.fujitsu.com> - 2017-06-22 05:20 +0200
      [tip:x86/apic] x86/apic: Make arch_init_msi/htirq_domain __init tip-bot for Dou Liyang <tipbot@zytor.com> - 2017-06-22 13:20 +0200
    [tip:x86/apic] x86/apic: Make init_legacy_irqs() __init tip-bot for Dou Liyang <tipbot@zytor.com> - 2017-06-22 13:20 +0200

#1671590 — [PATCH] x86/apic: Make init_legacy_irqs() __init

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-06-21 12:20 +0200
Subject[PATCH] x86/apic: Make init_legacy_irqs() __init
Message-ID<tUKTT-4Pr-5@gated-at.bofh.it>
This function is only called by arch_early_irq_init() which is an
`initialization' function, So mark it __init as well. In addition
mark it inline for the X86_IO_APIC=n case.

Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
---
 arch/x86/kernel/apic/vector.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c
index f3557a1..e66d8e4 100644
--- a/arch/x86/kernel/apic/vector.c
+++ b/arch/x86/kernel/apic/vector.c
@@ -405,7 +405,7 @@ int __init arch_probe_nr_irqs(void)
 }
 
 #ifdef	CONFIG_X86_IO_APIC
-static void init_legacy_irqs(void)
+static void __init init_legacy_irqs(void)
 {
 	int i, node = cpu_to_node(0);
 	struct apic_chip_data *data;
@@ -424,7 +424,7 @@ static void init_legacy_irqs(void)
 	}
 }
 #else
-static void init_legacy_irqs(void) { }
+static inline void init_legacy_irqs(void) { }
 #endif
 
 int __init arch_early_irq_init(void)
-- 
2.5.5

[toc] | [next] | [standalone]

#1672271 — [PATCH 2/2] x86/apic: Make arch_init_msi/htirq_domain __init

FromDou Liyang <douly.fnst@cn.fujitsu.com>
Date2017-06-22 05:20 +0200
Subject[PATCH 2/2] x86/apic: Make arch_init_msi/htirq_domain __init
Message-ID<tV0OZ-7gs-1@gated-at.bofh.it>
In reply to#1671590
These two functions are only called by arch_early_irq_init() which
is an `initialization' function, So mark them __init as well.

Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
---
 arch/x86/kernel/apic/htirq.c | 2 +-
 arch/x86/kernel/apic/msi.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/htirq.c b/arch/x86/kernel/apic/htirq.c
index ae50d34..81ff489 100644
--- a/arch/x86/kernel/apic/htirq.c
+++ b/arch/x86/kernel/apic/htirq.c
@@ -150,7 +150,7 @@ static const struct irq_domain_ops htirq_domain_ops = {
 	.deactivate	= htirq_domain_deactivate,
 };
 
-void arch_init_htirq_domain(struct irq_domain *parent)
+void __init arch_init_htirq_domain(struct irq_domain *parent)
 {
 	if (disable_apic)
 		return;
diff --git a/arch/x86/kernel/apic/msi.c b/arch/x86/kernel/apic/msi.c
index c61aec7..4c5d188 100644
--- a/arch/x86/kernel/apic/msi.c
+++ b/arch/x86/kernel/apic/msi.c
@@ -136,7 +136,7 @@ static struct msi_domain_info pci_msi_domain_info = {
 	.handler_name	= "edge",
 };
 
-void arch_init_msi_domain(struct irq_domain *parent)
+void __init arch_init_msi_domain(struct irq_domain *parent)
 {
 	if (disable_apic)
 		return;
-- 
2.5.5

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

#1672546 — [tip:x86/apic] x86/apic: Make arch_init_msi/htirq_domain __init

Fromtip-bot for Dou Liyang <tipbot@zytor.com>
Date2017-06-22 13:20 +0200
Subject[tip:x86/apic] x86/apic: Make arch_init_msi/htirq_domain __init
Message-ID<tV8jw-3Vn-17@gated-at.bofh.it>
In reply to#1672271
Commit-ID:  538ac46c64a6bc61e71982091fc1eef0026f322e
Gitweb:     http://git.kernel.org/tip/538ac46c64a6bc61e71982091fc1eef0026f322e
Author:     Dou Liyang <douly.fnst@cn.fujitsu.com>
AuthorDate: Thu, 22 Jun 2017 11:15:41 +0800
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Thu, 22 Jun 2017 10:34:42 +0200

x86/apic: Make arch_init_msi/htirq_domain __init

These two functions are only called by arch_early_irq_init(), which
is an __init function, so mark them __init as well.

Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1498101341-10182-1-git-send-email-douly.fnst@cn.fujitsu.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 arch/x86/kernel/apic/htirq.c | 2 +-
 arch/x86/kernel/apic/msi.c   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/htirq.c b/arch/x86/kernel/apic/htirq.c
index ae50d34..81ff489 100644
--- a/arch/x86/kernel/apic/htirq.c
+++ b/arch/x86/kernel/apic/htirq.c
@@ -150,7 +150,7 @@ static const struct irq_domain_ops htirq_domain_ops = {
 	.deactivate	= htirq_domain_deactivate,
 };
 
-void arch_init_htirq_domain(struct irq_domain *parent)
+void __init arch_init_htirq_domain(struct irq_domain *parent)
 {
 	if (disable_apic)
 		return;
diff --git a/arch/x86/kernel/apic/msi.c b/arch/x86/kernel/apic/msi.c
index c61aec7..4c5d188 100644
--- a/arch/x86/kernel/apic/msi.c
+++ b/arch/x86/kernel/apic/msi.c
@@ -136,7 +136,7 @@ static struct msi_domain_info pci_msi_domain_info = {
 	.handler_name	= "edge",
 };
 
-void arch_init_msi_domain(struct irq_domain *parent)
+void __init arch_init_msi_domain(struct irq_domain *parent)
 {
 	if (disable_apic)
 		return;

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

#1672551 — [tip:x86/apic] x86/apic: Make init_legacy_irqs() __init

Fromtip-bot for Dou Liyang <tipbot@zytor.com>
Date2017-06-22 13:20 +0200
Subject[tip:x86/apic] x86/apic: Make init_legacy_irqs() __init
Message-ID<tV8jw-3Vn-23@gated-at.bofh.it>
In reply to#1671590
Commit-ID:  a884d25f383133c845d23c2cce929ba15994ca62
Gitweb:     http://git.kernel.org/tip/a884d25f383133c845d23c2cce929ba15994ca62
Author:     Dou Liyang <douly.fnst@cn.fujitsu.com>
AuthorDate: Wed, 21 Jun 2017 18:14:21 +0800
Committer:  Ingo Molnar <mingo@kernel.org>
CommitDate: Thu, 22 Jun 2017 10:34:41 +0200

x86/apic: Make init_legacy_irqs() __init

This function is only called by arch_early_irq_init(), which is an
__init function, so mark the child function __init as well.

In addition mark it inline for the !CONFIG_X86_IO_APIC case.

Signed-off-by: Dou Liyang <douly.fnst@cn.fujitsu.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1498040061-5332-1-git-send-email-douly.fnst@cn.fujitsu.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
---
 arch/x86/kernel/apic/vector.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c
index f3557a1..e66d8e4 100644
--- a/arch/x86/kernel/apic/vector.c
+++ b/arch/x86/kernel/apic/vector.c
@@ -405,7 +405,7 @@ int __init arch_probe_nr_irqs(void)
 }
 
 #ifdef	CONFIG_X86_IO_APIC
-static void init_legacy_irqs(void)
+static void __init init_legacy_irqs(void)
 {
 	int i, node = cpu_to_node(0);
 	struct apic_chip_data *data;
@@ -424,7 +424,7 @@ static void init_legacy_irqs(void)
 	}
 }
 #else
-static void init_legacy_irqs(void) { }
+static inline void init_legacy_irqs(void) { }
 #endif
 
 int __init arch_early_irq_init(void)

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web