Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp230188imu; Fri, 16 Nov 2018 01:30:13 -0800 (PST) X-Google-Smtp-Source: AJdET5e04W3ttARD/fk/MyFU59wp7Fz3MZvCVaxnEdh1q5Ub1rfAeh+GyLHAwJnKNC84JEhpMvaM X-Received: by 2002:a63:a35c:: with SMTP id v28mr9159305pgn.205.1542360612961; Fri, 16 Nov 2018 01:30:12 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542360612; cv=none; d=google.com; s=arc-20160816; b=dQg6wjai0Y0vtrwZ030+noCJ0N9B1ks2U72fBuZQWQKyielPZ7CCMB4W3e7Me6w426 2BlTaqxCAfATSZWOP6bvIPOpI0j/1/e306M0RWBKUsTY4E1sQijSkR1whv9/J2+rKFgs d4vGNogbyVA80r3UJMLMudC8WGk86ysynMz/vHU5IFM1vq6NAfVJMu2U8QdB4OUUuqNs YWjteVNCWgBt5RSxMDKEmo/scTgGo9+e/CO3gmUnz+uC2hE73sBlHX4BtB3m4L12vptX 6pgt0DgcCBa/MWJHEarypWjFI6VNAaVQCQOJv4TsKLhvonf+FuJdFnWjLgAAqouNTboD Q9iw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=YEus7D/Gzcn4MHetFxsOOXwDjW2JH9JNswgSBlTJ684=; b=PLHiMebeaBa9c5hm6cDw27aCi8QzVhGsWXHbL3Kv72YlHZb2fMIkv8GdyALXnqbV+T GxxfbNoHHADKoYG5zAlYbMY/jLhD9dAlOPbLXVMuj7UEPsSqiikb18vW//x5ER3abm9u WDwI9I9PwU/G+s80ZexGIHi6QenpXbGOc0PZtEu/5jX2mIejpm/sOfwjyajkei1M/tZi 0wMr6AhunZigL3lymQ1cDf/3vdLqYd/YVuqmVFdrWnE482ObZbszEXlvUoW04mNUO14n LCGLGqrGIFUVUD70h1EAkVFGx5KD5TakzMc1bqI5Rue+nWFFex/IKeV8S67zPS0n0XDf TdCg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iJShhHlP; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c15si26893956pgg.446.2018.11.16.01.29.57; Fri, 16 Nov 2018 01:30:12 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=iJShhHlP; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727878AbeKPTjd (ORCPT + 99 others); Fri, 16 Nov 2018 14:39:33 -0500 Received: from mail-it1-f193.google.com ([209.85.166.193]:39047 "EHLO mail-it1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727527AbeKPTjd (ORCPT ); Fri, 16 Nov 2018 14:39:33 -0500 Received: by mail-it1-f193.google.com with SMTP id m15so33404555itl.4 for ; Fri, 16 Nov 2018 01:28:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=YEus7D/Gzcn4MHetFxsOOXwDjW2JH9JNswgSBlTJ684=; b=iJShhHlPjfyY6QIecPHWgyGglj0c/9Ml5E0z41tFGhkHiGQJCP0a4Bn+AWIVf7fNgJ VKFYVggYMc3ZGzpT4/hMmKKUsndnOghgbNNmGyAnUdj3TRDVR3frw5tGuIBgFLnsFwXu QLemV5iGTs3v7AU3wIwigOEdf0FXUg4JM3YZk= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=YEus7D/Gzcn4MHetFxsOOXwDjW2JH9JNswgSBlTJ684=; b=jwxR8aS1QmoB8TNqLO2kydsZ+2bb2uA4EcYhUg9ovq8kzn99y54yKhQfFD0g/X+w3D 530q9qTV3bPGbbotLNTIINZqWBu7M8jp6PCuTGXdpZumPEL1YdTHoNRUmNc0kFxVguxk QD4S3r0z0DaSY31aGDTF1Hy4/VZkm7kuq2lZUzm0+h7nz4rcJey75REAh30vx9dWfVY0 VQvJh/FO14sGKWyt8L1fMqjhe+VJrC4aeNacwKBQtR263/OVMpI2DKRyj6bDbQD/I/WV S9Fn+snWn9ocG9CSMvXYf0A+4omo+mgaj0Sr1+3KjF8Rqr6IXF/2gKUEEbDE3V/M6U2j q1zg== X-Gm-Message-State: AGRZ1gLOqqKyBRfWOxB7zi7FjTohrdQVqHxonXBR70cUwSjtUmbkuOdq 42pDJ7RzQqUhbBV9vI4Q0Su68Z18i5rQ1sCNZePgMQ== X-Received: by 2002:a05:660c:a1a:: with SMTP id e26mr4787652itk.83.1542360483178; Fri, 16 Nov 2018 01:28:03 -0800 (PST) MIME-Version: 1.0 References: <1542089074-20027-1-git-send-email-firoz.khan@linaro.org> <632c1d3f-26cf-7e65-4e7a-cc882ef168c4@monstr.eu> In-Reply-To: <632c1d3f-26cf-7e65-4e7a-cc882ef168c4@monstr.eu> From: Firoz Khan Date: Fri, 16 Nov 2018 14:57:51 +0530 Message-ID: Subject: Re: [PATCH v4 0/3] microblaze: system call table generation support To: Michal Simek Cc: Greg Kroah-Hartman , Philippe Ombredanne , Thomas Gleixner , Kate Stewart , y2038 Mailman List , Linux Kernel Mailing List , Linux-Arch , Arnd Bergmann , Deepa Dinamani , Marcin Juszkiewicz Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Michal, On Fri, 16 Nov 2018 at 14:01, Michal Simek wrote: > > On 13. 11. 18 7:04, Firoz Khan wrote: > > The purpose of this patch series is, we can easily > > add/modify/delete system call table support by cha- > > nging entry in syscall.tbl file instead of manually > > changing many files. The other goal is to unify the > > system call table generation support implementation > > across all the architectures. > > > > The system call tables are in different format in > > all architecture. It will be difficult to manually > > add, modify or delete the system calls in the resp- > > ective files manually. To make it easy by keeping a > > script and which'll generate uapi header file and > > syscall table file. > > > > syscall.tbl contains the list of available system > > calls along with system call number and correspond- > > ing entry point. Add a new system call in this arch- > > itecture will be possible by adding new entry in the > > syscall.tbl file. > > > > Adding a new table entry consisting of: > > - System call number. > > - ABI. > > - System call name. > > - Entry point name. > > > > ARM, s390 and x86 architecuture does exist the sim- > > ilar support. I leverage their implementation to > > come up with a generic solution. > > > > I have done the same support for work for alpha, ia64, > > m68k, mips, parisc, powerpc, sh, sparc and xtensa. > > Below mentioned git repository contains more details > > about the workflow. > > > > https://github.com/frzkhn/system_call_table_generator/ > > > > Finally, this is the ground work to solve the Y2038 > > issue. We need to add two dozen of system calls to > > solve Y2038 issue. So this patch series will help to > > add new system calls easily by adding new entry in > > the syscall.tbl. > > > > changes since v3: > > - changed from generic-y to generated-y in Kbuild. > > > > changes since v2: > > - optimized/updated the syscall table generation > > scripts. > > - fixed all mixed indentation issues in syscall.tbl. > > - added "comments" in syscall_*.tbl. > > > > changes since v1: > > - enclosed __NR_sycalls macro with __KERNEL__. > > - added missing new line. > > next time please also keep this changelog in every patch. Sure. Basically I haven't came across changelog till a point where Geert Uytterhoeven pointed out changelog is missing. Thanks Geert :) > > > > > Firoz Khan (3): > > microblaze: move __NR_syscalls macro from asm/unistd.h > > microblaze: add system call table generation support > > microblaze: generate uapi header and system call table files > > > > arch/microblaze/Makefile | 3 + > > arch/microblaze/include/asm/Kbuild | 1 + > > arch/microblaze/include/asm/unistd.h | 2 - > > arch/microblaze/include/uapi/asm/Kbuild | 1 + > > arch/microblaze/include/uapi/asm/unistd.h | 407 +------------------------ > > arch/microblaze/kernel/syscall_table.S | 406 +------------------------ > > arch/microblaze/kernel/syscalls/Makefile | 38 +++ > > arch/microblaze/kernel/syscalls/syscall.tbl | 410 ++++++++++++++++++++++++++ > > arch/microblaze/kernel/syscalls/syscallhdr.sh | 36 +++ > > arch/microblaze/kernel/syscalls/syscalltbl.sh | 32 ++ > > 10 files changed, 526 insertions(+), 810 deletions(-) > > create mode 100644 arch/microblaze/kernel/syscalls/Makefile > > create mode 100644 arch/microblaze/kernel/syscalls/syscall.tbl > > create mode 100644 arch/microblaze/kernel/syscalls/syscallhdr.sh > > create mode 100644 arch/microblaze/kernel/syscalls/syscalltbl.sh > > > > Looks good now. I have queue it to next. And also asked our team to > retest everything. Sounds good! Thanks Firoz > > Thanks, > Michal > > -- > Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91 > w: www.monstr.eu p: +42-0-721842854 > Maintainer of Linux kernel - Xilinx Microblaze > Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs > U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs > >