RE: [PATCH V12 0/8] Add architecture agnostic code to support vCPU Hotplug

Salil Mehta via posted 8 patches 5 months ago
Only 0 patches received!
There is a newer version of this series
RE: [PATCH V12 0/8] Add architecture agnostic code to support vCPU Hotplug
Posted by Salil Mehta via 5 months ago
Hi Gavin,

>  From: Gavin Shan <gshan@redhat.com>
>  Sent: Wednesday, June 26, 2024 5:13 AM
>  To: Salil Mehta <salil.mehta@opnsrc.net>; Igor Mammedov
>  <imammedo@redhat.com>
>  
>  Hi Salil and Igor,
>  
>  On 6/26/24 9:51 AM, Salil Mehta wrote:
>  > On Wed, Jun 5, 2024 at 3:03 PM Igor Mammedov
>  <imammedo@redhat.com <mailto:imammedo@redhat.com>> wrote:
>  >     On Sun, 2 Jun 2024 18:03:05 -0400
>  >     "Michael S. Tsirkin" <mst@redhat.com <mailto:mst@redhat.com>>
>  wrote:
>  >
>  >      > On Thu, May 30, 2024 at 12:42:33AM +0100, Salil Mehta wrote:
>  >      > > Virtual CPU hotplug support is being added across various architectures[1][3].
>  >      > > This series adds various code bits common across all architectures:
>  >      > >
>  >      > > 1. vCPU creation and Parking code refactor [Patch 1]
>  >      > > 2. Update ACPI GED framework to support vCPU Hotplug [Patch 2,3]
>  >      > > 3. ACPI CPUs AML code change [Patch 4,5]
>  >      > > 4. Helper functions to support unrealization of CPU objects [Patch  6,7]
>  >      > > 5. Docs [Patch 8]
>  >      > >
>  >      > >
>  >      > > Repository:
>  >      > >
>  >      > > [*] https://github.com/salil-mehta/qemu.git   <https://github.com/salil-mehta/qemu.git> virt-cpuhp-armv8/rfc- v3.arch.agnostic.v12
>  >      > >
>  >      > > NOTE: This series is meant to work in conjunction with Architecture specific patch-set.
>  >      > > For ARM, this will work in combination of the architecture specific part based on
>  >      > > RFC V2 [1]. This architecture specific patch-set RFC V3 shall be floated soon and is
>  >      > > present at below location
>  >      > >
>  >      > > [*] https://github.com/salil-mehta/qemu/tree/virt-cpuhp-armv8/rfc-v3-rc1 <https://github.com/salil-mehta/qemu/tree/virt-cpuhp-armv8/rfc-v3-rc1>
>  >      > >
>  >      >
>  >      >
>  >      > Igor plan to take a look?
>  >
>  >     Yep, I plan to review it
>  >
>  >
>  > A gentle reminder on this.
>  >
>  
>  Since the latest revision for this series is v13, so I guess Igor needs to do the
>  final screening on v13 instead?
>  
>  v13: https://lists.nongnu.org/archive/html/qemu-arm/2024-06/msg00129.html


Yes, thanks for pointing this. 😊


>  
>  Thanks,
>  Gavin
>  

Re: [PATCH V12 0/8] Add architecture agnostic code to support vCPU Hotplug
Posted by Igor Mammedov 4 months, 3 weeks ago
On Wed, 26 Jun 2024 17:53:52 +0000
Salil Mehta <salil.mehta@huawei.com> wrote:

> Hi Gavin,
> 
> >  From: Gavin Shan <gshan@redhat.com>
> >  Sent: Wednesday, June 26, 2024 5:13 AM
> >  To: Salil Mehta <salil.mehta@opnsrc.net>; Igor Mammedov
> >  <imammedo@redhat.com>
> >  
> >  Hi Salil and Igor,
> >  
> >  On 6/26/24 9:51 AM, Salil Mehta wrote:  
> >  > On Wed, Jun 5, 2024 at 3:03 PM Igor Mammedov  
> >  <imammedo@redhat.com <mailto:imammedo@redhat.com>> wrote:  
> >  >     On Sun, 2 Jun 2024 18:03:05 -0400
> >  >     "Michael S. Tsirkin" <mst@redhat.com <mailto:mst@redhat.com>>  
> >  wrote:  
> >  >  
> >  >      > On Thu, May 30, 2024 at 12:42:33AM +0100, Salil Mehta wrote:  
> >  >      > > Virtual CPU hotplug support is being added across various architectures[1][3].
> >  >      > > This series adds various code bits common across all architectures:
> >  >      > >
> >  >      > > 1. vCPU creation and Parking code refactor [Patch 1]
> >  >      > > 2. Update ACPI GED framework to support vCPU Hotplug [Patch 2,3]
> >  >      > > 3. ACPI CPUs AML code change [Patch 4,5]
> >  >      > > 4. Helper functions to support unrealization of CPU objects [Patch  6,7]
> >  >      > > 5. Docs [Patch 8]
> >  >      > >
> >  >      > >
> >  >      > > Repository:
> >  >      > >
> >  >      > > [*] https://github.com/salil-mehta/qemu.git   <https://github.com/salil-mehta/qemu.git> virt-cpuhp-armv8/rfc- v3.arch.agnostic.v12
> >  >      > >
> >  >      > > NOTE: This series is meant to work in conjunction with Architecture specific patch-set.
> >  >      > > For ARM, this will work in combination of the architecture specific part based on
> >  >      > > RFC V2 [1]. This architecture specific patch-set RFC V3 shall be floated soon and is
> >  >      > > present at below location
> >  >      > >
> >  >      > > [*] https://github.com/salil-mehta/qemu/tree/virt-cpuhp-armv8/rfc-v3-rc1 <https://github.com/salil-mehta/qemu/tree/virt-cpuhp-armv8/rfc-v3-rc1>
> >  >      > >  
> >  >      >
> >  >      >
> >  >      > Igor plan to take a look?  
> >  >
> >  >     Yep, I plan to review it
> >  >
> >  >
> >  > A gentle reminder on this.
> >  >  
> >  
> >  Since the latest revision for this series is v13, so I guess Igor needs to do the
> >  final screening on v13 instead?
> >  
> >  v13: https://lists.nongnu.org/archive/html/qemu-arm/2024-06/msg00129.html  
> 
> 
> Yes, thanks for pointing this. 😊

I have v13 tagged,
Sorry for delay, I'll try to review it this week

> 
> 
> >  
> >  Thanks,
> >  Gavin
> >    
> 
RE: [PATCH V12 0/8] Add architecture agnostic code to support vCPU Hotplug
Posted by Salil Mehta via 4 months, 3 weeks ago
Hi Igor,

>  From: Igor Mammedov <imammedo@redhat.com>
>  Sent: Monday, July 1, 2024 9:49 AM
>  To: Salil Mehta <salil.mehta@huawei.com>
>  
>  On Wed, 26 Jun 2024 17:53:52 +0000
>  Salil Mehta <salil.mehta@huawei.com> wrote:
>  
>  > Hi Gavin,
>  >
>  > >  From: Gavin Shan <gshan@redhat.com>
>  > >  Sent: Wednesday, June 26, 2024 5:13 AM
>  > >  To: Salil Mehta <salil.mehta@opnsrc.net>; Igor Mammedov
>  > > <imammedo@redhat.com>
>  > >
>  > >  Hi Salil and Igor,
>  > >
>  > >  On 6/26/24 9:51 AM, Salil Mehta wrote:
>  > >  > On Wed, Jun 5, 2024 at 3:03 PM Igor Mammedov
>  > > <imammedo@redhat.com <mailto:imammedo@redhat.com>> wrote:
>  > >  >     On Sun, 2 Jun 2024 18:03:05 -0400
>  > >  >     "Michael S. Tsirkin" <mst@redhat.com <mailto:mst@redhat.com>> wrote:
>  > >  >
>  > >  >      > On Thu, May 30, 2024 at 12:42:33AM +0100, Salil Mehta wrote:
>  > >  >      > > Virtual CPU hotplug support is being added across various architectures[1][3].
>  > >  >      > > This series adds various code bits common across all  architectures:
>  > >  >      > >
>  > >  >      > > 1. vCPU creation and Parking code refactor [Patch 1]
>  > >  >      > > 2. Update ACPI GED framework to support vCPU Hotplug [Patch 2,3]
>  > >  >      > > 3. ACPI CPUs AML code change [Patch 4,5]
>  > >  >      > > 4. Helper functions to support unrealization of CPU objects  [Patch  6,7]
>  > >  >      > > 5. Docs [Patch 8]
>  > >  >      > >
>  > >  >      > >
>  > >  >      > > Repository:
>  > >  >      > >
>  > >  >      > > [*] https://github.com/salil-mehta/qemu.git   <https://github.com/salil-mehta/qemu.git> virt-cpuhp-armv8/rfc-v3.arch.agnostic.v12
>  > >  >      > >
>  > >  >      > > NOTE: This series is meant to work in conjunction with Architecture specific patch-set.
>  > >  >      > > For ARM, this will work in combination of the architecture specific part based on
>  > >  >      > > RFC V2 [1]. This architecture specific patch-set RFC V3 shall be floated soon and is
>  > >  >      > > present at below location
>  > >  >      > >
>  > >  >      > > [*] https://github.com/salil-mehta/qemu/tree/virt-cpuhp-armv8/rfc-v3-rc1 <https://github.com/salil-mehta/qemu/tree/virt-cpuhp-
>  armv8/rfc-v3-rc1>
>  > >  >      > >
>  > >  >      >
>  > >  >      >
>  > >  >      > Igor plan to take a look?
>  > >  >
>  > >  >     Yep, I plan to review it
>  > >  >
>  > >  >
>  > >  > A gentle reminder on this.
>  > >  >
>  > >
>  > >  Since the latest revision for this series is v13, so I guess Igor
>  > > needs to do the  final screening on v13 instead?
>  > >
>  > >  v13:
>  > > https://lists.nongnu.org/archive/html/qemu-arm/2024-06/msg00129.html
>  >
>  >
>  > Yes, thanks for pointing this. 😊
>  
>  I have v13 tagged,
>  Sorry for delay, I'll try to review it this week

Sure. No problem. 

Thanks
Salil.

>  > >  Thanks,
>  > >  Gavin