From owner-svn-src-head@freebsd.org Sat Feb 23 03:05:29 2019 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 44F1D1503BDE; Sat, 23 Feb 2019 03:05:29 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail106.syd.optusnet.com.au (mail106.syd.optusnet.com.au [211.29.132.42]) by mx1.freebsd.org (Postfix) with ESMTP id 86F906E5B7; Sat, 23 Feb 2019 03:05:28 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from [192.168.0.102] (c110-21-101-228.carlnfd1.nsw.optusnet.com.au [110.21.101.228]) by mail106.syd.optusnet.com.au (Postfix) with ESMTPS id 5C5A93D8C5B; Sat, 23 Feb 2019 14:05:25 +1100 (AEDT) Date: Sat, 23 Feb 2019 14:05:24 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Maxim Sobolev cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r344479 - head/sys/mips/conf In-Reply-To: <201902230000.x1N00o8R011357@repo.freebsd.org> Message-ID: <20190223125748.S909@besplex.bde.org> References: <201902230000.x1N00o8R011357@repo.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.2 cv=P6RKvmIu c=1 sm=1 tr=0 a=PalzARQSbocsUSjMRkwAPg==:117 a=PalzARQSbocsUSjMRkwAPg==:17 a=kj9zAlcOel0A:10 a=60YepAZSkRJpfBsUMC8A:9 a=CjuIK1q_8ugA:10 X-Rspamd-Queue-Id: 86F906E5B7 X-Spamd-Bar: ------ Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-6.98 / 15.00]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[]; NEURAL_HAM_SHORT(-0.98)[-0.977,0] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 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: Sat, 23 Feb 2019 03:05:29 -0000 On Sat, 23 Feb 2019, Maxim Sobolev wrote: > Log: > o Get rid of silly comment which seems to have got life of its own via > copy-and-paste process; It is perhaps better than no comment on a misformatted options line. > o Return geom_uzip(4) usage back to how manual page prescribes it to be > used while I am here. Section 4 man pages unfortunately don't usually have examples of comments on options line, and never have correct formatting. > Modified: head/sys/mips/conf/AP94 > ============================================================================== > --- head/sys/mips/conf/AP94 Fri Feb 22 23:15:32 2019 (r344478) > +++ head/sys/mips/conf/AP94 Sat Feb 23 00:00:49 2019 (r344479) > @@ -21,7 +21,6 @@ hints "AP94.hints" > > # GEOM modules > device geom_redboot # to get access to the SPI flash partitions > -device geom_uzip # compressed in-memory filesystem hackery! > options GEOM_UZIP > > options ROOTDEVNAME=\"ufs:md0.uzip\" Now there is no comment on the remaining GEOM_UZIP line. "device foo" is sometimes abused to access an unsupported option (an option that is not listed in conf/options*). "foo" is then not available as a macro, but it is available as a keyword for selecting files from conf/files. Also, if DEV_FOO is listed in conf/options, then this is a syntax error if "foo" is _not_ used as a keyword in conf/files, but if "foo" is so used then DEV_FOO is not a syntax error and a #define of it is generated. Everything seems to be case-sensitive, except there is no way to prevent the conversion of "foo" to "FOO" in DEV_FOO. "device loop" is an example of use of this. This just selects the file if_loop.c, and there are no messy ifdefs, so DEV_LOOP is not needed and is not an option and not generated. "device bpf" is a messier example which needs ifdefs on DEV_BPF, so DEV_BPF must be an option. "device gzip" is an example of abuse of this. gzip is very far from being a device. > Modified: head/sys/mips/conf/DIR-825B1 > Modified: head/sys/mips/conf/ROUTERSTATION_MFS > Modified: head/sys/mips/conf/RSPRO_MFS > Modified: head/sys/mips/conf/WZR-300HP 4 more removals leaving no comment. > ============================================================================== > --- head/sys/mips/conf/WZR-300HP Fri Feb 22 23:15:32 2019 (r344478) > +++ head/sys/mips/conf/WZR-300HP Sat Feb 23 00:00:49 2019 (r344479) > @@ -26,7 +26,7 @@ options BOOTVERBOSE > > # GEOM modules > device geom_map # to get access to the SPI flash partitions > -device geom_uzip # compressed in-memory filesystem hackery! > +options GEOM_UZIP > > options ROOTDEVNAME=\"ufs:/dev/map/rootfs.uzip\" This is an addition with a misformatted options line and no comment. conf/NOTES provides an example with correct formatting. It even provides a comment suitable for copying. conf/NOTES has many misformatted options lines (27 with the keyword followed by 2 tabs and 1 with it followed by 1 tab), but all GEOM_FOO lines are formatted normally. */NOTES has too many lines not suitable for copying since they have no comment. NOTES tends to have more verbose comments written in separate paragraphs and lists. The lists are better made up of options lines with short comments. > Modified: head/sys/mips/conf/WZR-HPAG300H Another addition with a misformatted options line and no comment. Normal formatting of options lines lines things up nicely, but wastes 8 or 16 columns which would be better used for comments. Many options names are too long. GENERIC is more consistent than NOTES in wasting an extra 8 columns for short option names so as to line up the comments. device names are better, and not wasting an extra 8 columns to line up the comments for device directives gives another set of inconsistencies. Bruce