linux-kernel - Re: [PATCH] riscv: Cleanup rv32

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CAJF2gTThmVU+me_wdt4EesO13ioMM63VSwQV055mCsoqwFw3oQ@mail.gmail.com>
Date:   Wed, 8 Feb 2023 09:19:44 +0800
From:   Guo Ren <guoren@...nel.org>
To:     Conor Dooley <conor@...nel.org>
Cc:     palmer@...osinc.com, arnd@...db.de, atishp@...osinc.com,
        rdunlap@...radead.org, linux-arch@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-riscv@...ts.infradead.org,
        Guo Ren <guoren@...ux.alibaba.com>
Subject: Re: [PATCH] riscv: Cleanup rv32_defconfig

On Wed, Feb 8, 2023 at 2:44 AM Conor Dooley <conor@...nel.org> wrote:
>
> On Sun, Feb 05, 2023 at 08:33:07AM -0500, guoren@...nel.org wrote:
> > From: Guo Ren <guoren@...ux.alibaba.com>
> > Subject: riscv: Cleanup rv32_defconfig
>
> s/cleanup/remove redundant/, cleanup implies that you're doing some
> rework of the file IMO.
Okay

>
> > Remove the unused rv32_defconfig file,
>
> What do you mean by "unused"? (don't answer now, read on)
>
> > which has been replaced by
> > 'commit 72f045d19f25 ("riscv: Fixup difference with defconfig")'.
>
> Replaced how? One shouldn't have to go look up that commit to see what
> "replaced" means. Certainly from the subject it is not immediately
> obvious what that commit actually does.
>
> How about:
> "Remove the rv32_defconfig file, which has been made redundant by commit
> 112233445566 ("blah") which introduced an rv32_defconfig .PHONY target,
> that is prioritised over the file itself."
Thx, that's better.

>
> I would argue that there is still value in having an off-the-shelf 32-bit
> config for people to base things on though, and being ignored by the
> make target does not make it "unused" IMO.
>
> I did check the before/after of running the make target, so:
> Tested-by: Conor Dooley <conor.dooley@...rochip.com>
Thx

> Although, as you are probably aware, there are significant differences
> between the .config generated by the file and by the rv32_defconfig make
> target!
Two reasons:
 - Our Makefile has ignored the rv32_defconfig file.
 - We shouldn't keep the differences that violate the original design principle.
   See 'commit 1b937e8faa87 ("RISC-V: Add separate defconfig for 32bit
systems")'

commit 1b937e8faa87ccfb4b7d5b230796fa67bc8a183b
Author: Anup Patel <anup@...infault.org>
Date:   Tue Mar 12 22:08:12 2019 +0000

    RISC-V: Add separate defconfig for 32bit systems

    This patch adds rv32_defconfig for 32bit systems. The only
    difference between rv32_defconfig and defconfig is that
    rv32_defconfig has  CONFIG_ARCH_RV32I=y.

    Signed-off-by: Anup Patel <anup.patel@....com>
    Signed-off-by: Palmer Dabbelt <palmer@...ive.com>

>
> Cheers,
> Conor.
>
> > Also,
> > remove CONFIG_32BIT in 32-bit.config, which CONFIG_ARCH_RV32I has
> > selected.
> >
> > Signed-off-by: Guo Ren <guoren@...ux.alibaba.com>
> > Signed-off-by: Guo Ren <guoren@...nel.org>
> > ---
> >  arch/riscv/configs/32-bit.config  |   2 -
> >  arch/riscv/configs/rv32_defconfig | 139 ------------------------------
> >  2 files changed, 141 deletions(-)
> >  delete mode 100644 arch/riscv/configs/rv32_defconfig
> >
> > diff --git a/arch/riscv/configs/32-bit.config b/arch/riscv/configs/32-bit.config
> > index f6af0f708df4..eb87885c8640 100644
> > --- a/arch/riscv/configs/32-bit.config
> > +++ b/arch/riscv/configs/32-bit.config
> > @@ -1,4 +1,2 @@
> >  CONFIG_ARCH_RV32I=y
> > -CONFIG_32BIT=y
> > -# CONFIG_PORTABLE is not set
> >  CONFIG_NONPORTABLE=y
> > diff --git a/arch/riscv/configs/rv32_defconfig b/arch/riscv/configs/rv32_defconfig
> > deleted file mode 100644
> > index 38760e4296cf..000000000000
> > --- a/arch/riscv/configs/rv32_defconfig
> > +++ /dev/null
> > @@ -1,139 +0,0 @@
> > -CONFIG_SYSVIPC=y
> > -CONFIG_POSIX_MQUEUE=y
> > -CONFIG_NO_HZ_IDLE=y
> > -CONFIG_HIGH_RES_TIMERS=y
> > -CONFIG_BPF_SYSCALL=y
> > -CONFIG_IKCONFIG=y
> > -CONFIG_IKCONFIG_PROC=y
> > -CONFIG_CGROUPS=y
> > -CONFIG_CGROUP_SCHED=y
> > -CONFIG_CFS_BANDWIDTH=y
> > -CONFIG_CGROUP_BPF=y
> > -CONFIG_NAMESPACES=y
> > -CONFIG_USER_NS=y
> > -CONFIG_CHECKPOINT_RESTORE=y
> > -CONFIG_BLK_DEV_INITRD=y
> > -CONFIG_EXPERT=y
> > -# CONFIG_SYSFS_SYSCALL is not set
> > -CONFIG_PROFILING=y
> > -CONFIG_SOC_SIFIVE=y
> > -CONFIG_SOC_VIRT=y
> > -CONFIG_NONPORTABLE=y
> > -CONFIG_ARCH_RV32I=y
> > -CONFIG_SMP=y
> > -CONFIG_HOTPLUG_CPU=y
> > -CONFIG_PM=y
> > -CONFIG_CPU_IDLE=y
> > -CONFIG_VIRTUALIZATION=y
> > -CONFIG_KVM=m
> > -CONFIG_JUMP_LABEL=y
> > -CONFIG_MODULES=y
> > -CONFIG_MODULE_UNLOAD=y
> > -CONFIG_NET=y
> > -CONFIG_PACKET=y
> > -CONFIG_UNIX=y
> > -CONFIG_INET=y
> > -CONFIG_IP_MULTICAST=y
> > -CONFIG_IP_ADVANCED_ROUTER=y
> > -CONFIG_IP_PNP=y
> > -CONFIG_IP_PNP_DHCP=y
> > -CONFIG_IP_PNP_BOOTP=y
> > -CONFIG_IP_PNP_RARP=y
> > -CONFIG_NETLINK_DIAG=y
> > -CONFIG_NET_9P=y
> > -CONFIG_NET_9P_VIRTIO=y
> > -CONFIG_PCI=y
> > -CONFIG_PCIEPORTBUS=y
> > -CONFIG_PCI_HOST_GENERIC=y
> > -CONFIG_PCIE_XILINX=y
> > -CONFIG_DEVTMPFS=y
> > -CONFIG_DEVTMPFS_MOUNT=y
> > -CONFIG_BLK_DEV_LOOP=y
> > -CONFIG_VIRTIO_BLK=y
> > -CONFIG_BLK_DEV_SD=y
> > -CONFIG_BLK_DEV_SR=y
> > -CONFIG_SCSI_VIRTIO=y
> > -CONFIG_ATA=y
> > -CONFIG_SATA_AHCI=y
> > -CONFIG_SATA_AHCI_PLATFORM=y
> > -CONFIG_NETDEVICES=y
> > -CONFIG_VIRTIO_NET=y
> > -CONFIG_MACB=y
> > -CONFIG_E1000E=y
> > -CONFIG_R8169=y
> > -CONFIG_MICROSEMI_PHY=y
> > -CONFIG_INPUT_MOUSEDEV=y
> > -CONFIG_SERIAL_8250=y
> > -CONFIG_SERIAL_8250_CONSOLE=y
> > -CONFIG_SERIAL_OF_PLATFORM=y
> > -CONFIG_VIRTIO_CONSOLE=y
> > -CONFIG_HW_RANDOM=y
> > -CONFIG_HW_RANDOM_VIRTIO=y
> > -CONFIG_SPI=y
> > -CONFIG_SPI_SIFIVE=y
> > -# CONFIG_PTP_1588_CLOCK is not set
> > -CONFIG_DRM=y
> > -CONFIG_DRM_RADEON=y
> > -CONFIG_DRM_VIRTIO_GPU=y
> > -CONFIG_FB=y
> > -CONFIG_FRAMEBUFFER_CONSOLE=y
> > -CONFIG_USB=y
> > -CONFIG_USB_XHCI_HCD=y
> > -CONFIG_USB_XHCI_PLATFORM=y
> > -CONFIG_USB_EHCI_HCD=y
> > -CONFIG_USB_EHCI_HCD_PLATFORM=y
> > -CONFIG_USB_OHCI_HCD=y
> > -CONFIG_USB_OHCI_HCD_PLATFORM=y
> > -CONFIG_USB_STORAGE=y
> > -CONFIG_USB_UAS=y
> > -CONFIG_MMC=y
> > -CONFIG_MMC_SPI=y
> > -CONFIG_RTC_CLASS=y
> > -CONFIG_VIRTIO_PCI=y
> > -CONFIG_VIRTIO_BALLOON=y
> > -CONFIG_VIRTIO_INPUT=y
> > -CONFIG_VIRTIO_MMIO=y
> > -CONFIG_RPMSG_CHAR=y
> > -CONFIG_RPMSG_CTRL=y
> > -CONFIG_RPMSG_VIRTIO=y
> > -CONFIG_EXT4_FS=y
> > -CONFIG_EXT4_FS_POSIX_ACL=y
> > -CONFIG_AUTOFS4_FS=y
> > -CONFIG_MSDOS_FS=y
> > -CONFIG_VFAT_FS=y
> > -CONFIG_TMPFS=y
> > -CONFIG_TMPFS_POSIX_ACL=y
> > -CONFIG_HUGETLBFS=y
> > -CONFIG_NFS_FS=y
> > -CONFIG_NFS_V4=y
> > -CONFIG_NFS_V4_1=y
> > -CONFIG_NFS_V4_2=y
> > -CONFIG_ROOT_NFS=y
> > -CONFIG_9P_FS=y
> > -CONFIG_CRYPTO_USER_API_HASH=y
> > -CONFIG_CRYPTO_DEV_VIRTIO=y
> > -CONFIG_PRINTK_TIME=y
> > -CONFIG_DEBUG_FS=y
> > -CONFIG_DEBUG_PAGEALLOC=y
> > -CONFIG_SCHED_STACK_END_CHECK=y
> > -CONFIG_DEBUG_VM=y
> > -CONFIG_DEBUG_VM_PGFLAGS=y
> > -CONFIG_DEBUG_MEMORY_INIT=y
> > -CONFIG_DEBUG_PER_CPU_MAPS=y
> > -CONFIG_SOFTLOCKUP_DETECTOR=y
> > -CONFIG_WQ_WATCHDOG=y
> > -CONFIG_DEBUG_TIMEKEEPING=y
> > -CONFIG_DEBUG_RT_MUTEXES=y
> > -CONFIG_DEBUG_SPINLOCK=y
> > -CONFIG_DEBUG_MUTEXES=y
> > -CONFIG_DEBUG_RWSEMS=y
> > -CONFIG_DEBUG_ATOMIC_SLEEP=y
> > -CONFIG_STACKTRACE=y
> > -CONFIG_DEBUG_LIST=y
> > -CONFIG_DEBUG_PLIST=y
> > -CONFIG_DEBUG_SG=y
> > -# CONFIG_RCU_TRACE is not set
> > -CONFIG_RCU_EQS_DEBUG=y
> > -# CONFIG_FTRACE is not set
> > -# CONFIG_RUNTIME_TESTING_MENU is not set
> > -CONFIG_MEMTEST=y
> > --
> > 2.36.1
> >



-- 
Best Regards
 Guo Ren