Re: [PATCH] fs: ntfs: Mark NTFS_RW as BROKEN for PPC

From: Linus Torvalds
Date: Mon Nov 22 2021 - 15:24:04 EST


On Mon, Nov 22, 2021 at 12:13 PM Guenter Roeck <linux@xxxxxxxxxxxx> wrote:
>
> The Kconfig language does not support "depends on XXX if YYY",
> so this is the next best choice.

Ugh. This is hard to read, and it will get much worse if we then end
up making the conditional more complicated if we actually want to
introduce some page size logic..

> config NTFS_RW
> bool "NTFS write support"
> + depends on BROKEN || !PPC
> depends on NTFS_FS
> help
> This enables the partial, but safe, write support in the NTFS driver.

How about we do a

config DISABLE_NTFS_RW
bool
default y if PPC

and then in NTFS_RW we just do

depends on !DISABLE_NTFS_RW

which at least makes each case easy, and makes it much clearer if we
extend that DISABLE_NTFS_RW condition later.

Linus