From mboxrd@z Thu Jan 1 00:00:00 1970 From: Palmer Dabbelt Subject: Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h Date: Mon, 14 Sep 2015 15:50:12 -0700 (PDT) Message-ID: <55f74f24.a2eb420a.57553.fffff8c5@mx.google.com> References: <18247.1441883885@warthog.procyon.org.uk> Return-path: In-Reply-To: <18247.1441883885@warthog.procyon.org.uk> Sender: linux-arch-owner@vger.kernel.org Cc: dhowells@redhat.com, arnd@arndb.de, 3chas3@gmail.com, hpa@zytor.com, mingo@redhat.com, plagnioj@jcrosoft.com, jikos@kernel.org, linux-api@vger.kernel.org, linux-arch@vger.kernel.org, linux-atm-general@lists.sourceforge.net, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, tglx@linutronix.de, tomi.valkeinen@ti.com, x86@kernel.org List-Id: linux-api@vger.kernel.org On Thu, 10 Sep 2015 04:18:05 PDT (-0700), dhowells@redhat.com wrote: > David Howells wrote: > >> Rather than iterating through all the rest of your patches and saying the same >> thing, if there's something in a UAPI header that needs wrapping in __KERNEL__ >> to exclude it from userspace's use, then it should be transferred to the >> non-UAPI variant of that header (which should #include the UAPI variant). > > I should mention that there is the odd case where this is difficult to > achieve. See include/uapi/linux/acct.h for an example... OK, sorry about that. I'm submitting a v3 that should fix these problems. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f47.google.com ([209.85.220.47]:34063 "EHLO mail-pa0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751823AbbINWuO (ORCPT ); Mon, 14 Sep 2015 18:50:14 -0400 Received: by padhy16 with SMTP id hy16so155859956pad.1 for ; Mon, 14 Sep 2015 15:50:13 -0700 (PDT) Message-ID: <55f74f24.a2eb420a.57553.fffff8c5@mx.google.com> Date: Mon, 14 Sep 2015 15:50:12 -0700 (PDT) From: Palmer Dabbelt Subject: Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h In-Reply-To: <18247.1441883885@warthog.procyon.org.uk> Sender: linux-arch-owner@vger.kernel.org List-ID: To: dhowells@redhat.com Cc: arnd@arndb.de, 3chas3@gmail.com, hpa@zytor.com, mingo@redhat.com, plagnioj@jcrosoft.com, jikos@kernel.org, linux-api@vger.kernel.org, linux-arch@vger.kernel.org, linux-atm-general@lists.sourceforge.net, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, tglx@linutronix.de, tomi.valkeinen@ti.com, x86@kernel.org Message-ID: <20150914225012.oJia4d-clwyykZepHqMVsWofsGaFLwZvNgTwh_j1mpA@z> On Thu, 10 Sep 2015 04:18:05 PDT (-0700), dhowells@redhat.com wrote: > David Howells wrote: > >> Rather than iterating through all the rest of your patches and saying the same >> thing, if there's something in a UAPI header that needs wrapping in __KERNEL__ >> to exclude it from userspace's use, then it should be transferred to the >> non-UAPI variant of that header (which should #include the UAPI variant). > > I should mention that there is the odd case where this is difficult to > achieve. See include/uapi/linux/acct.h for an example... OK, sorry about that. I'm submitting a v3 that should fix these problems. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Palmer Dabbelt Date: Mon, 14 Sep 2015 22:50:12 +0000 Subject: Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h Message-Id: <55f74f24.a2eb420a.57553.fffff8c5@mx.google.com> List-Id: References: <18247.1441883885@warthog.procyon.org.uk> In-Reply-To: <18247.1441883885@warthog.procyon.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: dhowells@redhat.com, arnd@arndb.de, 3chas3@gmail.com, hpa@zytor.com, mingo@redhat.com, plagnioj@jcrosoft.com, jikos@kernel.org, linux-api@vger.kernel.org, linux-arch@vger.kernel.org, linux-atm-general@lists.sourceforge.net, linux-fbdev@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, tglx@linutronix.de, tomi.valkeinen@ti.com, x86@kernel.org On Thu, 10 Sep 2015 04:18:05 PDT (-0700), dhowells@redhat.com wrote: > David Howells wrote: > >> Rather than iterating through all the rest of your patches and saying the same >> thing, if there's something in a UAPI header that needs wrapping in __KERNEL__ >> to exclude it from userspace's use, then it should be transferred to the >> non-UAPI variant of that header (which should #include the UAPI variant). > > I should mention that there is the odd case where this is difficult to > achieve. See include/uapi/linux/acct.h for an example... OK, sorry about that. I'm submitting a v3 that should fix these problems. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752525AbbINWuT (ORCPT ); Mon, 14 Sep 2015 18:50:19 -0400 Received: from mail-pa0-f44.google.com ([209.85.220.44]:35591 "EHLO mail-pa0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751667AbbINWuO (ORCPT ); Mon, 14 Sep 2015 18:50:14 -0400 Message-ID: <55f74f24.a2eb420a.57553.fffff8c5@mx.google.com> X-Google-Original-Message-ID: Mime-Version: 1.0 (MHng) Date: Mon, 14 Sep 2015 15:50:12 -0700 (PDT) X-Google-Original-Date: Mon, 14 Sep 2015 15:50:08 PDT (-0700) From: Palmer Dabbelt To: dhowells@redhat.com CC: dhowells@redhat.com CC: arnd@arndb.de CC: 3chas3@gmail.com CC: hpa@zytor.com CC: mingo@redhat.com CC: plagnioj@jcrosoft.com CC: jikos@kernel.org CC: linux-api@vger.kernel.org CC: linux-arch@vger.kernel.org CC: linux-atm-general@lists.sourceforge.net CC: linux-fbdev@vger.kernel.org CC: linux-kernel@vger.kernel.org CC: netdev@vger.kernel.org CC: tglx@linutronix.de CC: tomi.valkeinen@ti.com CC: x86@kernel.org Subject: Re: [PATCH] Remove #ifdef CONFIG_64BIT from all asm-generic/fcntl.h In-Reply-To: <18247.1441883885@warthog.procyon.org.uk> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 10 Sep 2015 04:18:05 PDT (-0700), dhowells@redhat.com wrote: > David Howells wrote: > >> Rather than iterating through all the rest of your patches and saying the same >> thing, if there's something in a UAPI header that needs wrapping in __KERNEL__ >> to exclude it from userspace's use, then it should be transferred to the >> non-UAPI variant of that header (which should #include the UAPI variant). > > I should mention that there is the odd case where this is difficult to > achieve. See include/uapi/linux/acct.h for an example... OK, sorry about that. I'm submitting a v3 that should fix these problems.