Re: [PATCH] hfsplus: Add module parameter to enable force writes

From: Aditya Garg
Date: Sun Dec 04 2022 - 06:02:06 EST




> On 04-Dec-2022, at 1:37 PM, Christoph Hellwig <hch@xxxxxx> wrote:
>
> NAK. Global module options overriding mount options don't make sense.
> If you mount so much write yourself a script, or add the option to the
> configuration of the automounting tool of your choice.

Hi Christoph
My bad I wasn't aware udisks2 now supports custom configuration. I just came to know about this and thus this patch of mine indeed does not make any sense now.

Although, if you think its worth it, the following improvements can be made :-

1. There is no logging showing that writes have been force enabled. We could add that.
2. We could have separate mount options for journaled and locked volumes (although I dunno in what case we get locked volumes).