From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yury Norov Subject: Re: [PATCH 03/20] asm-generic: Drop getrlimit and setrlimit syscalls from default list Date: Wed, 7 Jun 2017 21:48:09 +0300 Message-ID: <20170607184809.e4hmsvewkrmigmww@yury-thinkpad> References: <20170604120009.342-1-ynorov@caviumnetworks.com> <20170604120009.342-4-ynorov@caviumnetworks.com> <20170605144406.GT6973@jhogan-linux.le.imgtec.org> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=Qv9KjpQg+C6SQFCqSqcGpqhF6Q38VZ6d/Sq4SBbCJf8=; b=GsCF6zNXP7UX1++n43bCkVqhKOIeAZL0ryhSgfeSM4+TJz2dTFYQWpr7ioFRfZMJGmY6hmAzP1dI4sLFvEpNgOhY6T1i7+OKTRu6SmS3UQIj8sHDLJCsOncKAR6VQUxwdwSDSB39+3UqwP5iZTIu3yYJfAjRpb/rdHKWjwP5l5Q= Content-Disposition: inline In-Reply-To: <20170605144406.GT6973-4bYivNCBEGTR3KXKvIWQxtm+Uo4AYnCiHZ5vskTnxNA@public.gmane.org> Sender: linux-metag-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: James Hogan Cc: Catalin Marinas , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Arnd Bergmann , Andrew Pinski , Andrew Pinski , Adam Borowski , Chris Metcalf , Steve Ellcey , Maxim Kuvyrkov , Ramana Radhakrishnan , Florian Weimer , Bamvor Zhangjian , Andreas Schwab , Chris Metcalf , Heiko Carstens , schwidefsky-tA70FqPdS9bQT0dZR+AlfA@public.gmane.org, broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Joseph Myers , christoph. On Mon, Jun 05, 2017 at 03:44:06PM +0100, James Hogan wrote: > Hi Yury, > > On Sun, Jun 04, 2017 at 02:59:52PM +0300, Yury Norov wrote: > > The newer prlimit64 syscall provides all the functionality provided by > > the getrlimit and setrlimit syscalls and adds the pid of target process, > > so future architectures won't need to include getrlimit and setrlimit. > > > > Therefore drop getrlimit and setrlimit syscalls from the generic syscall > > list unless __ARCH_WANT_SET_GET_RLIMIT is defined by the architecture's > > unistd.h prior to including asm-generic/unistd.h, and adjust all architectures > > using the generic syscall list to define it so that no in-tree architectures > > are affected. > > > > Cc: Arnd Bergmann > > Cc: James Hogan > > Cc: linux-arch-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Cc: linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > > Cc: Catalin Marinas > > Cc: Will Deacon > > Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org > > Cc: Mark Salter > > Cc: Aurelien Jacquiot > > Cc: linux-c6x-dev-jPsnJVOj+W6hPH1hqNUYSQ@public.gmane.org > > Cc: Richard Kuo > > Cc: linux-hexagon-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Cc: linux-metag-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Cc: Jonas Bonn > > Cc: linux-ZwoEplunGu2h2975yA52AaxOck334EZe@public.gmane.org > > Cc: Chen Liqin > > Cc: Lennox Wu > > Cc: Chris Metcalf > > Cc: Guan Xuetao > > Cc: Ley Foon Tan > > Cc: nios2-dev-g9ZBwUv/Ih/yUk5EbOjzuce+I+R0W71w@public.gmane.org > > Cc: Yoshinori Sato > > Cc: uclinux-h8-devel-5NWGOfrQmneRv+LV9MX5uooqe+aC9MnS@public.gmane.org > > Signed-off-by: Yury Norov > > Acked-by: Arnd Bergmann > > Acked-by: Mark Salter [c6x] > > Acked-by: James Hogan [metag] > > Acked-by: Ley Foon Tan [nios2] > > Acked-by: Stafford Horne [openrisc] > > Acked-by: Will Deacon [arm64] > > Acked-by: Vineet Gupta #arch/arc bits > > --- > > arch/arc/include/uapi/asm/unistd.h | 1 + > > arch/arm64/include/uapi/asm/unistd.h | 1 + > > arch/c6x/include/uapi/asm/unistd.h | 1 + > > arch/h8300/include/uapi/asm/unistd.h | 1 + > > arch/hexagon/include/uapi/asm/unistd.h | 1 + > > arch/metag/include/uapi/asm/unistd.h | 1 + > > arch/nios2/include/uapi/asm/unistd.h | 1 + > > arch/openrisc/include/uapi/asm/unistd.h | 1 + > > arch/score/include/uapi/asm/unistd.h | 1 + > > arch/tile/include/uapi/asm/unistd.h | 1 + > > arch/unicore32/include/uapi/asm/unistd.h | 1 + > > include/uapi/asm-generic/unistd.h | 5 +++++ > > Don't forget to add __IGNORE_getrlimit and __IGNORE_setrlimit to > scripts/checksyscalls.sh, or you'll get warnings about missing syscalls. > > Cheers > James Hi James, Thanks for the hint. Yury