[Linux-ia64] [PATCH][RFC][CFT] remove global errno from the kernel, make _syscallX kernel-only

[Linux-ia64] [PATCH][RFC][CFT] remove global errno from the kernel, make _syscallX kernel-only

[Lennert Buytenhek]

(Apologies for the massive cross-post, but this is a matter that concerns
all architectures.  For archs that don't have a maintainer listed in
MAINTAINERS I grabbed a random email address from arch/$arch/kernel/*.
If you're not the person to answer this for your $arch, please forward
to an appropriate person/list.)


Hi,

As I mentioned in my email to l-k with subject '[PATCH][RFC] global
errno considered harmful' earlier today, having a global errno in the
kernel doesn't really make sense.

Referenced patch [1] deletes all mention of a global errno from the
kernel, fixes up a very small number of callers that were depending on
it, and fixes up the syscall helpers in include/asm-$arch/unistd.h not
to write an error code to errno in case of error anymore.

This subtly breaks userspace code that uses these helpers, but the general
consensus seems to be that userspace code shouldn't be touching these in
the first place.  Patch [2] fixes up asm-$arch/unistd.h to only define
_syscallX in case __KERNEL_SYSCALLS__ is defined, to try and actively
break userspace (ab)users of this code (thanks to Ralf Baechle for
suggesting I should do something along these lines).

What I would like to know from each architecture team:
- What is your arch's policy on userspace usage of asm/unistd.h, and
  consequently, what is your opinion on the goal these patches
  aim for?
- Are the changes I made in [1] and [2] for your $arch technically
  correct?
Please CC me on replies as I'm not on any of the lists posted to.

My intention is to push these to Linus for 2.5 if everyone agrees.
They're probably too intrusive for 2.4 (although I'd love people
to convince me otherwise).


cheers,
Lennert

[1] http://www.math.leidenuniv.nl/~buytenh/errno_ectomy-1.diff
[2] http://www.math.leidenuniv.nl/~buytenh/errno_ectomy-1-to-2.diff

Re: [Linux-ia64] [PATCH][RFC][CFT] remove global errno from the kernel, make _syscallX kernel-only

[Andreas Schwab]

Lennert Buytenhek <buytenh@gnu.org> writes:

|> What I would like to know from each architecture team:
|> - What is your arch's policy on userspace usage of asm/unistd.h, and
|>   consequently, what is your opinion on the goal these patches
|>   aim for?

I very much appreciate the removal of the _syscallX macros.  FWIW, the
ia64 version of these macros never worked in userspace anyway.

|> - Are the changes I made in [1] and [2] for your $arch technically
|>   correct?

The m68k changes look ok.

|> Please CC me on replies as I'm not on any of the lists posted to.
|> 
|> My intention is to push these to Linus for 2.5 if everyone agrees.
|> They're probably too intrusive for 2.4 (although I'd love people
|> to convince me otherwise).

I wouldn't mind putting it into 2.4.

Andreas.

-- 
Andreas Schwab                                  "And now for something
Andreas.Schwab@suse.de				completely different."
SuSE Labs, SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg
Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5

Re: [Linux-ia64] [PATCH][RFC][CFT] remove global errno from the kernel, make _syscallX kernel-only

[David Mosberger]

>>>>> On Sun, 30 Dec 2001 22:05:00 -0500, Lennert Buytenhek <buytenh@gnu.org> said:

  Lennert> What I would like to know from each architecture team: -
  Lennert> What is your arch's policy on userspace usage of
  Lennert> asm/unistd.h, and consequently, what is your opinion on the
  Lennert> goal these patches aim for?  - Are the changes I made in
  Lennert> [1] and [2] for your $arch technically correct?  Please CC
  Lennert> me on replies as I'm not on any of the lists posted to.

  Lennert> My intention is to push these to Linus for 2.5 if everyone
  Lennert> agrees.  They're probably too intrusive for 2.4 (although
  Lennert> I'd love people to convince me otherwise).

The patch looks good to me as far as IA-64 Linux is concerned.  The
syscall() macros exported by asm-ia64/unistd.h were never supported in
user level.

Thanks,

	--david