[PATCH 2/3] afs: Fix afs_write_end() to handle short writes

David Howells dhowells at redhat.com
Mon Jun 14 14:11:26 PDT 2021


Matthew Wilcox <willy at infradead.org> wrote:

> > means you can't get there unless PageUptodate() is true by that point.
> 
> Isn't the point of an assertion to check that this is true?

The assertion was meant to check that that it was true given that the page was
set uptodate somewhere else before this function was even called.  With this
patch, however, it's now set in this function if it wasn't already right at
the top - so the assertion should now be redundant.  I can put it back if you
really insist.

David




More information about the linux-afs mailing list