Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-19 Thread Akihiko Odaki
On 2025/09/19 3:23, Peter Xu wrote: On Thu, Sep 18, 2025 at 12:20:49PM -0400, Peter Xu wrote: On Thu, Sep 18, 2025 at 05:29:34PM +0200, BALATON Zoltan wrote: On Thu, 18 Sep 2025, Peter Xu wrote: On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: Based-on: ("[PATCH v2 00/14] hw/pc

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-18 Thread Peter Xu
On Wed, Sep 17, 2025 at 02:23:35PM +0100, Daniel P. Berrangé wrote: > On Wed, Sep 17, 2025 at 02:17:35PM +0100, Daniel P. Berrangé wrote: > > On Wed, Sep 17, 2025 at 09:24:04PM +0900, Akihiko Odaki wrote: > > > On 2025/09/17 20:57, Daniel P. Berrangé wrote: > > > > On Wed, Sep 17, 2025 at 07:13:25P

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-18 Thread Peter Xu
On Thu, Sep 18, 2025 at 12:20:49PM -0400, Peter Xu wrote: > On Thu, Sep 18, 2025 at 05:29:34PM +0200, BALATON Zoltan wrote: > > On Thu, 18 Sep 2025, Peter Xu wrote: > > > On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > > > > Based-on: > > > > ("[PATCH v2 00/14] hw/pci-host/raven c

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-18 Thread Peter Xu
On Thu, Sep 18, 2025 at 05:29:34PM +0200, BALATON Zoltan wrote: > On Thu, 18 Sep 2025, Peter Xu wrote: > > On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > > > Based-on: > > > ("[PATCH v2 00/14] hw/pci-host/raven clean ups") > > > > Could I ask why this is a dependency? > > It re

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-18 Thread BALATON Zoltan
On Thu, 18 Sep 2025, Peter Xu wrote: On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: Based-on: ("[PATCH v2 00/14] hw/pci-host/raven clean ups") Could I ask why this is a dependency? It removes an address_space usage from raven so this series does not have to change that and

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-18 Thread Peter Xu
On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > Based-on: > ("[PATCH v2 00/14] hw/pci-host/raven clean ups") Could I ask why this is a dependency? -- Peter Xu

[PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-17 Thread Akihiko Odaki
Based-on: ("[PATCH v2 00/14] hw/pci-host/raven clean ups") Supersedes: <20240829-memory-v1-1-ac07af2f4...@daynix.com> ("[PATCH] docs/devel: Prohibit calling object_unparent() for memory region") Children are automatically unparented so manually unparenting is unnecessary. Worse, automatic unpar

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-17 Thread Akihiko Odaki
On 2025/09/17 20:57, Daniel P. Berrangé wrote: On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: Based-on: ("[PATCH v2 00/14] hw/pci-host/raven clean ups") Supersedes: <20240829-memory-v1-1-ac07af2f4...@daynix.com> ("[PATCH] docs/devel: Prohibit calling object_unparent() for memor

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-17 Thread Daniel P . Berrangé
On Wed, Sep 17, 2025 at 02:17:35PM +0100, Daniel P. Berrangé wrote: > On Wed, Sep 17, 2025 at 09:24:04PM +0900, Akihiko Odaki wrote: > > On 2025/09/17 20:57, Daniel P. Berrangé wrote: > > > On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > > > > Based-on: > > > > ("[PATCH v2 00/14]

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-17 Thread Daniel P . Berrangé
On Wed, Sep 17, 2025 at 09:24:04PM +0900, Akihiko Odaki wrote: > On 2025/09/17 20:57, Daniel P. Berrangé wrote: > > On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > > > Based-on: > > > ("[PATCH v2 00/14] hw/pci-host/raven clean ups") > > > > > > Supersedes: <20240829-memory-v1-1-a

Re: [PATCH v3 0/7] Do not unparent in instance_finalize()

2025-09-17 Thread Daniel P . Berrangé
On Wed, Sep 17, 2025 at 07:13:25PM +0900, Akihiko Odaki wrote: > Based-on: > ("[PATCH v2 00/14] hw/pci-host/raven clean ups") > > Supersedes: <20240829-memory-v1-1-ac07af2f4...@daynix.com> > ("[PATCH] docs/devel: Prohibit calling object_unparent() for memory region") > > Children are automatical