All of lore.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: "H. Peter Anvin" <hpa@zytor.com>,
	Andi Kleen <andi@firstfloor.org>,
	David Woodhouse <dwmw2@infradead.org>,
	akpm@osdl.org, linux-kernel@vger.kernel.org,
	linuxppc-dev@ozlabs.org, sam@ravnborg.org,
	linux-arch@vger.kernel.org
Subject: Re: [PATCH] Complain about missing system calls.
Date: Fri, 09 Mar 2007 12:52:15 -0800	[thread overview]
Message-ID: <45F1C8FF.9040600@zytor.com> (raw)
In-Reply-To: <20070309201308.GB8444@flint.arm.linux.org.uk>

Russell King wrote:
> On Fri, Mar 09, 2007 at 11:40:08AM -0800, H. Peter Anvin wrote:
>> Jan-Benedict Glaw wrote:
>>> Not everybody has a simple indexed list of pointers :)  For example,
>>> for vax-linux, we use a struct per syscall with the expected number of
>>> on-stack longwords for the call.
>>>
>>> So if something "new" is coming up, please keep in mind that it should
>>> be flexible enough to represent that. :)
>>>
>> I discussed with Al Viro a while ago about using something like the 
>> SYSCALLS.def file from klibc as the source format for the system calls. 
>>  That would deal very flexibly with almost all kinds of stub generation.
> 
> Hopefully with this idea in place, we can spot new syscalls before
> the final release of the kernel (maybe kautobuild can help there)
> and fix any silly system call argument ordering which requires
> different architectures to have different syscall prototypes (eg,
> sys_arm_fadvise64_64 vs sys_fadvise64_64, sys_arm_sync_file_range vs
> sys_sync_file_range).

That would definitely be nice.

> Otherwise the SYSCALLS.def file will probably end up being full of
> ifdefs.

... which exactly mirrors the pain and suffering which libc maintainers 
have to deal with.  The amount of time I spent per line of code in klibc 
is quite high, in part because I wanted it to be as self-porting as was 
possible.  I've really tried to avoid arch-specific hacks, and yet there 
are more there than there should be, in large part because of unusable 
or missing kernel header exports.

	-hpa

  reply	other threads:[~2007-03-09 20:54 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-08 23:01 [PATCH] Complain about missing system calls David Woodhouse
2007-03-09  0:14 ` David Miller
2007-03-09  0:14   ` David Miller
2007-03-09  0:18   ` David Woodhouse
2007-03-09  0:18     ` David Woodhouse
2007-03-20 12:12   ` Sam Ravnborg
2007-03-20 12:12     ` Sam Ravnborg
2007-03-09  3:31 ` Anton Blanchard
2007-03-09  3:31   ` Anton Blanchard
2007-03-09  8:43 ` Russell King
2007-03-09  8:43   ` Russell King
2007-03-09 16:11 ` Andi Kleen
2007-03-09 16:11   ` Andi Kleen
2007-03-09 16:38   ` Jan-Benedict Glaw
2007-03-09 16:38     ` Jan-Benedict Glaw
2007-03-09 19:00     ` Andi Kleen
2007-03-09 19:35       ` Jan-Benedict Glaw
2007-03-09 19:35         ` Jan-Benedict Glaw
2007-03-10  9:58       ` Martin Schwidefsky
2007-03-10  9:58         ` Martin Schwidefsky
2007-03-09 19:40     ` H. Peter Anvin
2007-03-09 20:13       ` Russell King
2007-03-09 20:13         ` Russell King
2007-03-09 20:52         ` H. Peter Anvin [this message]
2007-03-09 16:47   ` Benjamin Herrenschmidt
2007-03-09 16:47     ` Benjamin Herrenschmidt
2007-03-09 18:54     ` Andi Kleen
2007-03-09 18:54       ` Andi Kleen
2007-03-10  9:51       ` Martin Schwidefsky
2007-03-10  9:51         ` Martin Schwidefsky
2007-03-09 17:20   ` David Woodhouse
2007-03-09 17:20     ` David Woodhouse
2007-03-19 23:42 ` Andrew Morton
2007-03-19 23:42   ` Andrew Morton
2007-03-20  7:43   ` David Woodhouse
2007-03-20  7:43     ` David Woodhouse
2007-03-20 10:51     ` David Howells
2007-03-20 10:51       ` David Howells
2007-03-20 10:56     ` Russell King
2007-03-20 10:56       ` Russell King
2007-03-21 11:25   ` David Woodhouse
2007-03-21 11:25     ` David Woodhouse

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=45F1C8FF.9040600@zytor.com \
    --to=hpa@zytor.com \
    --cc=akpm@osdl.org \
    --cc=andi@firstfloor.org \
    --cc=dwmw2@infradead.org \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=sam@ravnborg.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 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.