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>] [day] [month] [year] [list]
Message-Id: <20260111-nls_iso8859_1_y_riscv-v1-1-2c992bb2c00d@gmail.com>
Date: Sun, 11 Jan 2026 20:16:13 +1300
From: Javier Carrasco <javier.carrasco.cruz@...il.com>
To: Paul Walmsley <pjw@...nel.org>, Palmer Dabbelt <palmer@...belt.com>, 
 Albert Ou <aou@...s.berkeley.edu>, Alexandre Ghiti <alex@...ti.fr>, 
 Heinrich Schuchardt <heinrich.schuchardt@...onical.com>
Cc: linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org, 
 Javier Carrasco <javier.carrasco.cruz@...il.com>
Subject: [PATCH] riscv: defconfig: enable NLS_ISO8859_1

NLS_ISO8859_1 was enabled as a module with commit efe1e08bca9a ("riscv:
defconfig: enable NLS_CODEPAGE_437, NLS_ISO8859_1"), but the
NLS_CODEPAGE_437 counterpart is selected as built-in. The commit does
not explain the reason behind, and it is not consistent with the
defconfig for ARM64 that also enables these modules to mount EFI system
partitions.

Select NLS_ISO8859_1 as built-in to provide both requirements within the
kernel image.

Signed-off-by: Javier Carrasco <javier.carrasco.cruz@...il.com>
---
I stumbled upon this while scripting commands to launch
qemu-system-aarch64 and qemu-system-riscv64 with debian no-cloud images
and -kernel to provide a defconfig kernel in both cases. I expected the
same behavior to mount the EFI partition, but it failed for riscv unless
the modules were installed/provided. That didn't happen for arm64 as
NLS_ISO8859_1 is built-in.

If there are no good reasons to keep these configurations differently
for these architectures and only have NLS_CODEPAGE_437 bult-in, having
NLS_ISO8859_1 built-in is more consistent and the EFI partitions are
mounted more easily. This is the only configuration that had to be
modified to boot Debian 12 and 13.
---
 arch/riscv/configs/defconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/riscv/configs/defconfig b/arch/riscv/configs/defconfig
index cd736a1d657e..5ca534a51809 100644
--- a/arch/riscv/configs/defconfig
+++ b/arch/riscv/configs/defconfig
@@ -295,7 +295,7 @@ CONFIG_NFS_V4_2=y
 CONFIG_ROOT_NFS=y
 CONFIG_9P_FS=y
 CONFIG_NLS_CODEPAGE_437=y
-CONFIG_NLS_ISO8859_1=m
+CONFIG_NLS_ISO8859_1=y
 CONFIG_SECURITY=y
 CONFIG_SECURITY_SELINUX=y
 CONFIG_SECURITY_APPARMOR=y

---
base-commit: 8f0b4cce4481fb22653697cced8d0d04027cb1e8
change-id: 20260111-nls_iso8859_1_y_riscv-6e8f262cc7da

Best regards,
--  
Javier Carrasco <javier.carrasco.cruz@...il.com>


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ