Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Peter Maydell
On Fri, 8 Sept 2023 at 13:22, Kevin Wolf wrote: > > Am 08.09.2023 um 11:53 hat Peter Maydell geschrieben: > > On Fri, 8 Sept 2023 at 10:26, Kevin Wolf wrote: > > > I have a patch to use QList (i.e. JSON lists) that seems to work for the > > > rocker case. Now I need to find and update all of thos

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Kevin Wolf
Am 08.09.2023 um 11:53 hat Peter Maydell geschrieben: > On Fri, 8 Sept 2023 at 10:26, Kevin Wolf wrote: > > > > Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > > > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > > > > > Kevin Wolf writes: > > > > > > > > > Am 04.09.2023 um

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Daniel P . Berrangé
On Fri, Sep 08, 2023 at 02:16:35PM +0200, Kevin Wolf wrote: > Am 08.09.2023 um 11:27 hat Daniel P. Berrangé geschrieben: > > On Fri, Sep 08, 2023 at 11:25:54AM +0200, Kevin Wolf wrote: > > > Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > > > > On Thu, 7 Sept 2023 at 10:33, Markus Armbruste

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Kevin Wolf
Am 08.09.2023 um 11:27 hat Daniel P. Berrangé geschrieben: > On Fri, Sep 08, 2023 at 11:25:54AM +0200, Kevin Wolf wrote: > > Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > > > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > > > > > Kevin Wolf writes: > > > > > > > > > Am 0

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Peter Maydell
On Fri, 8 Sept 2023 at 10:26, Kevin Wolf wrote: > > Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > > > Kevin Wolf writes: > > > > > > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > > > >> By the time of the

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Daniel P . Berrangé
On Fri, Sep 08, 2023 at 11:25:54AM +0200, Kevin Wolf wrote: > Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > > > Kevin Wolf writes: > > > > > > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > > > >> By the ti

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-08 Thread Kevin Wolf
Am 07.09.2023 um 11:35 hat Peter Maydell geschrieben: > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > Kevin Wolf writes: > > > > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > > >> By the time of the 8.2.0 release, it will have been 2 years and 6 > > >> releases s

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Markus Armbruster
Kevin Wolf writes: > Am 07.09.2023 um 11:33 hat Markus Armbruster geschrieben: >> Kevin Wolf writes: >> > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: >> >> I still think for user creatable devices we'd be better off just >> >> mandating the use of JSON syntax for -device and thus

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Kevin Wolf
Am 07.09.2023 um 11:33 hat Markus Armbruster geschrieben: > Kevin Wolf writes: > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > >> I still think for user creatable devices we'd be better off just > >> mandating the use of JSON syntax for -device and thus leveraging > >> the native

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Daniel P . Berrangé
On Thu, Sep 07, 2023 at 10:35:22AM +0100, Peter Maydell wrote: > On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > > > Kevin Wolf writes: > > > > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > > >> By the time of the 8.2.0 release, it will have been 2 years and 6 > > >> r

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Markus Armbruster
Daniel P. Berrangé writes: > By the time of the 8.2.0 release, it will have been 2 years and 6 > releases since we accidentally broke setting of array properties > for user creatable devices: > > https://gitlab.com/qemu-project/qemu/-/issues/1090 > > Some context: > > * Initial identification /

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Peter Maydell
On Thu, 7 Sept 2023 at 10:33, Markus Armbruster wrote: > > Kevin Wolf writes: > > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > >> By the time of the 8.2.0 release, it will have been 2 years and 6 > >> releases since we accidentally broke setting of array properties > >> for user

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-07 Thread Markus Armbruster
Kevin Wolf writes: > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: >> By the time of the 8.2.0 release, it will have been 2 years and 6 >> releases since we accidentally broke setting of array properties >> for user creatable devices: >> >> https://gitlab.com/qemu-project/qemu/-/i

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-05 Thread Peter Maydell
On Tue, 5 Sept 2023 at 09:59, Kevin Wolf wrote: > > Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > > By the time of the 8.2.0 release, it will have been 2 years and 6 > > releases since we accidentally broke setting of array properties > > for user creatable devices: > > > > https:

Re: [PATCH 0/1] qom: fix setting of qdev array properties

2023-09-05 Thread Kevin Wolf
Am 04.09.2023 um 18:25 hat Daniel P. Berrangé geschrieben: > By the time of the 8.2.0 release, it will have been 2 years and 6 > releases since we accidentally broke setting of array properties > for user creatable devices: > > https://gitlab.com/qemu-project/qemu/-/issues/1090 Oh, nice! Well,

[PATCH 0/1] qom: fix setting of qdev array properties

2023-09-04 Thread Daniel P . Berrangé
By the time of the 8.2.0 release, it will have been 2 years and 6 releases since we accidentally broke setting of array properties for user creatable devices: https://gitlab.com/qemu-project/qemu/-/issues/1090 Some context: * Initial identification / report on the mailing list https://list