9p queue | Patchew

[PULL 0/2] 9p queue

Christian Schoenebeck posted 2 patches 3 years, 7 months ago

Download series mbox

Test docker-quick@centos7 failed

Test docker-mingw@fedora failed

Test checkpatch failed

Test FreeBSD failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1600166281.git.qemu_oss@crudebyte.com

Maintainers: Christian Schoenebeck <qemu_oss@crudebyte.com>, Greg Kurz <groug@kaod.org>

fsdev/file-op-9p.h | 4 ++++
hw/9pfs/9p-synth.c | 2 ++
hw/9pfs/9p.c       | 9 +++++++++
3 files changed, 15 insertions(+)

Expand all Fold all

[PULL 0/2] 9p queue

Posted by Christian Schoenebeck 3 years, 7 months ago

The following changes since commit 2d2c73d0e3d504a61f868e46e6abd5643f38091b:

  Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20200914-1' into staging (2020-09-14 16:03:08 +0100)

are available in the Git repository at:

  https://github.com/cschoenebeck/qemu.git tags/pull-9p-20200915

for you to fetch changes up to c418f935ac6aed9aa1dc0ceb4b19044a9b7b7968:

  9pfs: disable msize warning for synth driver (2020-09-15 12:12:03 +0200)

----------------------------------------------------------------
The intention of the following two patches is making users aware about
the negative file I/O performance impact when using a very low value
for 9P client parameter 'msize', which especially is the case if no
'msize' parameter was supplied by the user with a 9P Linux client at all.

All it does is logging a performance warning on host side (once) in
that case. By setting 'msize' on client side to any value larger than
8192 the performance warning will disappear.

See https://wiki.qemu.org/Documentation/9psetup#msize for details.

----------------------------------------------------------------
Christian Schoenebeck (2):
      9pfs: log warning if msize <= 8192
      9pfs: disable msize warning for synth driver

 fsdev/file-op-9p.h | 4 ++++
 hw/9pfs/9p-synth.c | 2 ++
 hw/9pfs/9p.c       | 9 +++++++++
 3 files changed, 15 insertions(+)

Re: [PULL 0/2] 9p queue

Posted by Peter Maydell 3 years, 7 months ago

On Wed, 16 Sep 2020 at 10:20, Christian Schoenebeck
<qemu_oss@crudebyte.com> wrote:
>
> The following changes since commit 2d2c73d0e3d504a61f868e46e6abd5643f38091b:
>
>   Merge remote-tracking branch 'remotes/pmaydell/tags/pull-target-arm-20200914-1' into staging (2020-09-14 16:03:08 +0100)
>
> are available in the Git repository at:
>
>   https://github.com/cschoenebeck/qemu.git tags/pull-9p-20200915
>
> for you to fetch changes up to c418f935ac6aed9aa1dc0ceb4b19044a9b7b7968:
>
>   9pfs: disable msize warning for synth driver (2020-09-15 12:12:03 +0200)
>
> ----------------------------------------------------------------
> The intention of the following two patches is making users aware about
> the negative file I/O performance impact when using a very low value
> for 9P client parameter 'msize', which especially is the case if no
> 'msize' parameter was supplied by the user with a 9P Linux client at all.
>
> All it does is logging a performance warning on host side (once) in
> that case. By setting 'msize' on client side to any value larger than
> 8192 the performance warning will disappear.
>
> See https://wiki.qemu.org/Documentation/9psetup#msize for details.
>
> ----------------------------------------------------------------


Applied, thanks.

Please update the changelog at https://wiki.qemu.org/ChangeLog/5.2
for any user-visible changes.

-- PMM