2024-02-21 12:59:45

by Javier Martinez Canillas

[permalink] [raw]
Subject: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

These options are needed by some Linux distributions (e.g: Fedora), so
let's enable them to make it easier for developers using such distros.

Signed-off-by: Javier Martinez Canillas <[email protected]>
---

arch/arm64/configs/defconfig | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 056a6cc546a4..12aebbba814e 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -241,6 +241,8 @@ CONFIG_PCI_EPF_TEST=m
CONFIG_DEVTMPFS=y
CONFIG_DEVTMPFS_MOUNT=y
CONFIG_FW_LOADER_USER_HELPER=y
+CONFIG_FW_LOADER_COMPRESS=y
+CONFIG_MODULE_COMPRESS_XZ=y
CONFIG_HISILICON_LPC=y
CONFIG_TEGRA_ACONNECT=m
CONFIG_MHI_BUS_PCI_GENERIC=m
@@ -275,6 +277,8 @@ CONFIG_MTD_NAND_FSL_IFC=y
CONFIG_MTD_NAND_QCOM=y
CONFIG_MTD_SPI_NOR=y
CONFIG_MTD_UBI=m
+CONFIG_ZRAM=m
+CONFIG_ZRAM_WRITEBACK=y
CONFIG_BLK_DEV_LOOP=y
CONFIG_BLK_DEV_NBD=m
CONFIG_VIRTIO_BLK=y
@@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
CONFIG_EXT2_FS=y
CONFIG_EXT3_FS=y
CONFIG_EXT4_FS_POSIX_ACL=y
+CONFIG_XFS_FS=m
CONFIG_BTRFS_FS=m
CONFIG_BTRFS_FS_POSIX_ACL=y
CONFIG_FANOTIFY=y
--
2.43.0



2024-02-21 14:04:17

by Brian Masney

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

On Wed, Feb 21, 2024 at 01:59:17PM +0100, Javier Martinez Canillas wrote:
> These options are needed by some Linux distributions (e.g: Fedora), so
> let's enable them to make it easier for developers using such distros.
>
> Signed-off-by: Javier Martinez Canillas <[email protected]>

Reviewed-by: Brian Masney <[email protected]>


2024-02-21 15:32:03

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
> These options are needed by some Linux distributions (e.g: Fedora), so
> let's enable them to make it easier for developers using such distros.
>
> Signed-off-by: Javier Martinez Canillas <[email protected]>
> ---

Acked-by: Arnd Bergmann <[email protected]>

> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
> CONFIG_EXT2_FS=y
> CONFIG_EXT3_FS=y
> CONFIG_EXT4_FS_POSIX_ACL=y
> +CONFIG_XFS_FS=m
> CONFIG_BTRFS_FS=m
> CONFIG_BTRFS_FS_POSIX_ACL=y

Unfortunately this will increase build time noticeably, but
I agree it is the right thing to do.

Can you send it (with any other acks) to [email protected] so
I can merge it from there?

Arnd

2024-02-21 15:39:29

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

On 21/02/2024 16:31, Arnd Bergmann wrote:
> On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
>> These options are needed by some Linux distributions (e.g: Fedora), so
>> let's enable them to make it easier for developers using such distros.
>>
>> Signed-off-by: Javier Martinez Canillas <[email protected]>
>> ---
>
> Acked-by: Arnd Bergmann <[email protected]>
>
>> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
>> CONFIG_EXT2_FS=y
>> CONFIG_EXT3_FS=y
>> CONFIG_EXT4_FS_POSIX_ACL=y
>> +CONFIG_XFS_FS=m
>> CONFIG_BTRFS_FS=m
>> CONFIG_BTRFS_FS_POSIX_ACL=y
>
> Unfortunately this will increase build time noticeably, but
> I agree it is the right thing to do.
>
> Can you send it (with any other acks) to [email protected] so
> I can merge it from there?

Due to XFS, please include:

Nacked-by: Krzysztof Kozlowski <[email protected]>

so this will not be precedence used for further patches like this
(Maxime already used such argument, so I want to be sure it will not be
used against me in the future)

Best regards,
Krzysztof


2024-02-21 15:49:38

by Maxime Ripard

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

On Wed, Feb 21, 2024 at 04:39:18PM +0100, Krzysztof Kozlowski wrote:
> On 21/02/2024 16:31, Arnd Bergmann wrote:
> > On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
> >> These options are needed by some Linux distributions (e.g: Fedora), so
> >> let's enable them to make it easier for developers using such distros.
> >>
> >> Signed-off-by: Javier Martinez Canillas <[email protected]>
> >> ---
> >
> > Acked-by: Arnd Bergmann <[email protected]>
> >
> >> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
> >> CONFIG_EXT2_FS=y
> >> CONFIG_EXT3_FS=y
> >> CONFIG_EXT4_FS_POSIX_ACL=y
> >> +CONFIG_XFS_FS=m
> >> CONFIG_BTRFS_FS=m
> >> CONFIG_BTRFS_FS_POSIX_ACL=y
> >
> > Unfortunately this will increase build time noticeably, but
> > I agree it is the right thing to do.
> >
> > Can you send it (with any other acks) to [email protected] so
> > I can merge it from there?
>
> Due to XFS, please include:
>
> Nacked-by: Krzysztof Kozlowski <[email protected]>
>
> so this will not be precedence used for further patches like this
> (Maxime already used such argument, so I want to be sure it will not be
> used against me in the future)

For the record, I didn't use anything against *you*. I'd just like
consistency over how defconfig rules are applied.

Maxime


Attachments:
(No filename) (1.30 kB)
signature.asc (235.00 B)
Download all attachments

2024-02-21 15:53:00

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

On 21/02/2024 16:49, Maxime Ripard wrote:
> On Wed, Feb 21, 2024 at 04:39:18PM +0100, Krzysztof Kozlowski wrote:
>> On 21/02/2024 16:31, Arnd Bergmann wrote:
>>> On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
>>>> These options are needed by some Linux distributions (e.g: Fedora), so
>>>> let's enable them to make it easier for developers using such distros.
>>>>
>>>> Signed-off-by: Javier Martinez Canillas <[email protected]>
>>>> ---
>>>
>>> Acked-by: Arnd Bergmann <[email protected]>
>>>
>>>> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
>>>> CONFIG_EXT2_FS=y
>>>> CONFIG_EXT3_FS=y
>>>> CONFIG_EXT4_FS_POSIX_ACL=y
>>>> +CONFIG_XFS_FS=m
>>>> CONFIG_BTRFS_FS=m
>>>> CONFIG_BTRFS_FS_POSIX_ACL=y
>>>
>>> Unfortunately this will increase build time noticeably, but
>>> I agree it is the right thing to do.
>>>
>>> Can you send it (with any other acks) to [email protected] so
>>> I can merge it from there?
>>
>> Due to XFS, please include:
>>
>> Nacked-by: Krzysztof Kozlowski <[email protected]>
>>
>> so this will not be precedence used for further patches like this
>> (Maxime already used such argument, so I want to be sure it will not be
>> used against me in the future)
>
> For the record, I didn't use anything against *you*. I'd just like
> consistency over how defconfig rules are applied.

True, apologies if it seemed to harsh. I meant the discussion, so maybe
better:
"Maxime used such argument with discussion with me, so I want to be sure
there is no silent ack from me in the future."

Best regards,
Krzysztof


2024-02-21 19:24:12

by Javier Martinez Canillas

[permalink] [raw]
Subject: Re: [PATCH] arm64: defconfig: Enable zram, xfs and loading compressed FW support

Krzysztof Kozlowski <[email protected]> writes:

> On 21/02/2024 16:31, Arnd Bergmann wrote:
>> On Wed, Feb 21, 2024, at 13:59, Javier Martinez Canillas wrote:
>>> These options are needed by some Linux distributions (e.g: Fedora), so
>>> let's enable them to make it easier for developers using such distros.
>>>
>>> Signed-off-by: Javier Martinez Canillas <[email protected]>
>>> ---
>>
>> Acked-by: Arnd Bergmann <[email protected]>
>>
>>> @@ -1595,6 +1599,7 @@ CONFIG_HTE_TEGRA194_TEST=m
>>> CONFIG_EXT2_FS=y
>>> CONFIG_EXT3_FS=y
>>> CONFIG_EXT4_FS_POSIX_ACL=y
>>> +CONFIG_XFS_FS=m
>>> CONFIG_BTRFS_FS=m
>>> CONFIG_BTRFS_FS_POSIX_ACL=y
>>
>> Unfortunately this will increase build time noticeably, but
>> I agree it is the right thing to do.
>>
>> Can you send it (with any other acks) to [email protected] so
>> I can merge it from there?
>
> Due to XFS, please include:
>
> Nacked-by: Krzysztof Kozlowski <[email protected]>
>

Don't worry, I won't to re-send anything if you disagree with the change.

> so this will not be precedence used for further patches like this
> (Maxime already used such argument, so I want to be sure it will not be
> used against me in the future)
>

I used the same argument and is a fair one in my opinion. Removing other
filesystems should send the correct message, if that is the agreegment.

--
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat