From patchwork Thu Mar 3 18:37:32 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Michael Tremer X-Patchwork-Id: 5307 Return-Path: Received: from mail01.ipfire.org (mail01.haj.ipfire.org [172.28.1.202]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384 client-signature ECDSA (P-384) client-digest SHA384) (Client CN "mail01.haj.ipfire.org", Issuer "R3" (verified OK)) by web04.haj.ipfire.org (Postfix) with ESMTPS id 4K8fqW18ZWz3xgV for ; Thu, 3 Mar 2022 18:37:43 +0000 (UTC) Received: from mail02.haj.ipfire.org (mail02.haj.ipfire.org [172.28.1.201]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) client-signature ECDSA (P-384)) (Client CN "mail02.haj.ipfire.org", Issuer "R3" (verified OK)) by mail01.ipfire.org (Postfix) with ESMTPS id 4K8fqS4FBcz1yr; Thu, 3 Mar 2022 18:37:40 +0000 (UTC) Received: from mail02.haj.ipfire.org (localhost [127.0.0.1]) by mail02.haj.ipfire.org (Postfix) with ESMTP id 4K8fqS34Prz2yf3; Thu, 3 Mar 2022 18:37:40 +0000 (UTC) Received: from mail01.ipfire.org (mail01.haj.ipfire.org [172.28.1.202]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384 client-signature ECDSA (P-384) client-digest SHA384) (Client CN "mail01.haj.ipfire.org", Issuer "R3" (verified OK)) by mail02.haj.ipfire.org (Postfix) with ESMTPS id 4K8fqR1wxQz2yDF for ; Thu, 3 Mar 2022 18:37:39 +0000 (UTC) Received: from [127.0.0.1] (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384) server-digest SHA384) (No client certificate requested) by mail01.ipfire.org (Postfix) with ESMTPSA id 4K8fqP3sxYz1yr; Thu, 3 Mar 2022 18:37:37 +0000 (UTC) DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=ipfire.org; s=202003ed25519; t=1646332657; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=CzO2KJMazIBeVkK2dageDH5ph4fxVJ77s76rcXg2chQ=; b=0rDRr4dxiUUPZyrZgOrW+WbLn2edouqPC/0ohVLKrQMS87dPg4qhhbGNJQYOk8wDpFPzhs x0kgZQIsOG7xH7DA== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ipfire.org; s=202003rsa; t=1646332657; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version: content-transfer-encoding:content-transfer-encoding; bh=CzO2KJMazIBeVkK2dageDH5ph4fxVJ77s76rcXg2chQ=; b=ZnKY/Dq9ZRwGHESz+uEeZBUYdxrhKCWrNxRb2W0q6Sut2kYT7yz5wZS3s+Bus+Me3ac08o HxDeCsH3Zo9R8V+vbvm4GfoRTMeoBs4bDdDxyS1HIlEiSyBzrtkrmejKO0bv9TZKs3kB5J 7DY1Obm36I9gQHcrOyfOTVe+ox4gdmj0mXUu1/9ODWS70bKKruNmwyU8hLh27ygxIbrA74 2hoVS4SN0nlmNLdJ8/Y8M3cxYwv08K1+M5nGts/VwuTG5r2yjnqxLKqY90KTa5emN7UfuU 5lSNJ29eOJGr4hCWT2yMgSyYva3LzoP+ykus5ultv+fpkw3we+miDtbPTfSzRQ== From: Michael Tremer To: development@lists.ipfire.org Subject: [PATCH] installer: Remove support for ReiserFS Date: Thu, 3 Mar 2022 18:37:32 +0000 Message-Id: <20220303183732.1986097-1-michael.tremer@ipfire.org> MIME-Version: 1.0 X-BeenThere: development@lists.ipfire.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: IPFire development talk List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Michael Tremer Errors-To: development-bounces@lists.ipfire.org Sender: "Development" https://lore.kernel.org/lkml/YhIwUEpymVzmytdp@casper.infradead.org/ ReiserFS is an old file system which has not been actively maintained in the Linux kernel for a long time. It is potentially going to be removed soon which is why we shouldn't encourage people to create new installations with ReiserFS any more. This patch removes support for ReiserFS from the installer. We should keep it enabled in the kernel for as long as it is available, but we will have to encourage users potentially to re-install on a different file system. Since ReiserFS isn't very popular any more, I don't think that there will be many users left. Signed-off-by: Michael Tremer Acked-by: Peter Müller --- src/installer/dracut-module/module-setup.sh | 4 ++-- src/installer/hw.c | 7 ------- src/installer/hw.h | 9 ++++----- src/installer/main.c | 3 +-- 4 files changed, 7 insertions(+), 16 deletions(-) diff --git a/src/installer/dracut-module/module-setup.sh b/src/installer/dracut-module/module-setup.sh index c68b51d26..26a257cf8 100755 --- a/src/installer/dracut-module/module-setup.sh +++ b/src/installer/dracut-module/module-setup.sh @@ -30,8 +30,8 @@ install() { instmods virtio_net hv_netvsc vmxnet3 # Filesystem support - inst_multiple parted mkswap mke2fs mkreiserfs mkfs.xfs mkfs.vfat - instmods ext4 iso9660 reiserfs vfat xfs ntfs3 + inst_multiple parted mkswap mke2fs mkfs.xfs mkfs.vfat + instmods ext4 iso9660 vfat xfs ntfs3 # Extraction inst_multiple tar gzip zstd diff --git a/src/installer/hw.c b/src/installer/hw.c index 17e0bbb01..5aa162266 100644 --- a/src/installer/hw.c +++ b/src/installer/hw.c @@ -811,9 +811,6 @@ static int hw_format_filesystem(const char* path, int fs, const char* output) { // Swap if (fs == HW_FS_SWAP) { snprintf(cmd, sizeof(cmd), "/sbin/mkswap -v1 %s &>/dev/null", path); - // ReiserFS - } else if (fs == HW_FS_REISERFS) { - snprintf(cmd, sizeof(cmd), "/sbin/mkreiserfs -f %s ", path); // EXT4 } else if (fs == HW_FS_EXT4) { @@ -878,10 +875,6 @@ int hw_mount_filesystems(struct hw_destination* dest, const char* prefix) { const char* filesystem; switch (dest->filesystem) { - case HW_FS_REISERFS: - filesystem = "reiserfs"; - break; - case HW_FS_EXT4: case HW_FS_EXT4_WO_JOURNAL: filesystem = "ext4"; diff --git a/src/installer/hw.h b/src/installer/hw.h index b11dfa48f..6a7f23aa7 100644 --- a/src/installer/hw.h +++ b/src/installer/hw.h @@ -42,11 +42,10 @@ #define HW_PART_TABLE_GPT 1 #define HW_FS_SWAP 0 -#define HW_FS_REISERFS 1 -#define HW_FS_EXT4 2 -#define HW_FS_EXT4_WO_JOURNAL 3 -#define HW_FS_XFS 4 -#define HW_FS_FAT32 5 +#define HW_FS_EXT4 1 +#define HW_FS_EXT4_WO_JOURNAL 2 +#define HW_FS_XFS 3 +#define HW_FS_FAT32 4 #define HW_FS_DEFAULT HW_FS_EXT4 diff --git a/src/installer/main.c b/src/installer/main.c index b31b096a5..06dd9caf5 100644 --- a/src/installer/main.c +++ b/src/installer/main.c @@ -720,7 +720,6 @@ int main(int argc, char *argv[]) { { HW_FS_EXT4, _("ext4 Filesystem") }, { HW_FS_EXT4_WO_JOURNAL, _("ext4 Filesystem without journal") }, { HW_FS_XFS, _("XFS Filesystem") }, - { HW_FS_REISERFS, _("ReiserFS Filesystem") }, { 0, NULL }, }; unsigned int num_filesystems = sizeof(filesystems) / sizeof(*filesystems); @@ -735,7 +734,7 @@ int main(int argc, char *argv[]) { } rc = newtWinMenu(_("Filesystem Selection"), _("Please choose your filesystem:"), - 50, 5, 5, 6, fs_names, &fs_choice, _("OK"), _("Cancel"), NULL); + 50, 5, 5, 5, fs_names, &fs_choice, _("OK"), _("Cancel"), NULL); if (rc == 2) goto EXIT;