Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1704074 > unrolled thread

Re: [PATCH] uapi: fix another asm/shmbuf.h userspace compilation error

Started byJames Hogan <james.hogan@imgtec.com>
First post2017-08-04 17:30 +0200
Last post2017-08-04 17:30 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH] uapi: fix another asm/shmbuf.h userspace compilation  error James Hogan <james.hogan@imgtec.com> - 2017-08-04 17:30 +0200

#1704074 — Re: [PATCH] uapi: fix another asm/shmbuf.h userspace compilation error

FromJames Hogan <james.hogan@imgtec.com>
Date2017-08-04 17:30 +0200
SubjectRe: [PATCH] uapi: fix another asm/shmbuf.h userspace compilation error
Message-ID<uaMI2-iG-7@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

On Thu, Mar 02, 2017 at 02:07:20PM +0100, Arnd Bergmann wrote:
> On Thu, Mar 2, 2017 at 1:46 AM, Dmitry V. Levin <ldv@altlinux.org> wrote:
> > Replace size_t with __kernel_size_t to fix asm/shmbuf.h userspace
> > compilation errors like this:
> >
> > /usr/include/asm-generic/shmbuf.h:28:2: error: unknown type name 'size_t'
> >   size_t   shm_segsz; /* size of segment (bytes) */
> >
> > x32 is the only architecture where sizeof(size_t) is less than
> > sizeof(__kernel_size_t), but as the kernel treats shm_segsz field as
> > __kernel_size_t anyway, UAPI should follow.  Thanks to little-endiannes
> > of x32 and 64-bit alignment of the field following shm_segsz, this
> > change doesn't break ABI, and the difference doesn't manifest itself
> > easily.
> >
> > Signed-off-by: Dmitry V. Levin <ldv@altlinux.org>
> 
> Acked-by: Arnd Bergmann <arnd@arndb.de>

Out of interest, is there a plan for merging these patches from Dmitry?

Cheers
James

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web