Date: Thu, 30 Mar 2017 09:22:07 +0100 (BST) From: Robert Watson <rwatson@FreeBSD.org> To: Konstantin Belousov <kostikbel@gmail.com> 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 Message-ID: <alpine.BSF.2.20.1703300914100.54911@fledge.watson.org> In-Reply-To: <20170330034857.GU43712@kib.kiev.ua> References: <201703292233.v2TMXvot015371@repo.freebsd.org> <20170330034857.GU43712@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?alpine.BSF.2.20.1703300914100.54911>