Messages in this thread | | | Date | Wed, 9 Oct 2002 15:10:08 -0700 (PDT) | From | Linus Torvalds <> | Subject | Re: [PATCH] 2.5.41 s390 (8/8): 16 bit uid/gids. |
| |
On Wed, 9 Oct 2002, Tim Hockin wrote: > > > > In other words, that __UID16 thing should be a real CONFIG_XXX option. > > Because Sparc64/s390x/? still need to tell highuid.h to do macro magic for > NEW_TO_OLD_UID() and friends in some places and not others. A CONFIG_XXX > applies all the time to all files.
If __UID16 works, then renaming it to CONFIG_UID16_ONLY _must_ also work.
I don't understand your argument about other architectures. I'm claiming that __UID16 is a config option, and that it must be renamed to _show_ that it is a config option.
If the renaming results in code that doesn't work, then the code didn't work in the first place or your cpp is incredibly broken.
WE MUST NOT HAVE CONFIG OPTIONS THAT ARE HIDDEN AND CALLED __UID16! That's my whole point.
Linus
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |