On Wed, May 27, 2020 at 10:40 PM Christoph Hellwig <hch@xxxxxx> wrote: > > this series fixes a few issues and cleans up the helpers that read from > or write to kernel space buffers, and ensures that we don't change the > address limit if we are using the ->read_iter and ->write_iter methods > that don't need the changed address limit. Apart from the "please don't mix irrelevant whitespace changes with other changes" comment, this looks fine to me. And a rant related to that change: I'm really inclined to remove the checkpatch check for 80 columns entirely, but it shouldn't have been triggering for old lines even now. Or maybe make it check for something more reasonable, like 100 characters. I find it ironic and annoying how "checkpatch" warns about that silly legacy limit, when checkpatch itself then on the very next few lines has a line that is 124 columns wide And yes, that 124 character line has a good reason for it. But that's kind of the point. There are lots of perfectly fine reasons for longer lines. I'd much rather check for "no deep indentation" or "no unnecessarily complex conditionals" or other issues that are more likely to be _real_ problems. But do we really have 80x25 terminals any more that we'd care about? Linus