Re: options(4) improvements

2018-04-26 Thread Jason McIntyre
On Tue, Apr 24, 2018 at 07:13:01PM +0300, IL Ka wrote:
> Hello  Jason,
> here is new patch.
> 
> 

as indicated, now fixed.

thanks,
jmc

> 
> diff --git share/man/man4/options.4 share/man/man4/options.4
> index 9ef493cf6..b0b0605c2 100644
> --- share/man/man4/options.4
> +++ share/man/man4/options.4
> @@ -83,8 +83,7 @@ system call.
>  .It Cd option DDB
>  Compiles in a kernel debugger for diagnosing kernel problems.
>  See
> -.Xr ddb 4
> -for details.
> +.Xr ddb 4 .
>  .It Cd option DDB_SAFE_CONSOLE
>  Allows a break into the kernel debugger during boot.
>  Useful when debugging problems that can cause
> @@ -114,8 +113,7 @@ Adds code to the kernel for kernel profiling with
>  Adds hooks for the system call tracing facility, which allows users to
>  watch the system call invocation behavior of processes.
>  See
> -.Xr ktrace 1
> -for details.
> +.Xr ktrace 1 .
>  .It Cd option NO_PROPOLICE
>  Do not compile the kernel with the ProPolice stack protection.
>  See
> @@ -125,8 +123,7 @@ for more information about ProPolice.
>  Adds hooks for the process tracing facility, allowing a process to
>  control and observe another process.
>  See
> -.Xr ptrace 2
> -for details.
> +.Xr ptrace 2 .
>  .It Cd option SMALL_KERNEL
>  Removes some features and some optimizations from the kernel to reduce the
>  size of the resulting kernel binary.
> @@ -135,13 +132,11 @@ for general purpose kernels.
>  .It Cd option VFSLCKDEBUG
>  Turns on debugging for the Virtual File System interface.
>  See
> -.Xr vfs 9
> -for details.
> +.Xr vfs 9 .
>  .It Cd option WITNESS
>  Compiles in a lock checker for detecting lock order violations in the
> kernel.
>  See
> -.Xr witness 4
> -for details.
> +.Xr witness 4 .
>  .It Cd option WITNESS_COUNT= Ns Ar integer
>  Maximum number of lock types that are tracked by
>  .Xr witness 4 .
> @@ -154,8 +149,7 @@ Includes code for the ISO 9660 + Rock Ridge file
> system, which is the
>  standard file system used on many CD-ROMs.
>  It also supports Joliet extensions.
>  See
> -.Xr mount_cd9660 8
> -for details.
> +.Xr mount_cd9660 8 .
>  .It Cd option EXT2FS
>  Includes code implementing the Second Extended File System
>  .Em ( EXT2FS ) ,
> @@ -175,8 +169,7 @@ differing byte-orders.
>  That is, a big-endian machine will not be able to read an
>  ext2fs filesystem created on an i386 or other little-endian machine.
>  See
> -.Xr mount_ext2fs 8
> -for details.
> +.Xr mount_ext2fs 8 .
>  .It Cd option FFS
>  Includes code implementing the Berkeley Fast File System
>  .Em ( FFS ) .
> @@ -193,8 +186,7 @@ for
>  .Pa /tmp
>  or similar mount points.
>  See
> -.Xr mount_mfs 8
> -for details.
> +.Xr mount_mfs 8 .
>  .It Cd option MSDOSFS
>  Includes support for the MS-DOS FAT file system.
>  The kernel also implements the Windows 95
> @@ -202,8 +194,7 @@ extensions which permit the use of longer, mixed-case
> file names.
>  See
>  .Xr mount_msdos 8
>  and
> -.Xr fsck_msdos 8
> -for details.
> +.Xr fsck_msdos 8 .
>  .It Cd option NFSCLIENT
>  Include the client side of the
>  .Em NFS
> @@ -217,18 +208,15 @@ for details on NFS.
>  .It Cd option NTFS
>  Includes support for reading NTFS file systems.
>  See
> -.Xr mount_ntfs 8
> -for details.
> +.Xr mount_ntfs 8 .
>  .It Cd option UDF
>  Includes code for the UDF file systems typically found on DVD discs.
>  See
> -.Xr mount_udf 8
> -for details.
> +.Xr mount_udf 8 .
>  .It Cd option TMPFS
>  Includes code for the TMPFS efficient memory file system.
>  See
> -.Xr mount_tmpfs 8
> -for details.
> +.Xr mount_tmpfs 8 .
>  .El
>  .Sh FILE SYSTEM OPTIONS
>  .Bl -ohang
> @@ -256,8 +244,7 @@ retaining the safety of synchronous metadata updates.
>  .Pp
>  Soft updates must be enabled on a per-filesystem basis.
>  See
> -.Xr mount 8
> -for details.
> +.Xr mount 8 .
>  .Pp
>  Processors with a small kernel address space, such as the sun4 and sun4c,
> do
>  not have enough kernel memory to support soft updates.
> @@ -282,8 +269,7 @@ work.
>  See
>  .Xr mountd 8
>  and
> -.Xr nfsd 8
> -for details.
> +.Xr nfsd 8 .
>  .It Cd option QUOTA
>  Enables kernel support for file system quotas.
>  See
> @@ -291,8 +277,7 @@ See
>  .Xr edquota 8 ,
>  .Xr repquota 8 ,
>  and
> -.Xr quota 1
> -for details.
> +.Xr quota 1 .
>  Note that quotas only work on
>  .Dq ffs
>  file systems, although
> @@ -309,6 +294,8 @@ in large directories.
>  Provide in-kernel support for controlling VGA framebuffer mapping
>  and PCI configuration registers by user-processes
>  (such as an X Window System server).
> +See
> +.Xr xf86 4 .
>  This option is supported on the
>  .Va alpha ,
>  .Va amd64 ,
> @@ -323,15 +310,18 @@ Adds support for the
>  boot option (User Kernel Config).
>  Allows modification of kernel settings (e.g., device parameters) before
>  booting the system.
> +See
> +.Xr boot_config 8 .
>  .It Cd option CRYPTO
>  Enables support for the kernel cryptographic framework.
>  See
> -.Xr crypto 9
> -for details.
> +.Xr crypto 9 .
>  While not IP specific, this option is usually used in conjuncti

Re: options(4) improvements

2018-04-25 Thread Il Ka
>>patch:  malformed patch at line 44: kernel. 

Hm. Lets try again. 
Attached patch works for me on latest version from git.
I also uploaded it to https://pastebin.com/raw/LqsbWk5x
so, you can do
$ ftp -o- https://pastebin.com/raw/LqsbWk5x   | dos2unix  | patch

But I just ran patch attached below on latest commit ("New regress to handle
flags") and it worked.







--
Sent from: 
http://openbsd-archive.7691.n7.nabble.com/openbsd-dev-tech-f151936.html



Re: options(4) improvements

2018-04-24 Thread Jason McIntyre
On Tue, Apr 24, 2018 at 07:13:01PM +0300, IL Ka wrote:
> Hello  Jason,
> here is new patch.
> 

thanks. but it still doesn't apply, and it will be a pain to adjust by
hand...

Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--
|diff --git share/man/man4/options.4 share/man/man4/options.4
|index 9ef493cf6..b0b0605c2 100644
|--- share/man/man4/options.4
|+++ share/man/man4/options.4
--
Patching file options.4 using Plan A...
Hunk #1 succeeded at 83.
Hunk #2 succeeded at 113.
Hunk #3 succeeded at 123.
patch:  malformed patch at line 44: kernel.

jmc

> 
> 
> diff --git share/man/man4/options.4 share/man/man4/options.4
> index 9ef493cf6..b0b0605c2 100644
> --- share/man/man4/options.4
> +++ share/man/man4/options.4
> @@ -83,8 +83,7 @@ system call.
>  .It Cd option DDB
>  Compiles in a kernel debugger for diagnosing kernel problems.
>  See
> -.Xr ddb 4
> -for details.
> +.Xr ddb 4 .
>  .It Cd option DDB_SAFE_CONSOLE
>  Allows a break into the kernel debugger during boot.
>  Useful when debugging problems that can cause
> @@ -114,8 +113,7 @@ Adds code to the kernel for kernel profiling with
>  Adds hooks for the system call tracing facility, which allows users to
>  watch the system call invocation behavior of processes.
>  See
> -.Xr ktrace 1
> -for details.
> +.Xr ktrace 1 .
>  .It Cd option NO_PROPOLICE
>  Do not compile the kernel with the ProPolice stack protection.
>  See
> @@ -125,8 +123,7 @@ for more information about ProPolice.
>  Adds hooks for the process tracing facility, allowing a process to
>  control and observe another process.
>  See
> -.Xr ptrace 2
> -for details.
> +.Xr ptrace 2 .
>  .It Cd option SMALL_KERNEL
>  Removes some features and some optimizations from the kernel to reduce the
>  size of the resulting kernel binary.
> @@ -135,13 +132,11 @@ for general purpose kernels.
>  .It Cd option VFSLCKDEBUG
>  Turns on debugging for the Virtual File System interface.
>  See
> -.Xr vfs 9
> -for details.
> +.Xr vfs 9 .
>  .It Cd option WITNESS
>  Compiles in a lock checker for detecting lock order violations in the
> kernel.
>  See
> -.Xr witness 4
> -for details.
> +.Xr witness 4 .
>  .It Cd option WITNESS_COUNT= Ns Ar integer
>  Maximum number of lock types that are tracked by
>  .Xr witness 4 .
> @@ -154,8 +149,7 @@ Includes code for the ISO 9660 + Rock Ridge file
> system, which is the
>  standard file system used on many CD-ROMs.
>  It also supports Joliet extensions.
>  See
> -.Xr mount_cd9660 8
> -for details.
> +.Xr mount_cd9660 8 .
>  .It Cd option EXT2FS
>  Includes code implementing the Second Extended File System
>  .Em ( EXT2FS ) ,
> @@ -175,8 +169,7 @@ differing byte-orders.
>  That is, a big-endian machine will not be able to read an
>  ext2fs filesystem created on an i386 or other little-endian machine.
>  See
> -.Xr mount_ext2fs 8
> -for details.
> +.Xr mount_ext2fs 8 .
>  .It Cd option FFS
>  Includes code implementing the Berkeley Fast File System
>  .Em ( FFS ) .
> @@ -193,8 +186,7 @@ for
>  .Pa /tmp
>  or similar mount points.
>  See
> -.Xr mount_mfs 8
> -for details.
> +.Xr mount_mfs 8 .
>  .It Cd option MSDOSFS
>  Includes support for the MS-DOS FAT file system.
>  The kernel also implements the Windows 95
> @@ -202,8 +194,7 @@ extensions which permit the use of longer, mixed-case
> file names.
>  See
>  .Xr mount_msdos 8
>  and
> -.Xr fsck_msdos 8
> -for details.
> +.Xr fsck_msdos 8 .
>  .It Cd option NFSCLIENT
>  Include the client side of the
>  .Em NFS
> @@ -217,18 +208,15 @@ for details on NFS.
>  .It Cd option NTFS
>  Includes support for reading NTFS file systems.
>  See
> -.Xr mount_ntfs 8
> -for details.
> +.Xr mount_ntfs 8 .
>  .It Cd option UDF
>  Includes code for the UDF file systems typically found on DVD discs.
>  See
> -.Xr mount_udf 8
> -for details.
> +.Xr mount_udf 8 .
>  .It Cd option TMPFS
>  Includes code for the TMPFS efficient memory file system.
>  See
> -.Xr mount_tmpfs 8
> -for details.
> +.Xr mount_tmpfs 8 .
>  .El
>  .Sh FILE SYSTEM OPTIONS
>  .Bl -ohang
> @@ -256,8 +244,7 @@ retaining the safety of synchronous metadata updates.
>  .Pp
>  Soft updates must be enabled on a per-filesystem basis.
>  See
> -.Xr mount 8
> -for details.
> +.Xr mount 8 .
>  .Pp
>  Processors with a small kernel address space, such as the sun4 and sun4c,
> do
>  not have enough kernel memory to support soft updates.
> @@ -282,8 +269,7 @@ work.
>  See
>  .Xr mountd 8
>  and
> -.Xr nfsd 8
> -for details.
> +.Xr nfsd 8 .
>  .It Cd option QUOTA
>  Enables kernel support for file system quotas.
>  See
> @@ -291,8 +277,7 @@ See
>  .Xr edquota 8 ,
>  .Xr repquota 8 ,
>  and
> -.Xr quota 1
> -for details.
> +.Xr quota 1 .
>  Note that quotas only work on
>  .Dq ffs
>  file systems, although
> @@ -309,6 +294,8 @@ in large directories.
>  Provide in-kernel support for controlling VGA framebuffer mapping
>  and PCI configuration registers by user-processes
>  (such as an X Window System server).
> +Se

Re: options(4) improvements

2018-04-24 Thread IL Ka
Hello  Jason,
here is new patch.



diff --git share/man/man4/options.4 share/man/man4/options.4
index 9ef493cf6..b0b0605c2 100644
--- share/man/man4/options.4
+++ share/man/man4/options.4
@@ -83,8 +83,7 @@ system call.
 .It Cd option DDB
 Compiles in a kernel debugger for diagnosing kernel problems.
 See
-.Xr ddb 4
-for details.
+.Xr ddb 4 .
 .It Cd option DDB_SAFE_CONSOLE
 Allows a break into the kernel debugger during boot.
 Useful when debugging problems that can cause
@@ -114,8 +113,7 @@ Adds code to the kernel for kernel profiling with
 Adds hooks for the system call tracing facility, which allows users to
 watch the system call invocation behavior of processes.
 See
-.Xr ktrace 1
-for details.
+.Xr ktrace 1 .
 .It Cd option NO_PROPOLICE
 Do not compile the kernel with the ProPolice stack protection.
 See
@@ -125,8 +123,7 @@ for more information about ProPolice.
 Adds hooks for the process tracing facility, allowing a process to
 control and observe another process.
 See
-.Xr ptrace 2
-for details.
+.Xr ptrace 2 .
 .It Cd option SMALL_KERNEL
 Removes some features and some optimizations from the kernel to reduce the
 size of the resulting kernel binary.
@@ -135,13 +132,11 @@ for general purpose kernels.
 .It Cd option VFSLCKDEBUG
 Turns on debugging for the Virtual File System interface.
 See
-.Xr vfs 9
-for details.
+.Xr vfs 9 .
 .It Cd option WITNESS
 Compiles in a lock checker for detecting lock order violations in the
kernel.
 See
-.Xr witness 4
-for details.
+.Xr witness 4 .
 .It Cd option WITNESS_COUNT= Ns Ar integer
 Maximum number of lock types that are tracked by
 .Xr witness 4 .
@@ -154,8 +149,7 @@ Includes code for the ISO 9660 + Rock Ridge file
system, which is the
 standard file system used on many CD-ROMs.
 It also supports Joliet extensions.
 See
-.Xr mount_cd9660 8
-for details.
+.Xr mount_cd9660 8 .
 .It Cd option EXT2FS
 Includes code implementing the Second Extended File System
 .Em ( EXT2FS ) ,
@@ -175,8 +169,7 @@ differing byte-orders.
 That is, a big-endian machine will not be able to read an
 ext2fs filesystem created on an i386 or other little-endian machine.
 See
-.Xr mount_ext2fs 8
-for details.
+.Xr mount_ext2fs 8 .
 .It Cd option FFS
 Includes code implementing the Berkeley Fast File System
 .Em ( FFS ) .
@@ -193,8 +186,7 @@ for
 .Pa /tmp
 or similar mount points.
 See
-.Xr mount_mfs 8
-for details.
+.Xr mount_mfs 8 .
 .It Cd option MSDOSFS
 Includes support for the MS-DOS FAT file system.
 The kernel also implements the Windows 95
@@ -202,8 +194,7 @@ extensions which permit the use of longer, mixed-case
file names.
 See
 .Xr mount_msdos 8
 and
-.Xr fsck_msdos 8
-for details.
+.Xr fsck_msdos 8 .
 .It Cd option NFSCLIENT
 Include the client side of the
 .Em NFS
@@ -217,18 +208,15 @@ for details on NFS.
 .It Cd option NTFS
 Includes support for reading NTFS file systems.
 See
-.Xr mount_ntfs 8
-for details.
+.Xr mount_ntfs 8 .
 .It Cd option UDF
 Includes code for the UDF file systems typically found on DVD discs.
 See
-.Xr mount_udf 8
-for details.
+.Xr mount_udf 8 .
 .It Cd option TMPFS
 Includes code for the TMPFS efficient memory file system.
 See
-.Xr mount_tmpfs 8
-for details.
+.Xr mount_tmpfs 8 .
 .El
 .Sh FILE SYSTEM OPTIONS
 .Bl -ohang
@@ -256,8 +244,7 @@ retaining the safety of synchronous metadata updates.
 .Pp
 Soft updates must be enabled on a per-filesystem basis.
 See
-.Xr mount 8
-for details.
+.Xr mount 8 .
 .Pp
 Processors with a small kernel address space, such as the sun4 and sun4c,
do
 not have enough kernel memory to support soft updates.
@@ -282,8 +269,7 @@ work.
 See
 .Xr mountd 8
 and
-.Xr nfsd 8
-for details.
+.Xr nfsd 8 .
 .It Cd option QUOTA
 Enables kernel support for file system quotas.
 See
@@ -291,8 +277,7 @@ See
 .Xr edquota 8 ,
 .Xr repquota 8 ,
 and
-.Xr quota 1
-for details.
+.Xr quota 1 .
 Note that quotas only work on
 .Dq ffs
 file systems, although
@@ -309,6 +294,8 @@ in large directories.
 Provide in-kernel support for controlling VGA framebuffer mapping
 and PCI configuration registers by user-processes
 (such as an X Window System server).
+See
+.Xr xf86 4 .
 This option is supported on the
 .Va alpha ,
 .Va amd64 ,
@@ -323,15 +310,18 @@ Adds support for the
 boot option (User Kernel Config).
 Allows modification of kernel settings (e.g., device parameters) before
 booting the system.
+See
+.Xr boot_config 8 .
 .It Cd option CRYPTO
 Enables support for the kernel cryptographic framework.
 See
-.Xr crypto 9
-for details.
+.Xr crypto 9 .
 While not IP specific, this option is usually used in conjunction with
option
 .Em IPSEC .
 .It Cd option EISAVERBOSE
 Makes the boot process more verbose for EISA peripherals.
+See
+.Xr eisa 4 .
 .It Cd option KMEMSTATS
 The kernel memory allocator,
 .Xr malloc 9 ,
@@ -341,8 +331,12 @@ On those architectures that have it, this enables
multiprocessor support.
 .It Cd option PCIVERBOSE
 Makes the boot process more verbose for PCI peripherals
 (vendor names and other information is printed, etc.).
+Se

Re: options(4) improvements

2018-04-20 Thread Jason McIntyre
On Mon, Apr 16, 2018 at 12:13:10AM +0300, IL Ka wrote:
> Links to appropriate man pages added
> 

this doesn;t apply cleanly, so i'll comment: i'm ok with adding the
links. still, we have a mix of

See blah(x) for details.
and
See blah(x).

i think less is more here, so maybe you can resubmit a diff which
changes *all* refs to just "See blah(4)."

one more comment inline:

> 
> diff --git share/man/man4/options.4 share/man/man4/options.4
> index 9ef493cf6..cf146d9a7 100644
> --- share/man/man4/options.4
> +++ share/man/man4/options.4
> @@ -309,6 +309,9 @@ in large directories.
>  Provide in-kernel support for controlling VGA framebuffer mapping
>  and PCI configuration registers by user-processes
>  (such as an X Window System server).
> +See
> +.Xr xf86 4
> +for details.
>  This option is supported on the
>  .Va alpha ,
>  .Va amd64 ,
> @@ -323,6 +326,9 @@ Adds support for the
>  boot option (User Kernel Config).
>  Allows modification of kernel settings (e.g., device parameters) before
>  booting the system.
> +See
> +.Xr boot_config 8
> +for details.
>  .It Cd option CRYPTO
>  Enables support for the kernel cryptographic framework.
>  See
> @@ -332,6 +338,9 @@ While not IP specific, this option is usually used in
> conjunction with option
>  .Em IPSEC .
>  .It Cd option EISAVERBOSE
>  Makes the boot process more verbose for EISA peripherals.
> +See
> +.Xr eisa 4
> +for details.
>  .It Cd option KMEMSTATS
>  The kernel memory allocator,
>  .Xr malloc 9 ,
> @@ -341,8 +350,14 @@ On those architectures that have it, this enables
> multiprocessor support.
>  .It Cd option PCIVERBOSE
>  Makes the boot process more verbose for PCI peripherals
>  (vendor names and other information is printed, etc.).
> +See
> +.Xr pci 4
> +for details
>  .It Cd option PCMCIAVERBOSE
>  Makes the boot process more verbose for PCMCIA peripherals.
> +See
> +.Xr pcmcia 4
> +for details.
>  .It Cd option USER_PCICONF
>  Enables the user level access to the PCI bus configuration space
>  through ioctls on the
> @@ -350,7 +365,9 @@ through ioctls on the
>  device.
>  It's used by the
>  .Xr Xorg 1
> -server on some architectures.
> +server on some architectures,
> +.Xr pcidump 8
> +uses it also.

i would cut this to

It's used by
.Xr Xorg 1
and
.Xr pcidump 8 .

jmc

>  See
>  .Xr pci 4
>  for details.
> @@ -511,6 +528,9 @@ Double quotes are needed when specifying a negative
>  .Ar value .
>  .El
>  .Sh SCSI SUBSYSTEM OPTIONS
> +See
> +.Xr scsi 4
> +for details.
>  .Bl -ohang
>  .It Cd option SCSI_DELAY= Ns Ar value
>  Delay for



options(4) improvements

2018-04-15 Thread IL Ka
Links to appropriate man pages added


diff --git share/man/man4/options.4 share/man/man4/options.4
index 9ef493cf6..cf146d9a7 100644
--- share/man/man4/options.4
+++ share/man/man4/options.4
@@ -309,6 +309,9 @@ in large directories.
 Provide in-kernel support for controlling VGA framebuffer mapping
 and PCI configuration registers by user-processes
 (such as an X Window System server).
+See
+.Xr xf86 4
+for details.
 This option is supported on the
 .Va alpha ,
 .Va amd64 ,
@@ -323,6 +326,9 @@ Adds support for the
 boot option (User Kernel Config).
 Allows modification of kernel settings (e.g., device parameters) before
 booting the system.
+See
+.Xr boot_config 8
+for details.
 .It Cd option CRYPTO
 Enables support for the kernel cryptographic framework.
 See
@@ -332,6 +338,9 @@ While not IP specific, this option is usually used in
conjunction with option
 .Em IPSEC .
 .It Cd option EISAVERBOSE
 Makes the boot process more verbose for EISA peripherals.
+See
+.Xr eisa 4
+for details.
 .It Cd option KMEMSTATS
 The kernel memory allocator,
 .Xr malloc 9 ,
@@ -341,8 +350,14 @@ On those architectures that have it, this enables
multiprocessor support.
 .It Cd option PCIVERBOSE
 Makes the boot process more verbose for PCI peripherals
 (vendor names and other information is printed, etc.).
+See
+.Xr pci 4
+for details
 .It Cd option PCMCIAVERBOSE
 Makes the boot process more verbose for PCMCIA peripherals.
+See
+.Xr pcmcia 4
+for details.
 .It Cd option USER_PCICONF
 Enables the user level access to the PCI bus configuration space
 through ioctls on the
@@ -350,7 +365,9 @@ through ioctls on the
 device.
 It's used by the
 .Xr Xorg 1
-server on some architectures.
+server on some architectures,
+.Xr pcidump 8
+uses it also.
 See
 .Xr pci 4
 for details.
@@ -511,6 +528,9 @@ Double quotes are needed when specifying a negative
 .Ar value .
 .El
 .Sh SCSI SUBSYSTEM OPTIONS
+See
+.Xr scsi 4
+for details.
 .Bl -ohang
 .It Cd option SCSI_DELAY= Ns Ar value
 Delay for