Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old()

2024-07-15 Thread Zhao Liu
On Mon, Jul 15, 2024 at 02:07:00PM +0300, Michael Tokarev wrote: > Date: Mon, 15 Jul 2024 14:07:00 +0300 > From: Michael Tokarev > Subject: Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old() > > 15.07.2024 13:10, Zhao Liu wrote: > ... > > Thanks Philippe! I&#

Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old()

2024-07-15 Thread Michael Tokarev
15.07.2024 13:10, Zhao Liu wrote: ... Thanks Philippe! I'll wait patch 1's comment, after that I can post a new version with the change you mentioned. I don't think either of this is necessary. Patch 1 LGTM, and I'll drop the comment while applying. /mjt -- GPG Key transition (from rsa2048 t

Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old()

2024-07-15 Thread Zhao Liu
On Mon, Jul 15, 2024 at 11:46:54AM +0200, Philippe Mathieu-Daudé wrote: > Date: Mon, 15 Jul 2024 11:46:54 +0200 > From: Philippe Mathieu-Daudé > Subject: Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old() > > On 15/7/24 10:21, Zhao Liu wrote: > > For qemu_ope

Re: [PATCH 7/7] backends/rng-random: Get rid of qemu_open_old()

2024-07-15 Thread Philippe Mathieu-Daudé
On 15/7/24 10:21, Zhao Liu wrote: For qemu_open_old(), osdep.h said: Don't introduce new usage of this function, prefer the following qemu_open/qemu_create that take an "Error **errp". So replace qemu_open_old() with qemu_open(). And considering rng_random_opened() will lose its obvious error

[PATCH 7/7] backends/rng-random: Get rid of qemu_open_old()

2024-07-15 Thread Zhao Liu
For qemu_open_old(), osdep.h said: > Don't introduce new usage of this function, prefer the following > qemu_open/qemu_create that take an "Error **errp". So replace qemu_open_old() with qemu_open(). And considering rng_random_opened() will lose its obvious error handling case after removing erro