Re: CVS commit: src/sys

[Rin Okuyama <rokuyama.rk%gmail.com@localhost>]

On 2020/04/17 6:56, Rin Okuyama wrote:
> Module Name:	src
> Committed By:	rin
> Date:		Thu Apr 16 21:56:43 UTC 2020
>
> Modified Files:
> 	src/sys/arch/arm/omap: omap3_sdmareg.h omap3_sdmavar.h omapfbreg.h
> 	src/sys/arch/arm/ti: omap3_dssreg.h
> 	src/sys/arch/macppc/dev: batteryvar.h cudavar.h deqvar.h obiovar.h
> 	    platinumfbreg.h pmuvar.h valkyriefbreg.h videopllvar.h
> 	src/sys/arch/mips/adm5120/dev: if_admswvar.h
> 	src/sys/arch/powerpc/include: intr.h
> 	src/sys/arch/powerpc/pic: ipivar.h picvar.h
> 	src/sys/dev/acpi/wmi: wmi_acpivar.h
> 	src/sys/dev/adb: adbvar.h
> 	src/sys/dev/i2c: adm1026reg.h dbcool_reg.h dbcool_var.h sgsmixvar.h
> 	src/sys/dev/ic: i128reg.h i128var.h
> 	src/sys/dev/pci: gffbreg.h ppbvar.h voyagervar.h
> 	src/sys/dev/qbus: rlvar.h
> 	src/sys/external/bsd/ena-com: ena_plat.h
> 	src/sys/lib/libkern: crc16.h
> 	src/sys/ufs/ext2fs: ext2fs_xattr.h
>
> Log Message:
> Stop using __KERNEL_RCSID() in header files; it confuses ident(1) by
> overwriting RCSID in main source files.
>
> XXX
> The first argument of __KERNEL_RCSID() is neglected for ELF. If we wish
> to have RCSID of header files in kernel binary, we need something like
> __FBSDID() macro in FreeBSD.

Oops, this description is not correct. __KERNEL_RCSID() in header files
does *not* overwrite RCSID in main source files. The problem is that it
inserts its own RCSID in *every* object files.

As a result, ident(1) shows more than 1000 duplicated lines for GENERIC
kernel of macppc, for example.

Thanks,
rin