Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-27 Thread Claudio Fontana
On 9/27/22 13:53, Kevin Wolf wrote: > Am 27.09.2022 um 11:13 hat Claudio Fontana geschrieben: >> On 9/27/22 09:54, Markus Armbruster wrote: >>> Claudio Fontana writes: >>> On 9/26/22 12:38, Kevin Wolf wrote: > Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: >> improve error

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-27 Thread Kevin Wolf
Am 27.09.2022 um 09:54 hat Markus Armbruster geschrieben: > Potential issue before this patch: I can't see bdrv_open_child() being > undone. Shouldn't we close bs->file? And what about > bdrv_open_child()'s side effect on @options? bs->file is handled by the caller, bdrv_open_driver(). options

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-27 Thread Kevin Wolf
Am 27.09.2022 um 11:13 hat Claudio Fontana geschrieben: > On 9/27/22 09:54, Markus Armbruster wrote: > > Claudio Fontana writes: > > > >> On 9/26/22 12:38, Kevin Wolf wrote: > >>> Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: > improve error handling during module load, by

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-27 Thread Claudio Fontana
On 9/27/22 09:54, Markus Armbruster wrote: > Claudio Fontana writes: > >> On 9/26/22 12:38, Kevin Wolf wrote: >>> Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: improve error handling during module load, by changing: bool module_load(const char *prefix, const char

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-27 Thread Markus Armbruster
Claudio Fontana writes: > On 9/26/22 12:38, Kevin Wolf wrote: >> Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: >>> improve error handling during module load, by changing: >>> >>> bool module_load(const char *prefix, const char *lib_name); >>> void module_load_qom(const char *type); >>>

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-26 Thread Claudio Fontana
On 9/26/22 12:38, Kevin Wolf wrote: > Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: >> improve error handling during module load, by changing: >> >> bool module_load(const char *prefix, const char *lib_name); >> void module_load_qom(const char *type); >> >> to: >> >> int

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-26 Thread Claudio Fontana
On 9/26/22 12:38, Kevin Wolf wrote: > Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: >> improve error handling during module load, by changing: >> >> bool module_load(const char *prefix, const char *lib_name); >> void module_load_qom(const char *type); >> >> to: >> >> int

Re: [PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-26 Thread Kevin Wolf
Am 24.09.2022 um 01:21 hat Claudio Fontana geschrieben: > improve error handling during module load, by changing: > > bool module_load(const char *prefix, const char *lib_name); > void module_load_qom(const char *type); > > to: > > int module_load(const char *prefix, const char *name, Error

[PATCH v6 3/5] module: add Error arguments to module_load and module_load_qom

2022-09-23 Thread Claudio Fontana
improve error handling during module load, by changing: bool module_load(const char *prefix, const char *lib_name); void module_load_qom(const char *type); to: int module_load(const char *prefix, const char *name, Error **errp); int module_load_qom(const char *type, Error **errp); where the