From: Rusty Russell <rusty@rustcorp.com.au>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: "Luck, Tony" <tony.luck@intel.com>,
Dave Young <hidave.darkstar@gmail.com>,
Stephen Rothwell <sfr@canb.auug.org.au>,
"linux-next@vger.kernel.org" <linux-next@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>
Subject: Re: linux-next: Tree for June 3
Date: Sat, 5 Jun 2010 12:09:51 +0930 [thread overview]
Message-ID: <201006051209.51664.rusty@rustcorp.com.au> (raw)
In-Reply-To: <alpine.LFD.2.00.1006041548480.8175@i5.linux-foundation.org>
On Sat, 5 Jun 2010 08:27:13 am Linus Torvalds wrote:
>
> On Fri, 4 Jun 2010, Luck, Tony wrote:
> >
> > It gets rid of the oops. So that's good. Something is still
> > hokey in linux-next land though because no modules get loaded.
> > So no ehci/uhci available :-(
>
> So maybe the error (the one that caused us to exit and caused the oops due
> to the wrong return value) is the one that now causes it to not load.
>
> I note that ia64 has a pretty big/complex module_frob_arch_sections().
> Many architectures (like x86) has a trivial one ("return 0;"), and there
> might be some ordering differences in the setup that only matter with
> architectures that do odd things there..
Hmm, but at a glance nothing else in layout_and_allocate() can fail silently,
and ia64's module_arch_frob_sections() has a printk on fail.
But, the June 3 tree (beware dateline: we are talking next-20100603?)
had the "deref per-cpu ptr before allocating it" issue which was fixed
the following day.
There have been three bugfixes since then, too:
1) I have also thrown out the "clean up percpu" commit, which would fail
on any relocations in the percpu section.
2) Linus' error path fix (I did it slightly different for consistency)
3) SYSFS=n compile issue.
Since it's a weekend here already, we're not going to get another linux-next
until Monday.
Here's a tree with all current fixes: still no module load?
The following changes since commit ad8456361fa19068cf49b50a4f98e41b73c08e76:
Linus Torvalds (1):
Merge branch 'upstream-linus' of git://git.kernel.org/.../jgarzik/libata-dev
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux-2.6 module
Eric Dumazet (1):
module: module_unload_init() cleanup
Linus Torvalds (6):
module: Make the 'usage' lists be two-way
module: move find_module check to end
module: refactor load_module
module: refactor load_module part 2
module: reduce stack usage for each_symbol()
module: fix bne2 "gave up waiting for init of module libcrc32c"
Rusty Russell (18):
module: fix kdb's illicit use of struct module_use.
module: move sysfs exposure to end of load_module
module: Make module sysfs functions private.
module: make locking more fine-grained.
module: verify_export_symbols under the lock
module: fix bne2 "gave up waiting for init of module libcrc32c"
module: refactor load_module part 3
module: refactor load_module part 4
module: refactor load_module part 5
module: refactor out section header rewriting
module: kallsyms functions take struct load_info
module: layout_and_allocate
module: sysfs cleanup
module: fix sysfs cleanup for !CONFIG_SYSFS
module: pass load_info into other functions
module: move module args strndup_user to just before use
module: group post-relocation functions into post_relocation()
module: cleanup comments, remove noinline
include/linux/module.h | 44 +--
kernel/debug/kdb/kdb_main.c | 12 +-
kernel/module.c | 1362 ++++++++++++++++++++++++-------------------
3 files changed, 779 insertions(+), 639 deletions(-)
next prev parent reply other threads:[~2010-06-05 2:39 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-03 3:47 linux-next: Tree for June 3 Stephen Rothwell
2010-06-03 7:39 ` Dave Young
2010-06-03 8:00 ` Dave Young
2010-06-03 8:16 ` Dave Young
2010-06-03 12:52 ` Rusty Russell
2010-06-04 19:46 ` Tony Luck
2010-06-04 20:04 ` Linus Torvalds
2010-06-04 20:46 ` Luck, Tony
2010-06-04 22:05 ` Linus Torvalds
2010-06-04 22:50 ` Luck, Tony
2010-06-04 22:57 ` Linus Torvalds
2010-06-05 2:39 ` Rusty Russell [this message]
2010-06-05 2:51 ` Rusty Russell
2010-06-05 4:01 ` Rusty Russell
2010-06-07 18:16 ` Luck, Tony
2010-06-03 15:46 ` linux-next: Tree for June 3 (qlcnic) Randy Dunlap
2010-06-03 17:31 ` Anirban Chakraborty
[not found] ` <20100603134753.710a64b3.sfr-3FnU+UHB4dNDw9hX6IcOSA@public.gmane.org>
2010-06-03 15:55 ` linux-next: Tree for June 3 (iwlwifi) Randy Dunlap
2010-06-03 17:21 ` John W. Linville
[not found] ` <20100603172114.GA14597-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2010-06-03 17:42 ` Guy, Wey-Yi
2010-06-03 17:45 ` reinette chatre
2010-06-03 16:39 ` [PATCH -next] classmate-laptop: fix for RFKILL=m, CMPC=y Randy Dunlap
2010-06-09 20:02 ` Thadeu Lima de Souza Cascardo
2010-06-09 20:05 ` Randy Dunlap
2010-06-09 20:11 ` Matthew Garrett
-- strict thread matches above, loose matches on Subject: below --
2011-06-03 6:38 linux-next: Tree for June 3 Stephen Rothwell
2009-06-03 9:09 Stephen Rothwell
2008-06-03 8:30 Stephen Rothwell
2008-06-04 11:28 ` Dmitri Vorobiev
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=201006051209.51664.rusty@rustcorp.com.au \
--to=rusty@rustcorp.com.au \
--cc=hidave.darkstar@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=sfr@canb.auug.org.au \
--cc=tony.luck@intel.com \
--cc=torvalds@linux-foundation.org \
/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;
as well as URLs for NNTP newsgroup(s).