[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 0/8] 9p queue 2021-10-27
From: |
Christian Schoenebeck |
Subject: |
Re: [PULL 0/8] 9p queue 2021-10-27 |
Date: |
Wed, 27 Oct 2021 16:05:39 +0200 |
On Mittwoch, 27. Oktober 2021 15:18:33 CEST Christian Schoenebeck wrote:
> The following changes since commit 931ce30859176f0f7daac6bac255dae5eb21284e:
>
> Merge remote-tracking branch 'remotes/dagrh/tags/pull-virtiofs-20211026'
> into staging (2021-10-26 07:38:41 -0700)
>
> are available in the Git repository at:
>
> https://github.com/cschoenebeck/qemu.git tags/pull-9p-20211027
>
> for you to fetch changes up to 7e985780aaab93d2c5be9b62d8d386568dfb071e:
>
> 9pfs: use P9Array in v9fs_walk() (2021-10-27 14:45:22 +0200)
>
> ----------------------------------------------------------------
> 9pfs: performance fix and cleanup
>
> * First patch fixes suboptimal I/O performance on guest due to previously
> incorrect block size being transmitted to 9p client.
>
> * Subsequent patches are cleanup ones intended to reduce code complexity.
>
> ----------------------------------------------------------------
> Christian Schoenebeck (8):
> 9pfs: fix wrong I/O block size in Rgetattr
> 9pfs: deduplicate iounit code
> 9pfs: simplify blksize_to_iounit()
> 9pfs: introduce P9Array
> fsdev/p9array.h: check scalar type in P9ARRAY_NEW()
> 9pfs: make V9fsString usable via P9Array API
> 9pfs: make V9fsPath usable via P9Array API
> 9pfs: use P9Array in v9fs_walk()
>
> fsdev/9p-marshal.c | 2 +
> fsdev/9p-marshal.h | 3 +
> fsdev/file-op-9p.h | 2 +
> fsdev/p9array.h | 160
> +++++++++++++++++++++++++++++++++++++++++++++++++++++ hw/9pfs/9p.c |
> 70 +++++++++++++----------
> 5 files changed, 208 insertions(+), 29 deletions(-)
> create mode 100644 fsdev/p9array.h
Regarding last 5 patches: Daniel raised a concern that not using g_autoptr
would deviate from current QEMU coding patterns:
https://lists.gnu.org/archive/html/qemu-devel/2021-10/msg00081.html
Unfortunately I saw no way to address his concern without adding unnecessary
code complexity, so I decided to make this a 9p local type (QArray -> P9Array)
for now, which can easily be replaced in future (e.g. when there will be
something appropriate on glib side).
Best regards,
Christian Schoenebeck
- [PULL 0/8] 9p queue 2021-10-27, Christian Schoenebeck, 2021/10/27
- [PULL 1/8] 9pfs: fix wrong I/O block size in Rgetattr, Christian Schoenebeck, 2021/10/27
- [PULL 2/8] 9pfs: deduplicate iounit code, Christian Schoenebeck, 2021/10/27
- [PULL 3/8] 9pfs: simplify blksize_to_iounit(), Christian Schoenebeck, 2021/10/27
- [PULL 4/8] 9pfs: introduce P9Array, Christian Schoenebeck, 2021/10/27
- [PULL 5/8] fsdev/p9array.h: check scalar type in P9ARRAY_NEW(), Christian Schoenebeck, 2021/10/27
- [PULL 7/8] 9pfs: make V9fsPath usable via P9Array API, Christian Schoenebeck, 2021/10/27
- [PULL 6/8] 9pfs: make V9fsString usable via P9Array API, Christian Schoenebeck, 2021/10/27
- [PULL 8/8] 9pfs: use P9Array in v9fs_walk(), Christian Schoenebeck, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27,
Christian Schoenebeck <=
- Re: [PULL 0/8] 9p queue 2021-10-27, Philippe Mathieu-Daudé, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Christian Schoenebeck, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Philippe Mathieu-Daudé, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Christian Schoenebeck, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Philippe Mathieu-Daudé, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Richard Henderson, 2021/10/27
- Re: [PULL 0/8] 9p queue 2021-10-27, Christian Schoenebeck, 2021/10/28
Re: [PULL 0/8] 9p queue 2021-10-27, Greg Kurz, 2021/10/27
Re: [PULL 0/8] 9p queue 2021-10-27, Richard Henderson, 2021/10/27