Re: [RFC] Introduce __ARCH_WANT_SYS_SYSFS

["Will Newton" <will.newton-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

On Wed, Apr 23, 2008 at 6:59 PM, Mike Frysinger <vapier.adi-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>
> On Wed, Apr 23, 2008 at 12:05 PM, Mike Frysinger <vapier.adi-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>  > On Wed, Apr 23, 2008 at 11:50 AM, Will Newton <will.newton-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>  >  > On Wed, Apr 23, 2008 at 4:40 PM, Kyle McMartin <kyle-pfcGkIkfWfAsA/PxXw9srA@public.gmane.org> wrote:
>  >  >  > On Wed, Apr 23, 2008 at 03:36:23PM +0100, Will Newton wrote:
>  >  >  >  > +config ARCH_HAS_SYS_SYSFS
>  >  >  >  > +     bool
>  >  >  >  > +     default y
>  >  >  >  > +
>  >  >  >  >  source "init/Kconfig"
>  >  >  >  >
>  >  >  >
>  >  >  >  Sorry, I meant something more like
>  >  >  >
>  >  >  >
>  >  >  >  config ARCH_HAS_SYS_SYSFS
>  >  >  >         def_bool !BLACKFIN
>  >  >  >         help
>  >  >  >           Obsolete sys_sysfs syscall
>  >  >  >
>  >  >  >  in init/Kconfig
>  >  >  >
>  >  >  >  But, it's your patch, you can do it however you like. :)
>  >  >
>  >  >  That's definitely shorter - but it feels a bit more like #ifdef
>  >  >  CONFIG_BLACKFIN which is explicitly what I don't want to do, because
>  >  >  I'm not actually interested in blackfin. ;-)
>  >
>  >  i'd have to agree that updating asm/unistd.h fits better with existing
>  >  paradigm.  if we want to talk about converting *all cases* to Kconfig,
>  >  we can do it in a separate thread.  splitting the design between two
>  >  different files is simply confusing to everyone involved as they spend
>  >  their time going "well which way am *i* supposed to do it".
>
>  thinking about this some more ... we actually have three choices here,
>  not just two.  checksyscalls.sh introduced a new form in asm/unistd.h:
>  #define __IGNORE_sysfs
>  perhaps we should be unifying the __ARCH_WANT_XXX and the __IGNORE_XXX

I'm not sure this would be possible. IGNORE is saying "please don't
warn me about the fact that I don't define define this syscall".
ARCH_WANT is saying "give me the generic implementation of this
syscall". You should never define IGNORE and ARCH_WANT for the same
syscall, but you may want to have neither - you have defined the
syscall but you don't want the generic version.

>  -mike
>