mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* Re: Is sys_sysfs used?
       [not found] <Pine.LNX.4.44.0306041124250.13077-100000@cherise.suse.lists.linux.kernel>
@ 2003-06-05  7:09 ` Andi Kleen
  0 siblings, 0 replies; 5+ messages in thread
From: Andi Kleen @ 2003-06-05  7:09 UTC (permalink / raw)
  To: Patrick Mochel; +Cc: linux-kernel

Patrick Mochel <mochel@osdl.org> writes:

> I see that only one architecture defines __NR_sysfs: x86-64, though it
> appears most architectures mention it in arch/*/kernel/entry.S (or 
> equivalent). 

That's because x86-64 has a merged entry.S/unistd.h. The other architectures
probably use some other name in unistd.h

I think it can be removed safely from the AMD64 64bit table at least.

-Andi

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Is sys_sysfs used?
@ 2003-06-08 13:38 Andries.Brouwer
  0 siblings, 0 replies; 5+ messages in thread
From: Andries.Brouwer @ 2003-06-08 13:38 UTC (permalink / raw)
  To: linux-kernel, mochel

    From: Patrick Mochel <mochel@osdl.org>

    I see that only one architecture defines __NR_sysfs: x86-64, though it
    appears most architectures mention it in arch/*/kernel/entry.S (or 
    equivalent).

There are definitions in include/asm-*/unistd.h.

    Is it used anymore? Would it be possible to remove it? 

There should be a good reason to change the kernel API.
Just "why not remove this probably unused call?" is not a good reason.

(It indeed is almost unused. A grep on a random source tree shows
a use in mount.c in busybox.)

Andries

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Is sys_sysfs used?
  2003-06-04 18:48 ` Richard B. Johnson
@ 2003-06-04 18:50   ` Patrick Mochel
  0 siblings, 0 replies; 5+ messages in thread
From: Patrick Mochel @ 2003-06-04 18:50 UTC (permalink / raw)
  To: Richard B. Johnson; +Cc: linux-kernel


> But what would you use as a place-holder?? There are lots
> of unused sys-calls (break, acct, lock, mpx, etc). You
> certainly can't be running out of numbers, and you certainly
> can't remove  a number and change everything else, you'll
> not get up, even with static-linked files!

Mark it as unused in unistd.h (like #232 for asm-i386), and implement it 
as sys_ni_syscall in the entry.S table. 

Al has already reminded me that it must be marked deprecated for a full 
version before it can be removed, but when/if it does, then that's what 
would happen until some lucky sucker took its place.


	-pat


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Is sys_sysfs used?
  2003-06-04 18:31 Patrick Mochel
@ 2003-06-04 18:48 ` Richard B. Johnson
  2003-06-04 18:50   ` Patrick Mochel
  0 siblings, 1 reply; 5+ messages in thread
From: Richard B. Johnson @ 2003-06-04 18:48 UTC (permalink / raw)
  To: Patrick Mochel; +Cc: linux-kernel

On Wed, 4 Jun 2003, Patrick Mochel wrote:

>
> In fs/filesystems.c, we have sys_sysfs:
>
> /*
>  * Whee.. Weird sysv syscall.
>  */
> asmlinkage long sys_sysfs(int option, unsigned long arg1, unsigned long arg2)
> {
> 	...
> }
>
> Which is, as advertised, kinda weird.
>
> I see that only one architecture defines __NR_sysfs: x86-64, though it
> appears most architectures mention it in arch/*/kernel/entry.S (or
> equivalent).
>
> Is it used anymore? Would it be possible to remove it?
>
>
> 	-pat

But what would you use as a place-holder?? There are lots
of unused sys-calls (break, acct, lock, mpx, etc). You
certainly can't be running out of numbers, and you certainly
can't remove  a number and change everything else, you'll
not get up, even with static-linked files!


Cheers,
Dick Johnson
Penguin : Linux version 2.4.20 on an i686 machine (797.90 BogoMips).
Why is the government concerned about the lunatic fringe? Think about it.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Is sys_sysfs used?
@ 2003-06-04 18:31 Patrick Mochel
  2003-06-04 18:48 ` Richard B. Johnson
  0 siblings, 1 reply; 5+ messages in thread
From: Patrick Mochel @ 2003-06-04 18:31 UTC (permalink / raw)
  To: linux-kernel


In fs/filesystems.c, we have sys_sysfs: 

/*
 * Whee.. Weird sysv syscall. 
 */
asmlinkage long sys_sysfs(int option, unsigned long arg1, unsigned long arg2)
{
	...
}

Which is, as advertised, kinda weird. 

I see that only one architecture defines __NR_sysfs: x86-64, though it
appears most architectures mention it in arch/*/kernel/entry.S (or 
equivalent). 

Is it used anymore? Would it be possible to remove it? 


	-pat


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2003-06-08 13:24 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <Pine.LNX.4.44.0306041124250.13077-100000@cherise.suse.lists.linux.kernel>
2003-06-05  7:09 ` Is sys_sysfs used? Andi Kleen
2003-06-08 13:38 Andries.Brouwer
  -- strict thread matches above, loose matches on Subject: below --
2003-06-04 18:31 Patrick Mochel
2003-06-04 18:48 ` Richard B. Johnson
2003-06-04 18:50   ` Patrick Mochel

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®