From: Jeff Mahoney <jeffm@suse.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: Rusty Russell <rusty@rustcorp.com.au>, linux-kernel@vger.kernel.org
Subject: Re: [patch 6/6] virtio: fix section mismatch warnings
Date: Wed, 13 Jan 2010 21:01:47 -0500 [thread overview]
Message-ID: <4B4E7B0B.6050305@suse.com> (raw)
In-Reply-To: <20100113170744.b45c9220.akpm@linux-foundation.org>
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 01/13/2010 08:07 PM, Andrew Morton wrote:
> On Wed, 13 Jan 2010 21:27:26 +1030
> Rusty Russell <rusty@rustcorp.com.au> wrote:
>
>> On Wed, 13 Jan 2010 11:19:01 am Jeff Mahoney wrote:
>>> This patch fixes the following warnings by renaming the driver structures
>>> to be suffixed with _driver.
>>>
>>> WARNING: drivers/virtio/virtio_balloon.o(.data+0x88): Section mismatch in reference from the variable virtio_balloon to the function .devexit.text:virtballoon_remove()
>>>
>>> WARNING: drivers/char/hw_random/virtio-rng.o(.data+0x88): Section mismatch in reference from the variable virtio_rng to the function .devexit.text:virtrng_remove()
>>>
>>> Signed-off-by: Jeff Mahoney <jeffm@suse.com>
>>
>> Acked-by: Rusty Russell <rusty@rustcorp.com.au>
>>
>> Andrew, will you take these?
>>
>
> umm, sure. But I wish I knew how the patch worked. Is there some
> magic sauce which moves a variable called foo_driver into a different
> section or what?
>
> And if so, what was I doing when that got done :(
It doesn't move it into a different section, it just shuts the warning
up. modpost has a whitelist it uses to ignore the mismatch warning.
- -Jeff
- --
Jeff Mahoney
SUSE Labs
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.12 (GNU/Linux)
Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org/
iEYEARECAAYFAktOewsACgkQLPWxlyuTD7IGKgCfSETonDGCsM/AgsUmdGtvSzCi
omQAn3tMc0Ol6PkpaX0Qf6+tvcUnlqcU
=jUja
-----END PGP SIGNATURE-----
prev parent reply other threads:[~2010-01-14 2:02 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-13 0:48 [patch 0/6] Section mismatch fixes Jeff Mahoney
2010-01-13 0:48 ` [patch 2/6] hugetlb: Fix section mismatches Jeff Mahoney
2010-01-13 0:48 ` Jeff Mahoney
2010-01-13 9:28 ` David Rientjes
2010-01-13 9:28 ` David Rientjes
2010-01-13 14:29 ` Jeff Mahoney
2010-01-13 14:29 ` Jeff Mahoney
2010-01-13 15:03 ` [patch] hugetlb: Fix section mismatches #2 Jeff Mahoney
2010-01-13 15:03 ` Jeff Mahoney
2010-01-13 20:51 ` David Rientjes
2010-01-13 20:51 ` David Rientjes
2010-01-13 15:40 ` [patch 2/6] hugetlb: Fix section mismatches Lee Schermerhorn
2010-01-13 15:40 ` Lee Schermerhorn
2010-01-13 15:42 ` Jeff Mahoney
2010-01-13 15:42 ` Jeff Mahoney
2010-01-13 0:48 ` [patch 4/6] acpi: Fix section mismatch Jeff Mahoney
2010-01-13 3:58 ` Alex Chiang
2010-01-13 0:49 ` [patch 6/6] virtio: fix section mismatch warnings Jeff Mahoney
2010-01-13 10:57 ` Rusty Russell
2010-01-14 1:07 ` Andrew Morton
2010-01-14 2:01 ` Jeff Mahoney [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B4E7B0B.6050305@suse.com \
--to=jeffm@suse.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.