Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752305AbdCIVAy (ORCPT ); Thu, 9 Mar 2017 16:00:54 -0500 Received: from fieldses.org ([173.255.197.46]:59046 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751383AbdCIVAx (ORCPT ); Thu, 9 Mar 2017 16:00:53 -0500 Date: Thu, 9 Mar 2017 16:00:51 -0500 From: "J. Bruce Fields" To: "Dmitry V. Levin" Cc: Jeff Layton , linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] uapi: fix linux/nfsd/cld.h userspace compilation errors Message-ID: <20170309210051.GH3929@fieldses.org> References: <20170301001203.GA11188@altlinux.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170301001203.GA11188@altlinux.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2584 Lines: 69 Why aren't the unitX_t types OK here? Anyway, assuming this is right I'll apply for 4.12. (I'm assuming it's not urgent since this file's always been this way.) --b. On Wed, Mar 01, 2017 at 03:12:03AM +0300, Dmitry V. Levin wrote: > Include and consistently use types it provides > to fix the following linux/nfsd/cld.h userspace compilation errors: > > /usr/include/linux/nfsd/cld.h:40:2: error: unknown type name 'uint16_t' > uint16_t cn_len; /* length of cm_id */ > /usr/include/linux/nfsd/cld.h:46:2: error: unknown type name 'uint8_t' > uint8_t cm_vers; /* upcall version */ > /usr/include/linux/nfsd/cld.h:47:2: error: unknown type name 'uint8_t' > uint8_t cm_cmd; /* upcall command */ > /usr/include/linux/nfsd/cld.h:48:2: error: unknown type name 'int16_t' > int16_t cm_status; /* return code */ > /usr/include/linux/nfsd/cld.h:49:2: error: unknown type name 'uint32_t' > uint32_t cm_xid; /* transaction id */ > /usr/include/linux/nfsd/cld.h:51:3: error: unknown type name 'int64_t' > int64_t cm_gracetime; /* grace period start time */ > > Signed-off-by: Dmitry V. Levin > --- > include/uapi/linux/nfsd/cld.h | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/include/uapi/linux/nfsd/cld.h b/include/uapi/linux/nfsd/cld.h > index f14a9ab..ec26027 100644 > --- a/include/uapi/linux/nfsd/cld.h > +++ b/include/uapi/linux/nfsd/cld.h > @@ -22,6 +22,8 @@ > #ifndef _NFSD_CLD_H > #define _NFSD_CLD_H > > +#include > + > /* latest upcall version available */ > #define CLD_UPCALL_VERSION 1 > > @@ -37,18 +39,18 @@ enum cld_command { > > /* representation of long-form NFSv4 client ID */ > struct cld_name { > - uint16_t cn_len; /* length of cm_id */ > + __u16 cn_len; /* length of cm_id */ > unsigned char cn_id[NFS4_OPAQUE_LIMIT]; /* client-provided */ > } __attribute__((packed)); > > /* message struct for communication with userspace */ > struct cld_msg { > - uint8_t cm_vers; /* upcall version */ > - uint8_t cm_cmd; /* upcall command */ > - int16_t cm_status; /* return code */ > - uint32_t cm_xid; /* transaction id */ > + __u8 cm_vers; /* upcall version */ > + __u8 cm_cmd; /* upcall command */ > + __s16 cm_status; /* return code */ > + __u32 cm_xid; /* transaction id */ > union { > - int64_t cm_gracetime; /* grace period start time */ > + __s64 cm_gracetime; /* grace period start time */ > struct cld_name cm_name; > } __attribute__((packed)) cm_u; > } __attribute__((packed)); > -- > ldv