From: Andreas Gruenbacher <agruen@suse.de>
To: Eric Paris <eparis@redhat.com>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
Tvrtko Ursulin <tvrtko.ursulin@sophos.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Christoph Hellwig <hch@infradead.org>,
linux-fsdevel@vger.kernel.org
Subject: Re: Linux 2.6.36-rc7
Date: Fri, 8 Oct 2010 23:45:44 +0200 [thread overview]
Message-ID: <201010082345.45234.agruen@suse.de> (raw)
In-Reply-To: <1286555908.2682.62.camel@localhost.localdomain>
On Friday 08 October 2010 18:38:28 Eric Paris wrote:
> However, THIS is potentially a real ABI problem and something which
> deals with the interface. Alan seemed to lean towards pulling the
> syscalls. It is relatively easily solved without changing the interface
> or breaking userspace in 2.6.37. We use some set of the flags bits as a
> priority (we only use 2 of the 32 bits today so we have plenty) and
> order groups with highest priority first, 0 priority last, and 2+ groups
> with the same priority have unpredictable ordering. I'd then call
> priorities other than 0 a 2.6.37 feature. If we do it in flags I think
> that leaves us with say 8 bits and thus 255 priorities.
That's a possibility but it seems quite messy for a brand new system call.
I'd still pull the system call and work out the few remaining quirkses.
Thanks.
> ------
>
> I said I wouldn't refute your claims but I can't help myself on one
> account which I think might mislead people.
>
> * Some weaknesses in the interface design were only identified and fixed
> late in the -rc phase, changing the ABI. There may be more issues, like
> the priority discussion. This might leave us with a broken ABI we would
> need to support forever.
>
> Between rc2 and rc3 we switched the order and size of a couple of fields
> to help alignment, it did break ABI, but it wasn't an interface failing.
> See: 0fb85621df4f.
See, a weakness, not a failure. What I said.
The main issue that Andreas Schwab has pointed out (and which also lead to
this commit) was the packing of the structs which leads to inefficient code,
though. This hasn't been fixed but it still can be, in a backwards compatible
way.
Thanks,
Andreas
prev parent reply other threads:[~2010-10-08 21:45 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <AANLkTi=LsBNU+O2hqZUcM2nYM_ze6qPq3thwSZBMtY_v@mail.gmail.com>
[not found] ` <20101007190741.2dc62626@lxorguk.ukuu.org.uk>
[not found] ` <1286473768.2656.21.camel@dhcp231-98.rdu.redhat.com>
2010-10-08 12:06 ` Linux 2.6.36-rc7 Andreas Gruenbacher
2010-10-08 16:33 ` David Daney
2010-10-08 21:50 ` Andreas Gruenbacher
2010-10-08 21:59 ` H. Peter Anvin
2010-10-11 22:13 ` fanotify: disable fanotify syscalls Eric Paris
2010-10-08 16:38 ` Linux 2.6.36-rc7 Eric Paris
2010-10-08 21:45 ` Andreas Gruenbacher [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=201010082345.45234.agruen@suse.de \
--to=agruen@suse.de \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=eparis@redhat.com \
--cc=hch@infradead.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=tvrtko.ursulin@sophos.com \
/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).