Re: Section mismatch warnings




WARNING: vmlinux.o(.init.text+0x29d40): Section mismatch: reference to .exit.text:wbsd_release_resources (between 'wbsd_init' and 'wbsd_probe')
WARNING: vmlinux.o(.init.text+0x29d49): Section mismatch: reference to .exit.text:wbsd_free_mmc (between 'wbsd_init' and 'wbsd_probe')
WARNING: vmlinux.o(.init.text+0x29f28): Section mismatch: reference to .exit.text:wbsd_free_mmc (between 'wbsd_init' and 'wbsd_probe')
When a function is marked __exit and used from an init function then the safe choice
is to just remove the __exit marking. Usually this is small functions used
only in error situations.
Should be trivial too.

I'll look at those too

Sorry - no patch this time. Could you try to look at it yorself then I will review.

Ok I'll try.


The warnings gone with this patch but I'm not really sure it is correct.


drivers/mmc/host/wbsd.c | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/mmc/host/wbsd.c b/drivers/mmc/host/wbsd.c
index e0c9808..9bf2a87 100644
--- a/drivers/mmc/host/wbsd.c
+++ b/drivers/mmc/host/wbsd.c
@@ -1266,7 +1266,7 @@ static int __devinit wbsd_alloc_mmc(struct device *dev)
return 0;
}

-static void __devexit wbsd_free_mmc(struct device *dev)
+static void wbsd_free_mmc(struct device *dev)
{
struct mmc_host *mmc;
struct wbsd_host *host;
@@ -1358,7 +1358,7 @@ static int __devinit wbsd_request_region(struct wbsd_host *host, int base)
return 0;
}

-static void __devexit wbsd_release_regions(struct wbsd_host *host)
+static void wbsd_release_regions(struct wbsd_host *host)
{
if (host->base)
release_region(host->base, 8);
@@ -1434,7 +1434,7 @@ err:
"Falling back on FIFO.\n", dma);
}

-static void __devexit wbsd_release_dma(struct wbsd_host *host)
+static void wbsd_release_dma(struct wbsd_host *host)
{
if (host->dma_addr) {
dma_unmap_single(mmc_dev(host->mmc), host->dma_addr,
@@ -1484,7 +1484,7 @@ static int __devinit wbsd_request_irq(struct wbsd_host *host, int irq)
return 0;
}

-static void __devexit wbsd_release_irq(struct wbsd_host *host)
+static void wbsd_release_irq(struct wbsd_host *host)
{
if (!host->irq)
return;
@@ -1535,7 +1535,7 @@ static int __devinit wbsd_request_resources(struct wbsd_host *host,
* Release all resources for the host.
*/

-static void __devexit wbsd_release_resources(struct wbsd_host *host)
+static void wbsd_release_resources(struct wbsd_host *host)
{
wbsd_release_dma(host);
wbsd_release_irq(host);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: Section mismatch warnings
    ... is to just remove the __exit marking. ... only in error situations. ... struct wbsd_host *host; ... * Release all resources for the host. ...
    (Linux-Kernel)
  • Re: Section mismatch warnings
    ... is to just remove the __exit marking. ... only in error situations. ... struct wbsd_host *host; ... * Release all resources for the host. ...
    (Linux-Kernel)
  • [PATCH] Remove trailing whitespace in wbsd
    ... static inline void wbsd_unlock_config(struct wbsd_host* host) ... static void wbsd_request_end(struct wbsd_host* host, ... The buffer for DMA is only 64 kB. ...
    (Linux-Kernel)
  • Re: [PATCH] Remove trailing whitespace in wbsd
    ... static inline void wbsd_unlock_config(struct wbsd_host* host) ... static void wbsd_request_end(struct wbsd_host* host, ... The buffer for DMA is only 64 kB. ...
    (Linux-Kernel)
  • [PATCH] new uvisor.c driver
    ... Palm Tungsten T ... * This code is derived from software contributed to The NetBSD Foundation ... * are available to be transfered to the host for the specified endpoint. ... struct uvisor_palm_connection_info { ...
    (freebsd-current)