On Sat, 2007-11-17 at 08:42 +1100, James Morris wrote: > On Fri, 16 Nov 2007, Eric Paris wrote: > > > +#ifdef CONFIG_SECURITY > > + /* > > + * If a hint addr is less than mmap_min_addr change addr to be as > > + * low as possible but still greater than mmap_min_addr > > + */ > > + if (!(flags & MAP_FIXED) && ((void *)addr != NULL) && > > + (addr < mmap_min_addr)) > > + addr = PAGE_ALIGN(mmap_min_addr); > > +#endif > > Please make this a static inline which is optimized away with > !CONFIG_SECURITY. Where do you think these should go? You think I should instead have a static inline function in both mmap.c and nommu.c so the CONFIG check isn't scattered in this code but still in this file? Or did you actually want something in security.h so I only have the code once? -Eric -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
