From owner-svn-src-head@freebsd.org Thu Mar 30 08:22:09 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 33A64D2314B; Thu, 30 Mar 2017 08:22:09 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from cyrus.watson.org (cyrus.watson.org [198.74.231.69]) by mx1.freebsd.org (Postfix) with ESMTP id 159A1D89; Thu, 30 Mar 2017 08:22:09 +0000 (UTC) (envelope-from rwatson@FreeBSD.org) Received: from fledge.watson.org (fledge.watson.org [198.74.231.63]) by cyrus.watson.org (Postfix) with ESMTPS id 3746046BBA; Thu, 30 Mar 2017 04:22:08 -0400 (EDT) Date: Thu, 30 Mar 2017 09:22:07 +0100 (BST) From: Robert Watson X-X-Sender: robert@fledge.watson.org To: Konstantin Belousov cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r316182 - in head/sys: compat/freebsd32 kern sys In-Reply-To: <20170330034857.GU43712@kib.kiev.ua> Message-ID: References: <201703292233.v2TMXvot015371@repo.freebsd.org> <20170330034857.GU43712@kib.kiev.ua> User-Agent: Alpine 2.20 (BSF 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 30 Mar 2017 08:22:09 -0000 On Thu, 30 Mar 2017, Konstantin Belousov wrote: >> Hook up new audit event identifiers for various non-Orange Book/CAPP >> system calls supported by OpenBSM 1.2-alpha5. >> >> Obtained from: TrustedBSD Project >> MFC after: 3 weeks >> Sponsored by: DARPA, AFRL >> >> Modified: >> head/sys/compat/freebsd32/freebsd32_proto.h >> head/sys/compat/freebsd32/freebsd32_sysent.c >> head/sys/compat/freebsd32/syscalls.master >> head/sys/kern/init_sysent.c >> head/sys/kern/syscalls.master >> head/sys/sys/sysproto.h > > This was already discussed somewhere, might be the place was not public > enough. > > The change, as committed, is impossible to read. Please do not mix > human-made changes and generated files in one commit. For head it is no much > harm to split the syscalls.master commits in way it was always done, for > sake of the people reading either commit mails or VCS diffs. > > I have no opinion on the split/single commit on stable branches. Hi Kostik: My apologies -- I'm afriad I lost track of that convention during the merge. I have to say, though, that if we want to make changes easier to follow, the problem here is not so much generated files as poor formatting in generated files. Reading syscalls.master changes is always hard when multiple system calls are affected, due to the choice of putting all metadata for a system call on one line in the file. We could fix this in the generated files easily, though, by having the structs in init_sysent.c add carriage returns after each field assignment -- in which case the generated changes would make it easier to understand what was going on for multi-syscall commits, and help catch errors better. (I.e., by having the assignment of various system-call-related fields in the struct each appear on their own line.) At which point I suspect I'd generally prefer to see the commits combined. (The other reason we used to not combine commits related to generated $FreeBSD$ IDs in the files .. but that seems to have gone away. Robert