* [PATCH 1/4] eisa/pci_eisa.c: fix section mismatch
@ 2011-07-21 17:16 Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done' Arnaud Lacombe
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Arnaud Lacombe @ 2011-07-21 17:16 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnaud Lacombe, Greg KH, Andrew Morton
WARNING: vmlinux.o(.data+0x15d3ac): Section mismatch in reference from the variable pci_eisa_driver to the function .init.text:pci_eisa_init()
The variable pci_eisa_driver references the function __init pci_eisa_init()
If the reference is valid then annotate the variable with __init* or __refdata (see linux/init.h) or name the variable:
*_template, *_timer, *_sht, *_ops, *_probe, *_probe_one, *_console
Cc: Greg KH <greg@kroah.com>
Cc: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
---
drivers/eisa/pci_eisa.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/eisa/pci_eisa.c b/drivers/eisa/pci_eisa.c
index 0dd0f63..30da70d 100644
--- a/drivers/eisa/pci_eisa.c
+++ b/drivers/eisa/pci_eisa.c
@@ -45,13 +45,13 @@ static int __init pci_eisa_init(struct pci_dev *pdev,
return 0;
}
-static struct pci_device_id pci_eisa_pci_tbl[] = {
+static struct pci_device_id __initdata pci_eisa_pci_tbl[] = {
{ PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID, PCI_ANY_ID,
PCI_CLASS_BRIDGE_EISA << 8, 0xffff00, 0 },
{ 0, }
};
-static struct pci_driver pci_eisa_driver = {
+static struct pci_driver __initdata pci_eisa_driver = {
.name = "pci_eisa",
.id_table = pci_eisa_pci_tbl,
.probe = pci_eisa_init,
--
1.7.6.153.g78432
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done'
2011-07-21 17:16 [PATCH 1/4] eisa/pci_eisa.c: fix section mismatch Arnaud Lacombe
@ 2011-07-21 17:16 ` Arnaud Lacombe
2011-07-25 1:19 ` Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 3/4] usb/host/pci-quirks.c: correct annotation of `ehci_dmi_nohandoff_table' Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 4/4] drivers/base/devtmpfs.c: fix uninitialized variable Arnaud Lacombe
2 siblings, 1 reply; 6+ messages in thread
From: Arnaud Lacombe @ 2011-07-21 17:16 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnaud Lacombe, Greg Kroah-Hartman
This fixes the following section mismatch issue:
WARNING: vmlinux.o(.text+0x1192bf): Section mismatch in reference from the function devtmpfsd() to the variable .init.data:setup_done
The function devtmpfsd() references the variable __initdata setup_done.
This is often because devtmpfsd lacks a __initdata annotation or the annotation of setup_done is wrong.
WARNING: vmlinux.o(.text+0x119342): Section mismatch in reference from the function devtmpfsd() to the variable .init.data:setup_done
The function devtmpfsd() references the variable __initdata setup_done.
This is often because devtmpfsd lacks a __initdata annotation or the annotation of setup_done is wrong.
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
---
drivers/base/devtmpfs.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/base/devtmpfs.c b/drivers/base/devtmpfs.c
index 6d678c9..38e6464 100644
--- a/drivers/base/devtmpfs.c
+++ b/drivers/base/devtmpfs.c
@@ -376,7 +376,7 @@ int devtmpfs_mount(const char *mntdir)
return err;
}
-static __initdata DECLARE_COMPLETION(setup_done);
+static DECLARE_COMPLETION(setup_done);
static int handle(const char *name, mode_t mode, struct device *dev)
{
--
1.7.6.153.g78432
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 3/4] usb/host/pci-quirks.c: correct annotation of `ehci_dmi_nohandoff_table'
2011-07-21 17:16 [PATCH 1/4] eisa/pci_eisa.c: fix section mismatch Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done' Arnaud Lacombe
@ 2011-07-21 17:16 ` Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 4/4] drivers/base/devtmpfs.c: fix uninitialized variable Arnaud Lacombe
2 siblings, 0 replies; 6+ messages in thread
From: Arnaud Lacombe @ 2011-07-21 17:16 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnaud Lacombe, Sarah Sharp, linux-usb
ehci_bios_handoff() is marked __devinit, `ehci_dmi_nohandoff_table' should be
marked __devinitconst, not __initconst. This fixes the following section
mismatch:
WARNING: vmlinux.o(.devinit.text+0x4f08): Section mismatch in reference from the function ehci_bios_handoff() to the variable .init.rodata:ehci_dmi_nohandoff_table
The function __devinit ehci_bios_handoff() references a variable __initconst ehci_dmi_nohandoff_table.
If ehci_dmi_nohandoff_table is only used by ehci_bios_handoff then annotate ehci_dmi_nohandoff_table with a matching annotation.
Cc: Sarah Sharp <sarah.a.sharp@linux.intel.com>
Cc: linux-usb@vger.kernel.org
Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
---
drivers/usb/host/pci-quirks.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/usb/host/pci-quirks.c b/drivers/usb/host/pci-quirks.c
index a9d3159..31ad147 100644
--- a/drivers/usb/host/pci-quirks.c
+++ b/drivers/usb/host/pci-quirks.c
@@ -535,7 +535,7 @@ static void __devinit quirk_usb_handoff_ohci(struct pci_dev *pdev)
iounmap(base);
}
-static const struct dmi_system_id __initconst ehci_dmi_nohandoff_table[] = {
+static const struct dmi_system_id __devinitconst ehci_dmi_nohandoff_table[] = {
{
/* Pegatron Lucid (ExoPC) */
.matches = {
--
1.7.6.153.g78432
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 4/4] drivers/base/devtmpfs.c: fix uninitialized variable
2011-07-21 17:16 [PATCH 1/4] eisa/pci_eisa.c: fix section mismatch Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done' Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 3/4] usb/host/pci-quirks.c: correct annotation of `ehci_dmi_nohandoff_table' Arnaud Lacombe
@ 2011-07-21 17:16 ` Arnaud Lacombe
2 siblings, 0 replies; 6+ messages in thread
From: Arnaud Lacombe @ 2011-07-21 17:16 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnaud Lacombe, Greg Kroah-Hartman
drivers/base/devtmpfs.c: In function 'create_path':
drivers/base/devtmpfs.c:169:6: warning: 'err' may be used uninitialized in this function
Cc: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
---
drivers/base/devtmpfs.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/base/devtmpfs.c b/drivers/base/devtmpfs.c
index 38e6464..256c78a 100644
--- a/drivers/base/devtmpfs.c
+++ b/drivers/base/devtmpfs.c
@@ -166,7 +166,7 @@ static int create_path(const char *nodepath)
{
char *path;
char *s;
- int err;
+ int err = 0;
/* parent directories do not exist, create them */
path = kstrdup(nodepath, GFP_KERNEL);
--
1.7.6.153.g78432
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done'
2011-07-21 17:16 ` [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done' Arnaud Lacombe
@ 2011-07-25 1:19 ` Arnaud Lacombe
2011-08-01 23:27 ` Greg KH
0 siblings, 1 reply; 6+ messages in thread
From: Arnaud Lacombe @ 2011-07-25 1:19 UTC (permalink / raw)
To: linux-kernel; +Cc: Greg Kroah-Hartman, Al Viro
Hi,
I saw that devtmpfs stuff went on through Al Viro's tree, maybe I
should have Cc: him on this ?
Thanks,
- Arnaud
On Thu, Jul 21, 2011 at 1:16 PM, Arnaud Lacombe <lacombar@gmail.com> wrote:
> This fixes the following section mismatch issue:
>
> WARNING: vmlinux.o(.text+0x1192bf): Section mismatch in reference from the function devtmpfsd() to the variable .init.data:setup_done
> The function devtmpfsd() references the variable __initdata setup_done.
> This is often because devtmpfsd lacks a __initdata annotation or the annotation of setup_done is wrong.
>
> WARNING: vmlinux.o(.text+0x119342): Section mismatch in reference from the function devtmpfsd() to the variable .init.data:setup_done
> The function devtmpfsd() references the variable __initdata setup_done.
> This is often because devtmpfsd lacks a __initdata annotation or the annotation of setup_done is wrong.
>
> Cc: Greg Kroah-Hartman <gregkh@suse.de>
> Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
> ---
> drivers/base/devtmpfs.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/base/devtmpfs.c b/drivers/base/devtmpfs.c
> index 6d678c9..38e6464 100644
> --- a/drivers/base/devtmpfs.c
> +++ b/drivers/base/devtmpfs.c
> @@ -376,7 +376,7 @@ int devtmpfs_mount(const char *mntdir)
> return err;
> }
>
> -static __initdata DECLARE_COMPLETION(setup_done);
> +static DECLARE_COMPLETION(setup_done);
>
> static int handle(const char *name, mode_t mode, struct device *dev)
> {
> --
> 1.7.6.153.g78432
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done'
2011-07-25 1:19 ` Arnaud Lacombe
@ 2011-08-01 23:27 ` Greg KH
0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2011-08-01 23:27 UTC (permalink / raw)
To: Arnaud Lacombe; +Cc: linux-kernel, Greg Kroah-Hartman, Al Viro
On Sun, Jul 24, 2011 at 09:19:34PM -0400, Arnaud Lacombe wrote:
> Hi,
>
> I saw that devtmpfs stuff went on through Al Viro's tree, maybe I
> should have Cc: him on this ?
No need, I'll get it.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-08-01 23:28 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-21 17:16 [PATCH 1/4] eisa/pci_eisa.c: fix section mismatch Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 2/4] drivers/base/devtmpfs.c: correct annotation of `setup_done' Arnaud Lacombe
2011-07-25 1:19 ` Arnaud Lacombe
2011-08-01 23:27 ` Greg KH
2011-07-21 17:16 ` [PATCH 3/4] usb/host/pci-quirks.c: correct annotation of `ehci_dmi_nohandoff_table' Arnaud Lacombe
2011-07-21 17:16 ` [PATCH 4/4] drivers/base/devtmpfs.c: fix uninitialized variable Arnaud Lacombe
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox