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: 14+ 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 9:28 ` David Rientjes
2010-01-13 14:29 ` Jeff Mahoney
2010-01-13 15:03 ` [patch] hugetlb: Fix section mismatches #2 Jeff Mahoney
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: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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox