From mboxrd@z Thu Jan 1 00:00:00 1970 From: Firoz Khan Subject: Re: [PATCH v4 5/6] parisc: wire up rseq system call Date: Sat, 13 Oct 2018 11:12:12 +0530 Message-ID: References: <1539337442-3676-1-git-send-email-firoz.khan@linaro.org> <1539337442-3676-6-git-send-email-firoz.khan@linaro.org> <71ea9dc5-7502-40af-2022-b2196dd2d690@gmx.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <71ea9dc5-7502-40af-2022-b2196dd2d690@gmx.de> Sender: linux-kernel-owner@vger.kernel.org To: Helge Deller Cc: Arnd Bergmann , linux-parisc@vger.kernel.org, "James E . J . Bottomley" , Thomas Gleixner , Greg Kroah-Hartman , Philippe Ombredanne , Kate Stewart , y2038 Mailman List , Linux Kernel Mailing List , Linux-Arch , Deepa Dinamani , Marcin Juszkiewicz List-Id: linux-arch.vger.kernel.org Hi Arnd, Helge, On Sat, 13 Oct 2018 at 01:53, Helge Deller wrote: > > On 12.10.2018 13:52, Arnd Bergmann wrote: > > On Fri, Oct 12, 2018 at 12:16 PM Firoz Khan wrote: > >> > >> Hi Arnd, > >> > >> On Fri, 12 Oct 2018 at 15:26, Arnd Bergmann wrote: > >>> > >>> On Fri, Oct 12, 2018 at 11:45 AM Firoz Khan wrote: > >>>> > >>>> Wire up rseq system call requires an architecture specific > >>>> implementation as it not present now. > >>>> > >>>> Signed-off-by: Firoz Khan > >>> > >>> Didn't we conclude that this should be skipped? > >> > >> Helge told "I prefer to keep the warning for rseq for now. > >> It reminds me that we still may want the rseq syscall. > >> If the warning is a problem, you may simply add the __IGNORE_rseq define." > >> > >> I chose this option; I feel it looks clean > >> "# rseq requires an architecture specific implementation" in syscall.tbl > >> hopefully remind him to add rseq syscall implementation in this architecture. > > > > But he said he wants the warning instead. > > Correct. > Firoz, please simply drop this patch completely. > That way one will see generated warnings about the missing rseq (which is what > we want). Sure. will do! Firoz > > Thanks, > Helge From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-yw1-f66.google.com ([209.85.161.66]:36164 "EHLO mail-yw1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726125AbeJMNSL (ORCPT ); Sat, 13 Oct 2018 09:18:11 -0400 Received: by mail-yw1-f66.google.com with SMTP id e201-v6so5768027ywa.3 for ; Fri, 12 Oct 2018 22:42:23 -0700 (PDT) MIME-Version: 1.0 References: <1539337442-3676-1-git-send-email-firoz.khan@linaro.org> <1539337442-3676-6-git-send-email-firoz.khan@linaro.org> <71ea9dc5-7502-40af-2022-b2196dd2d690@gmx.de> In-Reply-To: <71ea9dc5-7502-40af-2022-b2196dd2d690@gmx.de> From: Firoz Khan Date: Sat, 13 Oct 2018 11:12:12 +0530 Message-ID: Subject: Re: [PATCH v4 5/6] parisc: wire up rseq system call Content-Type: text/plain; charset="UTF-8" Sender: linux-arch-owner@vger.kernel.org List-ID: To: Helge Deller Cc: Arnd Bergmann , linux-parisc@vger.kernel.org, "James E . J . Bottomley" , Thomas Gleixner , Greg Kroah-Hartman , Philippe Ombredanne , Kate Stewart , y2038 Mailman List , Linux Kernel Mailing List , Linux-Arch , Deepa Dinamani , Marcin Juszkiewicz Message-ID: <20181013054212.eOnF3eP8CmGmGE6P49E_MrylJy955yt5oxn1vWNRXu4@z> Hi Arnd, Helge, On Sat, 13 Oct 2018 at 01:53, Helge Deller wrote: > > On 12.10.2018 13:52, Arnd Bergmann wrote: > > On Fri, Oct 12, 2018 at 12:16 PM Firoz Khan wrote: > >> > >> Hi Arnd, > >> > >> On Fri, 12 Oct 2018 at 15:26, Arnd Bergmann wrote: > >>> > >>> On Fri, Oct 12, 2018 at 11:45 AM Firoz Khan wrote: > >>>> > >>>> Wire up rseq system call requires an architecture specific > >>>> implementation as it not present now. > >>>> > >>>> Signed-off-by: Firoz Khan > >>> > >>> Didn't we conclude that this should be skipped? > >> > >> Helge told "I prefer to keep the warning for rseq for now. > >> It reminds me that we still may want the rseq syscall. > >> If the warning is a problem, you may simply add the __IGNORE_rseq define." > >> > >> I chose this option; I feel it looks clean > >> "# rseq requires an architecture specific implementation" in syscall.tbl > >> hopefully remind him to add rseq syscall implementation in this architecture. > > > > But he said he wants the warning instead. > > Correct. > Firoz, please simply drop this patch completely. > That way one will see generated warnings about the missing rseq (which is what > we want). Sure. will do! Firoz > > Thanks, > Helge