On Mon, May 23, 2011 at 1:30 PM, Richard Weinberger <rich...@nod.at> wrote:
>
>  config RWSEM_XCHGADD_ALGORITHM
> -       def_bool X86_XADD
> +       def_bool X86_XADD && 64BIT
>
>  config RWSEM_GENERIC_SPINLOCK
> -       def_bool !X86_XADD
> +       def_bool !X86_XADD || !64BIT

May I suggest just changing that second one to

  config RWSEM_GENERIC_SPINLOCK
      def_bool !RWSEM_XCHGADD_ALGORITHM

because I had to look twice just to make sure that they are inverses
of each other.

So why not make that inverse relationship more obvious?

                         Linus

------------------------------------------------------------------------------
What Every C/C++ and Fortran developer Should Know!
Read this article and learn how Intel has extended the reach of its 
next-generation tools to help Windows* and Linux* C/C++ and Fortran 
developers boost performance applications - including clusters. 
http://p.sf.net/sfu/intel-dev2devmay
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Reply via email to