> Presumably anyone who enables this knows the risk, and if it's a > convenience, why not. > > Documentation/filesystems/hfsplus.rst would be a good place to document > this module parameter please. I’ll add it to the v2 > All these super long lines are an eyesore. How about > > pr_warn("write access to a journaled filesystem is " > "not supported, but has been force enabled.\n"); I’ll fix this, but you'll find a lot of eyesore lines in this driver, which I guess someone would have to fix then.