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

[David Howells]

Matthew Wilcox <willy@xxxxxxxxxxxxx> 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