Re: [PATCH 1/4] MAINTAINERS: drop audio maintainership

2024-05-23 Thread Christian Schoenebeck
On Wednesday, May 22, 2024 2:54:51 PM CEST Thomas Huth wrote: > On 16/05/2024 14.03, Gerd Hoffmann wrote: [...] > > ALSA Audio backend > > -M: Gerd Hoffmann > > R: Christian Schoenebeck > > S: Odd Fixes > > F: audio/alsaaudio.c > > I'd also sugge

Re: [PATCH for-9.0 v3 2/2] qtest/virtio-9p-test.c: remove g_test_slow() gate

2024-04-17 Thread Christian Schoenebeck
On Wednesday, April 17, 2024 1:16:02 AM CEST Daniel Henrique Barboza wrote: > > On 4/16/24 16:54, Michael Tokarev wrote: > > 27.03.2024 17:20, Daniel Henrique Barboza : > >> Commit 558f5c42ef gated the local tests with g_test_slow() to skip them > >> in 'make check'. The reported issue back then

[PULL for-9.0 2/2] qtest/virtio-9p-test.c: remove g_test_slow() gate

2024-03-30 Thread Christian Schoenebeck
istian Schoenebeck Tested-by: Thomas Huth Signed-off-by: Christian Schoenebeck --- tests/qtest/virtio-9p-test.c | 9 - 1 file changed, 9 deletions(-) diff --git a/tests/qtest/virtio-9p-test.c b/tests/qtest/virtio-9p-test.c index 0179b3a394..3c8cd235cf 100644 --- a/tests/qtest/virtio-9p-test.c

[PULL for-9.0 1/2] qtest/virtio-9p-test.c: create/remove temp dirs after each test

2024-03-30 Thread Christian Schoenebeck
Message-Id: <20240327142011.805728-2-dbarb...@ventanamicro.com> Reviewed-by: Greg Kurz Reviewed-by: Christian Schoenebeck Tested-by: Thomas Huth Signed-off-by: Christian Schoenebeck --- tests/qtest/virtio-9p-test.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletion

[PULL for-9.0 0/2] 9p queue 2024-03-29

2024-03-30 Thread Christian Schoenebeck
The following changes since commit 5012e522aca161be5c141596c66e5cc6082538a9: Update version for v9.0.0-rc1 release (2024-03-26 19:46:55 +) are available in the Git repository at: https://github.com/cschoenebeck/qemu.git tags/pull-9p-20240329 for you to fetch changes up to

Re: [PATCH for-9.0 v3 0/2] qtest/virtio-9p-test.c: fix slow tests

2024-03-28 Thread Christian Schoenebeck
On Wednesday, March 27, 2024 3:20:09 PM CET Daniel Henrique Barboza wrote: > Hi, > > In this new version we took a different approach after the discussions > we had in [1]. The tests are now untouched, and we're addressing the root > cause directly: the fact that we have a single temp dir for all

Re: [PATCH for-9.0 v3 0/2] qtest/virtio-9p-test.c: fix slow tests

2024-03-27 Thread Christian Schoenebeck
): > qtest/virtio-9p-test.c: create/remove temp dirs after each test > qtest/virtio-9p-test.c: remove g_test_slow() gate > > tests/qtest/virtio-9p-test.c | 32 +++- > 1 file changed, 11 insertions(+), 21 deletions(-) > > Awesome! Revie

Re: [PATCH for-9.0 1/3] qtest/virtio-9p-test.c: consolidate create dir, file and symlink tests

2024-03-27 Thread Christian Schoenebeck
On Wednesday, March 27, 2024 12:28:17 PM CET Daniel Henrique Barboza wrote: > On 3/27/24 07:14, Christian Schoenebeck wrote: > > On Wednesday, March 27, 2024 10:33:27 AM CET Daniel Henrique Barboza wrote: > >> On 3/27/24 05:47, Christian Schoenebeck wrote: > >>> On T

Re: [PATCH for-9.0 1/3] qtest/virtio-9p-test.c: consolidate create dir, file and symlink tests

2024-03-27 Thread Christian Schoenebeck
On Wednesday, March 27, 2024 10:33:27 AM CET Daniel Henrique Barboza wrote: > On 3/27/24 05:47, Christian Schoenebeck wrote: > > On Tuesday, March 26, 2024 6:47:17 PM CET Daniel Henrique Barboza wrote: > >> On 3/26/24 14:05, Greg Kurz wrote: > >>> On Tue, 26 Mar 20

Re: [PATCH for-9.0 0/3] qtest/virtio-9p-test.c: fix slow tests

2024-03-27 Thread Christian Schoenebeck
On Tuesday, March 26, 2024 5:07:16 PM CET Daniel Henrique Barboza wrote: > > On 3/26/24 12:55, Greg Kurz wrote: > > Bom dia Daniel ! > > Bonne après-midi ! > > > > > On Tue, 26 Mar 2024 10:26:03 -0300 > > Daniel Henrique Barboza wrote: > > > >> Hi, > >> > >> Thomas reported in [1] a problem

Re: [PATCH for-9.0 1/3] qtest/virtio-9p-test.c: consolidate create dir, file and symlink tests

2024-03-27 Thread Christian Schoenebeck
On Tuesday, March 26, 2024 6:47:17 PM CET Daniel Henrique Barboza wrote: > On 3/26/24 14:05, Greg Kurz wrote: > > On Tue, 26 Mar 2024 10:26:04 -0300 > > Daniel Henrique Barboza wrote: > > > >> The local 9p driver in virtio-9p-test.c its temporary dir right at the > >> start of qos-test (via

Re: [PULL 20/34] tests/libqos: add riscv/virt machine nodes

2024-03-25 Thread Christian Schoenebeck
systems as reported here: https://lists.nongnu.org/archive/html/qemu-devel/2020-11/msg05510.html This will need to be investigated and addressed later. Let's go for a workaround in the meantime : mark the "local" tests as "slow" so that they aren

[PATCH 1/1] tests/9p: add 'use-after-unlink' test

2024-02-21 Thread Christian Schoenebeck
After removing a file from the file system, we should still be able to do I/O on the file if we already had it open before removal. Signed-off-by: Christian Schoenebeck --- OK, this was a bit surprising to me. I was expecting this test to fail, but it works! tests/qtest/virtio-9p-test.c

Re: [PATCH 2/6] hw/9pfs/9p-synth: Use RCU_READ macro

2024-01-24 Thread Christian Schoenebeck
thieu-Daudé > --- Acked-by: Christian Schoenebeck > hw/9pfs/9p-synth.c | 24 > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/hw/9pfs/9p-synth.c b/hw/9pfs/9p-synth.c > index 0ac79a500b..419ea69e3a 100644 > --- a/hw/9pfs/9p-synth.c &

Re: [PATCH 20/71] hw/9pfs: Constify VMState

2023-11-06 Thread Christian Schoenebeck
On Monday, November 6, 2023 7:57:36 AM CET Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- Acked-by: Christian Schoenebeck > hw/9pfs/virtio-9p-device.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/9pfs/virtio-9p-device.c b

Re: [PATCH 2/2] audio/jackaudio: Avoid dynamic stack allocation in qjack_process()

2023-08-22 Thread Christian Schoenebeck
it might make sense to limit that option to 16 channels as well. I mean anything beyond 16th channel was a dead channel anyway, right? Probably a separate battle field though: Reviewed-by: Christian Schoenebeck > audio/jackaudio.c | 16 +++- > 1 file changed, 11 insertions(+), 5

Re: [PATCH 1/2] audio/jackaudio: Avoid dynamic stack allocation in qjack_client_init

2023-08-21 Thread Christian Schoenebeck
? "out" : "in", > c->opt->client_name ? c->opt->client_name : > audio_application_name()); Unrelated, but this could be shortened by Elvis operator BTW: c->opt->client_name ?: audio_application_name() Anyway: Reviewed-by: Christian Schoenebeck Best regards, Christian Schoenebeck

Re: [PATCH, trivial 07/29] tree-wide spelling fixes in comments and some messages: hw/9pfs

2023-07-14 Thread Christian Schoenebeck
g of > - * dev_t is system dependant, we have to convert them for Linux guests if > + * dev_t is system dependent, we have to convert them for Linux guests if > * host is not running Linux. That particular one is American English vs. British English, but OK, patch LGTM: Reviewed-by: C

[PULL 0/2] 9p queue 2023-07-06

2023-07-06 Thread Christian Schoenebeck
to 'Maintained'. Christian Schoenebeck (2): 9pfs: deprecate 'proxy' backend MAINTAINERS: raise status of 9p to 'Maintained' MAINTAINERS| 11 +-- docs/about/deprecated.rst | 23

[PULL 2/2] MAINTAINERS: raise status of 9p to 'Maintained'

2023-07-06 Thread Christian Schoenebeck
Change status of 9p from 'Odd Fixes' to 'Maintained', as this better reflects current situation. I already take care of 9p patches for a while, which included new features as well. Based-on: Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz Reviewed-by: Philippe Mathieu-Daudé

[PULL 1/2] 9pfs: deprecate 'proxy' backend

2023-07-06 Thread Christian Schoenebeck
, but now it is time to officially deprecate the 'proxy' backend. Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz Message-Id: --- MAINTAINERS| 9 - docs/about/deprecated.rst | 23 +++ docs/tools/virtfs-proxy-helper.rst

Re: [PATCH] MAINTAINERS: raise status of 9p to 'Maintained'

2023-07-04 Thread Christian Schoenebeck
On Monday, July 3, 2023 4:50:42 PM CEST Greg Kurz wrote: > On Mon, 3 Jul 2023 16:34:17 +0200 > Christian Schoenebeck wrote: > > > Change status of 9p from 'Odd Fixes' to 'Maintained', as this better > > reflects current situation. I already take care of 9p patches for a >

[PATCH] MAINTAINERS: raise status of 9p to 'Maintained'

2023-07-03 Thread Christian Schoenebeck
Change status of 9p from 'Odd Fixes' to 'Maintained', as this better reflects current situation. I already take care of 9p patches for a while, which included new features as well. Based-on: Signed-off-by: Christian Schoenebeck --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion

Re: [PATCH v5] 9pfs: deprecate 'proxy' backend

2023-06-29 Thread Christian Schoenebeck
On Monday, June 26, 2023 1:49:06 PM CEST Christian Schoenebeck wrote: > As recent CVE-2023-2861 (fixed by f6b0de53fb) once again showed, the 9p > 'proxy' fs driver is in bad shape. Using the 'proxy' backend was already > discouraged for safety reasons before and we recommended to use the

[PATCH v5] 9pfs: deprecate 'proxy' backend

2023-06-26 Thread Christian Schoenebeck
, but now it is time to officially deprecate the 'proxy' backend. Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz --- v4 -> v5: - Eliminate repititve use of word 'instead' in text. - Commit log: mention exact commit which fixed CVE-2023-2861. MAINTAIN

Re: [PATCH v4] 9pfs: deprecate 'proxy' backend

2023-06-26 Thread Christian Schoenebeck
On Monday, June 26, 2023 9:27:35 AM CEST Greg Kurz wrote: > On Fri, 23 Jun 2023 14:41:15 +0200 > Christian Schoenebeck wrote: > > > As recent CVE-2023-2861 once again showed, the 9p 'proxy' fs driver is in > > bad shape. Using the 'proxy' backend was already discouraged f

[PATCH v4] 9pfs: deprecate 'proxy' backend

2023-06-23 Thread Christian Schoenebeck
to officially deprecate the 'proxy' backend. Signed-off-by: Christian Schoenebeck --- v3 -> v4: - MAINTAINERS: also move virtfs-proxy-helper.rst to 'obsolete' section - deprecated.rst: suggest virtiofsd as alternative. - deprecated.rst: mention a considerable future reimplementation of 'pr

Re: [PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-21 Thread Christian Schoenebeck
of 'proxy' or virtiofs" And that's probably still ambiguous, becauses that might suggest people that they could simply s/-virtfs proxy/-virtfs virtiofs/. Low care rate on my end though. Best regards, Christian Schoenebeck

Re: [PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-21 Thread Christian Schoenebeck
On Wednesday, June 21, 2023 3:46:39 PM CEST Daniel P. Berrangé wrote: > On Sat, Jun 10, 2023 at 03:39:44PM +0200, Christian Schoenebeck wrote: > > +``-fsdev proxy`` and ``-virtfs proxy`` (since 8.1) > > +^^ > > + > > +

Re: [PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-21 Thread Christian Schoenebeck
On Wednesday, June 21, 2023 3:41:36 PM CEST Greg Kurz wrote: > On Wed, 21 Jun 2023 15:32:39 +0200 > Christian Schoenebeck wrote: > > > On Thursday, June 15, 2023 11:35:05 AM CEST Christian Schoenebeck wrote: > > > On Saturday, June 10, 2023 3:39:44 PM CEST Chr

Re: [PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-21 Thread Christian Schoenebeck
On Thursday, June 15, 2023 11:35:05 AM CEST Christian Schoenebeck wrote: > On Saturday, June 10, 2023 3:39:44 PM CEST Christian Schoenebeck wrote: > > As recent CVE-2023-2861 once again showed, the 9p 'proxy' fs driver is in > > bad shape. Using the 'proxy' backend was alre

Re: [PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-15 Thread Christian Schoenebeck
On Saturday, June 10, 2023 3:39:44 PM CEST Christian Schoenebeck wrote: > As recent CVE-2023-2861 once again showed, the 9p 'proxy' fs driver is in > bad shape. Using the 'proxy' backend was already discouraged for safety > reasons before and we recommended to use the 'local' backen

[PATCH v3] 9pfs: deprecate 'proxy' backend

2023-06-10 Thread Christian Schoenebeck
-by: Christian Schoenebeck --- v2 -> v3: * Fix copy wasted typo (-> 'backend'). MAINTAINERS| 7 +++ docs/about/deprecated.rst | 17 + docs/tools/virtfs-proxy-helper.rst | 3 +++ fsdev/qemu-fsdev.c | 5 + fsdev/virtfs

Re: [PATCH v2] 9pfs: deprecate 'proxy' backend

2023-06-10 Thread Christian Schoenebeck
On Saturday, June 10, 2023 1:07:27 PM CEST Christian Schoenebeck wrote: > As recent CVE-2023-2861 once again showed, the 9p 'proxy' fs driver is in > bad shape. Using the 'proxy' backend was already discouraged for safety > reasons before and we recommended to use the 'local' backen

[PATCH v2] 9pfs: deprecate 'proxy' backend

2023-06-10 Thread Christian Schoenebeck
-by: Christian Schoenebeck --- v1 -> v2: * Add deprecation notice also to virtfs-proxy-helper.rst, qemu-options.hx and to the 'proxy' source files (virtfs-proxy-helper.c, 9p-proxy.c, 9p-proxy.h). MAINTAINERS| 7 +++ docs/about/deprecated.rst |

[PATCH] 9pfs: deprecate 'proxy' backend

2023-06-09 Thread Christian Schoenebeck
-by: Christian Schoenebeck --- MAINTAINERS | 7 +++ docs/about/deprecated.rst | 17 + fsdev/qemu-fsdev.c| 5 + meson.build | 2 +- softmmu/vl.c | 5 + 5 files changed, 35 insertions(+), 1 deletion(-) diff --git

[PULL 0/1] 9p security fix 2023-06-08

2023-06-08 Thread Christian Schoenebeck
. Christian Schoenebeck (1): 9pfs: prevent opening special files (CVE-2023-2861) fsdev/virtfs-proxy-helper.c | 27 +-- hw/9pfs/9p-util.h | 39 +++ 2 files changed, 64 insertions(+), 2 deletions(-)

[PULL 1/1] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-08 Thread Christian Schoenebeck
on this. Fixes: CVE-2023-2861 Reported-by: Yanwu Shen Reported-by: Jietao Xiao Reported-by: Jinku Li Reported-by: Wenbo Shen Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz Reviewed-by: Michael Tokarev Message-Id: --- fsdev/virtfs-proxy-helper.c | 27 +++-- hw

[PATCH v4] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-07 Thread Christian Schoenebeck
on this. Fixes: CVE-2023-2861 Reported-by: Yanwu Shen Reported-by: Jietao Xiao Reported-by: Jinku Li Reported-by: Wenbo Shen Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz --- v3 -> v4: - Rename function check_is_regular_file_or_dir() -> close_if_special_file() and add de

[PATCH v3] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-07 Thread Christian Schoenebeck
on this. Fixes: CVE-2023-2861 Reported-by: Yanwu Shen Reported-by: Jietao Xiao Reported-by: Jinku Li Reported-by: Wenbo Shen Signed-off-by: Christian Schoenebeck --- v2 -> v3: - Drop O_CREAT check and its comment. - Eliminate code duplication. fsdev/virtfs-proxy-helper.c |

Re: [PATCH v2] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-07 Thread Christian Schoenebeck
On Wednesday, June 7, 2023 1:02:17 PM CEST Christian Schoenebeck wrote: > On Tuesday, June 6, 2023 6:00:28 PM CEST Greg Kurz wrote: > > Hi Christian, > > > > On Tue, 06 Jun 2023 15:57:50 +0200 > > Christian Schoenebeck wrote: > > > > > The 9p protoc

Re: [PATCH v2] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-07 Thread Christian Schoenebeck
On Tuesday, June 6, 2023 8:48:49 PM CEST Michael Tokarev wrote: > 06.06.2023 16:57, Christian Schoenebeck wrote: [...] > > +/* CVE-2023-2861: Prohibit opening any special file directly on host > > + * (especially device files), as a compromised client could potentially &

Re: [PATCH v2] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-07 Thread Christian Schoenebeck
On Tuesday, June 6, 2023 6:00:28 PM CEST Greg Kurz wrote: > Hi Christian, > > On Tue, 06 Jun 2023 15:57:50 +0200 > Christian Schoenebeck wrote: > > > The 9p protocol does not specifically define how server shall behave when > > client tries to open a special file

[PATCH v2] 9pfs: prevent opening special files (CVE-2023-2861)

2023-06-06 Thread Christian Schoenebeck
on this. Fixes: CVE-2023-2861 Reported-by: Yanwu Shen Reported-by: Jietao Xiao Reported-by: Jinku Li Reported-by: Wenbo Shen Signed-off-by: Christian Schoenebeck --- v1 -> v2: - Add equivalent fix for 'proxy' fs driver. - Minor adjustments on commit log. fsdev/virtfs-proxy-helper.c |

[PULL 1/4] Don't require libcap-ng for virtfs support

2023-05-16 Thread Christian Schoenebeck
merge conflict. ] Signed-off-by: Christian Schoenebeck --- meson.build | 12 +--- meson_options.txt | 2 ++ scripts/meson-buildoptions.sh | 4 3 files changed, 15 insertions(+), 3 deletions(-) diff --git a/meson.build b/meson.build index 25a4

[PULL 3/4] 9pfs/xen: Fix segfault on shutdown

2023-05-16 Thread Christian Schoenebeck
-by: Stefano Stabellini Message-Id: <20230502143722.15613-1-jandr...@gmail.com> [C.S.: - Remove redundant return in xen_9pfs_free(). - Add comment to trace-events. ] Signed-off-by: Christian Schoenebeck --- hw/9pfs/trace-events | 6 ++ hw/9pfs/xen-9p-backend.

[PULL 2/4] tests/9p: fix potential leak in v9fs_rreaddir()

2023-05-16 Thread Christian Schoenebeck
-by: Christian Schoenebeck Reviewed-by: Greg Kurz Message-Id: --- tests/qtest/libqos/virtio-9p-client.c | 5 + 1 file changed, 5 insertions(+) diff --git a/tests/qtest/libqos/virtio-9p-client.c b/tests/qtest/libqos/virtio-9p-client.c index e4a368e036..b8adc8d4b9 100644 --- a/tests/qtest/libqos

[PULL 0/4] 9p queue 2023-05-16

2023-05-16 Thread Christian Schoenebeck
. Christian Schoenebeck (2): tests/9p: fix potential leak in v9fs_rreaddir() configure: make clear that VirtFS is 9p Jason Andryuk (1): 9pfs/xen: Fix segfault on shutdown Peter Foley (1): Don't require libcap-ng for virtfs support hw

[PULL 4/4] configure: make clear that VirtFS is 9p

2023-05-16 Thread Christian Schoenebeck
Add '9P' to the summary output section of 'VirtFS' to avoid being confused with virtiofs. Based-on: <20230503130757.863824-1-pefo...@google.com> Signed-off-by: Christian Schoenebeck Reviewed-by: Greg Kurz Reviewed-by: Thomas Huth Message-Id: --- meson.build | 4 ++-- 1 file chan

Re: [PATCH] configure: make clear that VirtFS is 9p

2023-05-13 Thread Christian Schoenebeck
On Thursday, May 11, 2023 4:12:34 PM CEST Christian Schoenebeck wrote: > Add '9P' to the summary output section of 'VirtFS' to avoid being > confused with virtiofs. > > Based-on: <20230503130757.863824-1-pefo...@google.com> > Signed-off-by: Christian Schoenebeck >

Re: [PATCH] 9pfs/xen: Fix segfault on shutdown

2023-05-11 Thread Christian Schoenebeck
: https://github.com/cschoenebeck/qemu/commits/9p.next Thanks! Best regards, Christian Schoenebeck

[PATCH] configure: make clear that VirtFS is 9p

2023-05-11 Thread Christian Schoenebeck
Add '9P' to the summary output section of 'VirtFS' to avoid being confused with virtiofs. Based-on: <20230503130757.863824-1-pefo...@google.com> Signed-off-by: Christian Schoenebeck --- meson.build | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meson.build b/meson

Re: [PATCH] 9pfs/xen: Fix segfault on shutdown

2023-05-05 Thread Christian Schoenebeck
Hi Jason, as this is a Xen specific change, I would like Stefano or another Xen developer to take a look at it, just few things from my side ... On Tuesday, May 2, 2023 4:37:22 PM CEST Jason Andryuk wrote: > xen_9pfs_free can't use gnttabdev since it is already closed and NULL-ed Where exactly

Re: [PATCH] tests/9p: fix potential leak in v9fs_rreaddir()

2023-05-04 Thread Christian Schoenebeck
On Saturday, April 29, 2023 11:25:33 AM CEST Christian Schoenebeck wrote: > Free allocated directory entries in v9fs_rreaddir() if argument > `entries` was passed as NULL, to avoid a memory leak. It is > explicitly allowed by design for `entries` to be NULL. [1] > > [1] https://

Re: [PATCH v3] Don't require libcap-ng for virtfs support

2023-05-04 Thread Christian Schoenebeck
t; --- LGTM now, queued on 9p.next: https://github.com/cschoenebeck/qemu/commits/9p.next Thanks! Best regards, Christian Schoenebeck > meson.build | 12 +--- > meson_options.txt | 2 ++ > scripts/meson-buildoptions.sh | 4 > 3 files ch

Re: [PATCH] tests/9p: fix potential leak in v9fs_rreaddir()

2023-04-29 Thread Christian Schoenebeck
On Saturday, April 29, 2023 2:04:30 PM CEST Greg Kurz wrote: > Hi Christian ! Hi there, it's been a while! :) > On Sat, 29 Apr 2023 11:25:33 +0200 > Christian Schoenebeck wrote: > > > Free allocated directory entries in v9fs_rreaddir() if argument > > `entries` was

[PATCH] tests/9p: fix potential leak in v9fs_rreaddir()

2023-04-29 Thread Christian Schoenebeck
-by: Christian Schoenebeck --- tests/qtest/libqos/virtio-9p-client.c | 5 + 1 file changed, 5 insertions(+) diff --git a/tests/qtest/libqos/virtio-9p-client.c b/tests/qtest/libqos/virtio-9p-client.c index e4a368e036..b8adc8d4b9 100644 --- a/tests/qtest/libqos/virtio-9p-client.c +++ b/tests/qtest

Re: [PATCH] tests: virtio-9p-client: Rreaddir fields are all mandatory

2023-04-28 Thread Christian Schoenebeck
9fs_rreaddir. > > Reported by Coverity. That's an old defects report, right? I remember I saw something like this last year and ignored it as being purely theoretical. > Cc: Christian Schoenebeck > Signed-off-by: Paolo Bonzini > --- > tests/qtest/libqos/virtio-9p-client.c

Re: [PATCH] Don't require libcap-ng for virtfs support

2023-04-24 Thread Christian Schoenebeck
On Friday, April 21, 2023 3:53:27 PM CEST Peter Foley wrote: > It's only required for the proxy helper. > > Signed-off-by: Peter Foley > --- > meson.build | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/meson.build b/meson.build > index c44d05a13f..1d4888b1c0

Re: [PATCH 09/10] hw/9pfs: use qemu_xxhash4

2023-04-20 Thread Christian Schoenebeck
c b/hw/9pfs/9p.c > index 9621ec1341..9bf502c45f 100644 > --- a/hw/9pfs/9p.c > +++ b/hw/9pfs/9p.c > @@ -741,12 +741,12 @@ static VariLenAffix affixForIndex(uint64_t index) > /* creative abuse of tb_hash_func7, which is based on xxhash */ I guess this old comment could be dropped then.

Re: [PATCH v5 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-03-17 Thread Christian Schoenebeck
On Friday, March 17, 2023 5:36:37 AM CET Shi, Guohuai wrote: [...] > > > > > > +do { > > > > > > +full_dir_entry = get_full_path_win32(hDir, > > > > > > + dd_data.name); > > > > > > + > > > > > > +if (full_dir_entry == NULL) { > > > > > > +err = ENOMEM; > > > > > >

Re: [PATCH v5 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-03-16 Thread Christian Schoenebeck
On Wednesday, March 15, 2023 8:05:34 PM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Wednesday, March 15, 2023 00:06 > > To: Greg Kurz ; qemu-devel@nongnu.org > > Cc: Shi, Guohuai ; Meng, Bin > > &g

Re: [PATCH v5 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-03-14 Thread Christian Schoenebeck
On Monday, February 20, 2023 11:08:03 AM CET Bin Meng wrote: > From: Guohuai Shi > > This commit implements Windows specific xxxdir() APIs for safety > directory access. That comment is seriously too short for this patch. 1. You should describe the behaviour implementation that you have chosen

Re: [PATCH v7] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-03-14 Thread Christian Schoenebeck
On Monday, March 13, 2023 8:06:15 PM CET Dorinda Bassey wrote: > > > > Are you sure about sizeof(n_bytes) here? That's 4. ;-) > > > my bad! > > > > > Volker's point was that "silence" is the center of the wave range. With > > signed > > range that's zero, yes, but with unsigned range that's

Re: [PATCH] ui/cocoa: user friendly characters for release mouse

2023-03-13 Thread Christian Schoenebeck
On Monday, March 13, 2023 2:42:36 PM CET BALATON Zoltan wrote: > On Mon, 13 Mar 2023, Christian Schoenebeck wrote: > > On Monday, February 27, 2023 12:28:02 PM CET Christian Schoenebeck wrote: > >> On Tuesday, December 27, 2022 5:15:31 PM CET Christian Schoenebeck wrote:

Re: [PATCH v7] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-03-13 Thread Christian Schoenebeck
point was that "silence" is the center of the wave range. With signed range that's zero, yes, but with unsigned range that's 2^(bitdepth) / 2. So you need to memset() the correct value to generate "silence". Best regards, Christian Schoenebeck

Re: [PATCH v5 16/16] meson.build: Turn on virtfs for Windows

2023-03-13 Thread Christian Schoenebeck
On Monday, February 20, 2023 11:08:15 AM CET Bin Meng wrote: > From: Guohuai Shi > > Enable virtfs configuration option for Windows host. > > Signed-off-by: Guohuai Shi > Signed-off-by: Bin Meng > --- > > meson.build | 10 +- > fsdev/meson.build | 1 + >

Re: [PATCH] ui/cocoa: user friendly characters for release mouse

2023-03-13 Thread Christian Schoenebeck
On Monday, February 27, 2023 12:28:02 PM CET Christian Schoenebeck wrote: > On Tuesday, December 27, 2022 5:15:31 PM CET Christian Schoenebeck wrote: > > While mouse is grabbed, window title contains a hint for the user what > > keyboard keys to press to release the mouse. Make

Re: [PATCH 5/9] 9pfs: mark more coroutine_fns

2023-03-09 Thread Christian Schoenebeck
_t > offset, > +int32_t maxsize, bool dostat) You should probably fix wrapping here, as the line exceeds 80 characters. Except of that: Reviewed-by: Christian Schoenebeck > { > V9fsState *s = pdu->s; > V9fsString name; >

Re: [PATCH v5 00/16] hw/9pfs: Add 9pfs support for Windows

2023-03-07 Thread Christian Schoenebeck
On Monday, March 6, 2023 3:56:49 PM CET Bin Meng wrote: > On Mon, Mar 6, 2023 at 10:15 PM Christian Schoenebeck > wrote: > > > > On Monday, February 20, 2023 11:07:59 AM CET Bin Meng wrote: > > > At present there is no Windows support for 9p file system. > > &

Re: [PATCH v5 00/16] hw/9pfs: Add 9pfs support for Windows

2023-03-06 Thread Christian Schoenebeck
plementation I didn't have the chance to look at this v5 yet. In general it would help for review to point out in the cover letter which patch(es) have changed, what decisions you have made and why. In this case I guess that's patch 4. Best regards, Christian Schoenebeck > Bin Meng (2): >

Re: [PATCH] ui/cocoa: user friendly characters for release mouse

2023-02-27 Thread Christian Schoenebeck
On Tuesday, December 27, 2022 5:15:31 PM CET Christian Schoenebeck wrote: > While mouse is grabbed, window title contains a hint for the user what > keyboard keys to press to release the mouse. Make that hint text a bit > more user friendly for a Mac user: > > - Replace

Re: [PATCH v4] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-02-22 Thread Christian Schoenebeck
llbacks to be called. > Built a connection to the Pipewire sound system server in the > qpw_audio_init() method. > > Signed-off-by: Dorinda Bassey Acked-by: Christian Schoenebeck > --- > fix typo > raise version dependency > > audio/audio.c |

Re: [PATCH v3] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-02-17 Thread Christian Schoenebeck
On Friday, February 17, 2023 8:11:43 AM CET Dorinda Bassey wrote: > This commit adds a new audiodev backend to allow QEMU to use Pipewire as > both an audio sink and source. This backend is available on most systems > > Add Pipewire entry points for QEMU Pipewire audio backend > Add wrappers for

Re: [PATCH] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-02-16 Thread Christian Schoenebeck
On Wednesday, February 15, 2023 4:59:47 PM CET Daniel P. Berrangé wrote: > On Wed, Feb 15, 2023 at 05:18:50PM +0400, Marc-André Lureau wrote: > > Hi > > > > On Wed, Feb 15, 2023 at 12:51 PM Dorinda Bassey wrote: > > > > > > This commit adds a new audiodev backend to allow QEMU to use Pipewire as

Re: [PATCH v2] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-02-16 Thread Christian Schoenebeck
On Thursday, February 16, 2023 9:25:44 AM CET Dorinda Bassey wrote: > This commit adds a new audiodev backend to allow QEMU to use Pipewire as both > an audio sink and source. > Please wrap commit log. > Signed-off-by: Dorinda Bassey > --- > v2: > * Shorten commit message > * fix copyright

Re: [PATCH] audio/pwaudio.c: Add Pipewire audio backend for QEMU

2023-02-15 Thread Christian Schoenebeck
JACK, another app that only supports PulseAudio, another that only supports ALSA and so on. So it tries to provide a universal plug on a system for all. Best regards, Christian Schoenebeck

Re: [PATCH v4 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-02-07 Thread Christian Schoenebeck
On Monday, February 6, 2023 6:37:16 AM CET Shi, Guohuai wrote: [...] > > I know, it's an n-square performance issue and what I already wrote in the > > summary of the linked original suggestion [1] in v3 before, quote: > > > > + Relatively straight-forward to implement. > > > > + No (major)

Re: [PATCH v4 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-02-03 Thread Christian Schoenebeck
On Friday, February 3, 2023 5:30:35 PM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Friday, February 3, 2023 22:41 > > To: Greg Kurz ; qemu-devel@nongnu.org > > Cc: Meng, Bin ; Marc-André Lureau &g

Re: [PATCH v4 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-02-03 Thread Christian Schoenebeck
On Friday, February 3, 2023 2:34:13 PM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Friday, February 3, 2023 20:25 > > To: Greg Kurz ; qemu-devel@nongnu.org > > Cc: Shi, Guohuai ; Meng, Bin > > ; Mar

Re: [PATCH v4 04/16] hw/9pfs: Implement Windows specific xxxdir() APIs

2023-02-03 Thread Christian Schoenebeck
On Monday, January 30, 2023 10:51:50 AM CET Bin Meng wrote: > From: Guohuai Shi > > This commit implements Windows specific xxxdir() APIs for safety > directory access. > This issue deserves a link to either the previous discussion Link:

Re: [PATCH v4 17/19] Don't include headers already included by qemu/osdep.h

2023-01-19 Thread Christian Schoenebeck
On Thursday, January 19, 2023 7:59:57 AM CET Markus Armbruster wrote: > This commit was created with scripts/clean-includes. > > Signed-off-by: Markus Armbruster > --- For 9p changes: Acked-by: Christian Schoenebeck > backends/tpm/tpm_ioctl.h | 2 -- >

Re: [PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h

2023-01-19 Thread Christian Schoenebeck
On Thursday, January 19, 2023 11:37:00 AM CET Markus Armbruster wrote: > Christian Schoenebeck writes: > > > On Thursday, January 19, 2023 7:59:58 AM CET Markus Armbruster wrote: > >> Signed-off-by: Markus Armbruster > >> --- > >> hw/9pfs/9p.c |

Re: [PATCH v4 18/19] 9p: Drop superfluous include of linux/limits.h

2023-01-19 Thread Christian Schoenebeck
se version come from? I don't see it anywhere in history. Last relevant change in context was a136d17590a. Best regards, Christian Schoenebeck

Re: [PATCH 10/11] alsaaudio: change default playback settings

2022-12-30 Thread Christian Schoenebeck
On Friday, December 30, 2022 10:01:47 AM CET Volker Rümelin wrote: > Am 28.12.22 um 14:52 schrieb Christian Schoenebeck: > > On Monday, December 26, 2022 4:08:37 PM CET Volker Rümelin wrote: > >> Am 21.12.22 um 12:03 schrieb Christian Schoenebeck: > >>> On Sunday, De

Re: [PATCH] ui/cocoa: user friendly characters for release mouse

2022-12-30 Thread Christian Schoenebeck
On Thursday, December 29, 2022 1:31:09 PM CET Philippe Mathieu-Daudé wrote: > On 27/12/22 17:15, Christian Schoenebeck wrote: > > While mouse is grabbed, window title contains a hint for the user what > > keyboard keys to press to release the mouse. Make that hint text a bit > &g

Re: [PATCH v3 07/17] hw/9pfs: Support getting current directory offset for Windows

2022-12-30 Thread Christian Schoenebeck
On Thursday, December 29, 2022 7:03:54 AM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Wednesday, December 28, 2022 19:51 > > To: Greg Kurz ; qemu-devel@nongnu.org > > Cc: Meng, Bin ; Shi, Guohuai > >

Re: [PATCH 10/11] alsaaudio: change default playback settings

2022-12-28 Thread Christian Schoenebeck
On Monday, December 26, 2022 4:08:37 PM CET Volker Rümelin wrote: > Am 21.12.22 um 12:03 schrieb Christian Schoenebeck: > > On Sunday, December 18, 2022 6:15:38 PM CET Volker Rümelin wrote: > >> The currently used default playback settings in the ALSA audio > >> ba

Re: [PATCH v3 07/17] hw/9pfs: Support getting current directory offset for Windows

2022-12-28 Thread Christian Schoenebeck
On Wednesday, December 21, 2022 7:02:43 PM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Wednesday, December 21, 2022 22:48 > > To: Greg Kurz ; qemu-devel@nongnu.org > > Cc: Shi, Guohuai ; Meng, Bin > >

[PATCH] ui/cocoa: user friendly characters for release mouse

2022-12-27 Thread Christian Schoenebeck
dditional space at start and end of key stroke set, to visually separate the key strokes from the rest of the text. Signed-off-by: Christian Schoenebeck --- ui/cocoa.m | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/ui/cocoa.m b/ui/cocoa.m index e915c344a8..289a2b193e 10064

Re: [PATCH] hw/audio/c97: fix abort in audio_calloc()

2022-12-25 Thread Christian Schoenebeck
\n", val); > +open_voice(s, PO_INDEX, val); > +} else { > +dolog("Attempt to set front DAC rate to %d, but valid is" > + "8-48kHZ\n", val); > +} Missing space between "i

[PULL 4/5] hw/9pfs: Drop unnecessary *xattr wrapper API declarations

2022-12-23 Thread Christian Schoenebeck
From: Bin Meng These are not used anywhere in the source tree. Drop them. Signed-off-by: Bin Meng Reviewed-by: Greg Kurz Message-Id: <20221219102022.2167736-3-bin.m...@windriver.com> Signed-off-by: Christian Schoenebeck --- hw/9pfs/9p-util.h | 11 --- 1 file changed, 11 del

[PULL 5/5] hw/9pfs: Replace the direct call to xxxat() APIs with a wrapper

2022-12-23 Thread Christian Schoenebeck
From: Bin Meng xxxat() APIs are only available on POSIX platforms. For future extension to Windows, let's replace the direct call to xxxat() APIs with a wrapper. Signed-off-by: Bin Meng Message-Id: <20221219102022.2167736-4-bin.m...@windriver.com> Signed-off-by: Christian Schoenebeck -

[PULL 3/5] qemu/xattr.h: Exclude for Windows

2022-12-23 Thread Christian Schoenebeck
From: Bin Meng Windows does not have . Signed-off-by: Bin Meng Message-Id: <20221219102022.2167736-2-bin.m...@windriver.com> Signed-off-by: Christian Schoenebeck --- include/qemu/xattr.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/qemu/xattr.h b/i

[PULL 0/5] 9p queue 2022-12-23

2022-12-23 Thread Christian Schoenebeck
with a wrapper Christian Schoenebeck (1): MAINTAINERS: Add 9p test client to section "virtio-9p" Greg Kurz (1): 9pfs: Fix some return statements in the synth backend MAINTAINERS | 1 + hw/9pfs/9p-local.c | 32 hw/9pfs/9p-synt

[PULL 1/5] 9pfs: Fix some return statements in the synth backend

2022-12-23 Thread Christian Schoenebeck
ed-off-by: Christian Schoenebeck --- hw/9pfs/9p-synth.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/hw/9pfs/9p-synth.c b/hw/9pfs/9p-synth.c index 38d787f494..f62c40b639 100644 --- a/hw/9pfs/9p-synth.c +++ b/hw/9pfs/9p-synth.c @@ -75,10 +75,10 @@ int qemu_v9fs_synth

[PULL 2/5] MAINTAINERS: Add 9p test client to section "virtio-9p"

2022-12-23 Thread Christian Schoenebeck
The 9p test cases use a dedicated, lite-weight 9p client implementation (using virtio transport) under tests/qtest/libqos/ to communicate with QEMU's 9p server. It's already there for a long time. Let's officially assign it to 9p maintainers. Signed-off-by: Christian Schoenebeck Reviewed

Re: [PATCH v3 07/17] hw/9pfs: Support getting current directory offset for Windows

2022-12-21 Thread Christian Schoenebeck
On Monday, December 19, 2022 11:20:11 AM CET Bin Meng wrote: > From: Guohuai Shi > > On Windows 'struct dirent' does not have current directory offset. > Update qemu_dirent_off() to support Windows. > > While we are here, add a build time check to error out if a new > host does not implement

Re: [PATCH v3 00/17] hw/9pfs: Add 9pfs support for Windows

2022-12-21 Thread Christian Schoenebeck
a Queued patches 1..3 for now on 9p.next: https://github.com/cschoenebeck/qemu/commits/9p.next Thanks! Best regards, Christian Schoenebeck

Re: [PATCH 10/11] alsaaudio: change default playback settings

2022-12-21 Thread Christian Schoenebeck
On Sunday, December 18, 2022 6:15:38 PM CET Volker Rümelin wrote: > The currently used default playback settings in the ALSA audio > backend are a bit unfortunate. With a few emulated audio devices, > audio playback does not work properly. Here is a short part of > the debug log while audio is

  1   2   3   4   5   6   7   8   9   10   >