From: Roland McGrath <roland@redhat.com>
To: James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>,
Helge Deller <deller@gmx.de>,
linux-parisc <linux-parisc@vger.kernel.org>,
linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: kernel segv with 2.6.31-rc6 ?
Date: Wed, 19 Aug 2009 11:10:41 -0700 (PDT) [thread overview]
Message-ID: <20090819181041.7215E4730F@magilla.sf.frob.com> (raw)
In-Reply-To: James Bottomley's message of Tuesday, 18 August 2009 19:38:36 -0600 <1250645916.18426.12.camel@mulgrave.site>
> Using ET_DYN would have made our life easier when trying to code the
> kernel module loader as well. The basic problem is, of course, that
> this is simple on an x86, so it didn't matter that much for the initial
> implementation. It just becomes less simple on anything else.
There are generic ways that ET_DYN is better for all, too.
A little birdie told me it was all the fault of mips.
Anyway, water under the bridge.
> Well, since they're still making parisc machines, I assume you mean
> further back than when the production lines knocked off today?
I'm pretty sure you knew what I meant.
> So that leaves us stuck with the current implementation and still
> needing a solution for the duplicate section names?
Something like that. Your fix makes module loading work again, so we
could just let /sys/module/*/{sections,notes}/ be missing on parisc
and see how long it takes anybody to complain, assuming you talk Rusty
out of his preference to change those error cases to refuse to load
the module. OTOH, you said earlier that it was only some mistake that
produced multiple same-named sections to begin with, so you could just
try to fix that and then forget about it.
Finally, we could decide to really support the general case including
duplicate section names in these features. That's really not very
hard to do, if we just completely change the interfaces for those
features. e.g. instead of a /sys/module/*/sections directory, it
could just be a single file that contains lines of:
SHNDX 0xADDR NAME
For /sys/module/*/notes, no consumer really cares about the section
distinctions at all. It would be easy enough just to have a single
pseudo-file that delivers all note sections concatenated together.
(In practice, there is usually only one note section at most anyway.)
Of course, such changes would be a new incompatible change to the
userland sysfs ABI, require compatibility concerns, etc. It seems
unlikely anyone really wants to bother with all that. It would be
more natural presentation of the info IMHO, but it hardly matters.
Thanks,
Roland
next prev parent reply other threads:[~2009-08-19 18:10 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <4A89CC4D.5040801@gmx.de>
2009-08-17 22:49 ` James Bottomley
2009-08-17 23:54 ` Roland McGrath
2009-08-18 3:18 ` Rusty Russell
2009-08-18 3:55 ` James Bottomley
2009-08-18 5:06 ` Roland McGrath
2009-08-19 0:09 ` James Bottomley
2009-08-19 0:14 ` Roland McGrath
2009-08-19 0:54 ` James Bottomley
2009-08-19 1:31 ` Roland McGrath
2009-08-19 1:38 ` James Bottomley
2009-08-19 18:10 ` Roland McGrath [this message]
2009-08-25 7:59 ` Rusty Russell
2009-08-25 19:24 ` Roland McGrath
2009-08-26 12:20 ` Rusty Russell
2009-08-26 17:54 ` Roland McGrath
2009-08-20 11:51 ` Helge Deller
2009-08-20 12:25 ` Helge Deller
2009-08-20 18:55 ` John David Anglin
2009-08-20 21:45 ` Helge Deller
2009-08-20 21:50 ` James Bottomley
2009-08-20 22:07 ` Roland McGrath
2009-08-20 23:01 ` John David Anglin
2009-08-20 23:23 ` Roland McGrath
2009-08-21 0:03 ` John David Anglin
2009-08-25 21:49 ` Mike Frysinger
2009-08-25 7:37 ` Rusty Russell
2009-08-20 11:58 ` Helge Deller
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=20090819181041.7215E4730F@magilla.sf.frob.com \
--to=roland@redhat.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=deller@gmx.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parisc@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
all inboxes | Powered by JetHome®