Re: 9p read corruption of mmaped content (Was: [PATCH] 9p/virtio: restrict page pinning to user_backed_iter() iovec)

[David Howells]

Dominique Martinet <asmadeus@xxxxxxxxxxxxx> wrote:

>    netfs_collect_folio: R=00001b55 ix=00003 r=3000-4000 t=3000/5fb2
>    netfs_folio: i=157f3 ix=00003-00003 read-done
>    netfs_folio: i=157f3 ix=00003-00003 read-unlock
>    netfs_collect_folio: R=00001b55 ix=00004 r=4000-5000 t=4000/5fb2
>    netfs_folio: i=157f3 ix=00004-00004 read-done
>    netfs_folio: i=157f3 ix=00004-00004 read-unlock
>    netfs_collect_folio: R=00001b55 ix=00005 r=5000-5fb2 t=5000/5fb2
>    netfs_folio: i=157f3 ix=00005-00005 read-done
>    netfs_folio: i=157f3 ix=00005-00005 read-unlock
>    ...
>    netfs_collect_stream: R=00001b55[0:] cto=5fb2 frn=ffffffff
>    netfs_collect_state: R=00001b55 col=5fb2 cln=6000 n=c
>    netfs_collect_stream: R=00001b55[0:] cto=5fb2 frn=ffffffff
>    netfs_collect_state: R=00001b55 col=5fb2 cln=6000 n=8
>    ...
>    netfs_sreq: R=00001b55[2] ZERO SUBMT f=000 s=5fb2 0/4e s=0 e=0
>    netfs_sreq: R=00001b55[2] ZERO TERM  f=102 s=5fb2 4e/4e s=5 e=0

This would seem to show a problem, if not the problem.

We unlocked page ix=00005 before doing the ZERO subreq that clears the page
tail.  That shouldn't have happened since the collection point hasn't reached
the end of the folio yet.

David