Re: CVS commit: src/sys

[Rin Okuyama]

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