Skip to contentSkip to conversation

Test12 ll_rw_block error.

DISCUSSION Linus Torvalds 1 participant 1 message Dec 15, 2000 1 message not in the archive

lore
Share & tools
lore

A message that is not in the archive (sent off-list or not yet received): [email protected]

#1 lore raw
Quoted 8 lines · Russell Cattelan

So one more observation in
filemap_sync_pte

lock_page(page);
error = filemap_write_page(page, 1);
-> UnlockPage(page);
This unlock page was removed? is that correct?

Yes. The "writepage" thing changed: "struct file" disappeared (as I'm sure you also noticed), and the page writer is supposed to unlock the page itself. Which it may do at any time, of course.

There are some reasons to do it only after the IO has actually completed:
this way the VM layer won't try to write it out _again_ before the first IO hasn't even finished yet, and the writing logic can possibly be simplified if you know that nobody else will be touching that page.

But that is up to you: you can do the UnlockPage before even returning from your "->writepage()" function, if you choose to do so.

Linus

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected]
Please read the FAQ at http://www.tux.org/lkml/