[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 0/7] 9pfs: fix 'Twalk' protocol violation
From: |
Christian Schoenebeck |
Subject: |
Re: [PATCH v4 0/7] 9pfs: fix 'Twalk' protocol violation |
Date: |
Tue, 29 Mar 2022 12:21:13 +0200 |
On Dienstag, 15. März 2022 11:10:25 CEST Christian Schoenebeck wrote:
> Currently the implementation of 'Twalk' does not behave exactly as specified
> by the 9p2000 protocol specification. Actual fix is patch 5; see the
> description of that patch for details of what this overall fix and series
> is about.
>
> PREREQUISITES
> =============
>
> This series requires the following additional patch to work correctly:
> E1nTpyU-0000yR-9o@lizzy.crudebyte.com/">https://lore.kernel.org/qemu-devel/E1nTpyU-0000yR-9o@lizzy.crudebyte.com/
>
> OVERVIEW OF PATCHES
> ===================
>
> Patch 4 is a preparatory (pure) refactoring change to make actual 'Twalk'
> fix patch 5 better readable.
>
> All the other patches are just additional test cases for guarding 'Twalk'
> behaviour.
>
> v3 -> v4:
>
> * QID returned by Twalk request in fs_walk_2nd_nonexistent() test should
> NOT be identical to root node's QID. [patch 7]
>
> * Fix actual 'fid unaffected' check in fs_walk_2nd_nonexistent() test by
> sending a subsequent 'Tgetattr' request. [patch 7]
>
> Christian Schoenebeck (7):
> tests/9pfs: walk to non-existent dir
> tests/9pfs: Twalk with nwname=0
> tests/9pfs: compare QIDs in fs_walk_none() test
> 9pfs: refactor 'name_idx' -> 'nwalked' in v9fs_walk()
> 9pfs: fix 'Twalk' to only send error if no component walked
> tests/9pfs: guard recent 'Twalk' behaviour fix
> tests/9pfs: check fid being unaffected in fs_walk_2nd_nonexistent
>
> hw/9pfs/9p.c | 57 ++++++----
> tests/qtest/virtio-9p-test.c | 201 ++++++++++++++++++++++++++++++++++-
> 2 files changed, 231 insertions(+), 27 deletions(-)
ping
No hurry, as this is something for the subsequent QEMU release cycle, but
would be good to know whether you will be able to look at this at all.
Best regards,
Christian Schoenebeck
- [PATCH v4 0/7] 9pfs: fix 'Twalk' protocol violation, Christian Schoenebeck, 2022/03/15
- [PATCH v4 1/7] tests/9pfs: walk to non-existent dir, Christian Schoenebeck, 2022/03/15
- [PATCH v4 2/7] tests/9pfs: Twalk with nwname=0, Christian Schoenebeck, 2022/03/15
- [PATCH v4 3/7] tests/9pfs: compare QIDs in fs_walk_none() test, Christian Schoenebeck, 2022/03/15
- [PATCH v4 4/7] 9pfs: refactor 'name_idx' -> 'nwalked' in v9fs_walk(), Christian Schoenebeck, 2022/03/15
- [PATCH v4 5/7] 9pfs: fix 'Twalk' to only send error if no component walked, Christian Schoenebeck, 2022/03/15
- [PATCH v4 7/7] tests/9pfs: check fid being unaffected in fs_walk_2nd_nonexistent, Christian Schoenebeck, 2022/03/15
- [PATCH v4 6/7] tests/9pfs: guard recent 'Twalk' behaviour fix, Christian Schoenebeck, 2022/03/15
- Re: [PATCH v4 0/7] 9pfs: fix 'Twalk' protocol violation,
Christian Schoenebeck <=