lkml.org 
[lkml]   [2007]   [Jun]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [patch] use __asm__ and __volatile__ in i386/arm/s390 byteorder.h
Date
On Monday 18 June 2007, Christoph Hellwig wrote:
> On Mon, Jun 18, 2007 at 01:24:24AM +0200, Arnd Bergmann wrote:
> > On Monday 18 June 2007, Arjan van de Ven wrote:
> > > On Sun, 2007-06-17 at 18:33 -0400, Mike Frysinger wrote:
> > > > This changes asm() to __asm__() and volatile to __volatile__ so that
> > > > these headers can be used with gcc's -std=c99.
> > >
> > > hmm but the kernel doesn't use -std=c99...
> >
> > The byteorder headers are exported to user space through
> > include/asm-generic/Kbuild.asm, and they are used by a number
> > of other exported headers, so they should work with any
> > gcc flags that a user might want to use.
>
> No, they should not be exported and the headers using them
> should be fixed to not require this. Userspace has it's own
> endianess handling already.

user applications arent pulling these things in themselves ... you have to
also think of the cascading of header includes ... asm/byteorder.h gets
pulled in by many other things

if we want to scrub the userspace headers so that asm/byteorder.h isnt even
installed, that works for me as well, however i wouldnt discount the patch i
proposed on this alone ... the headers are inconsistent between using asm and
__asm__ and if anything, my patch makes them consistent
-mike
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2007-06-18 20:23    [W:0.057 / U:1.880 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site