All of lore.kernel.org
 help / color / mirror / Atom feed
From: Salil Mehta <salil.mehta@huawei.com>
To: Nicholas Piggin <npiggin@gmail.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>
Cc: "maz@kernel.org" <maz@kernel.org>,
	"jean-philippe@linaro.org" <jean-philippe@linaro.org>,
	Jonathan Cameron <jonathan.cameron@huawei.com>,
	"lpieralisi@kernel.org" <lpieralisi@kernel.org>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"richard.henderson@linaro.org" <richard.henderson@linaro.org>,
	"imammedo@redhat.com" <imammedo@redhat.com>,
	"andrew.jones@linux.dev" <andrew.jones@linux.dev>,
	"david@redhat.com" <david@redhat.com>,
	"philmd@linaro.org" <philmd@linaro.org>,
	"eric.auger@redhat.com" <eric.auger@redhat.com>,
	"oliver.upton@linux.dev" <oliver.upton@linux.dev>,
	"pbonzini@redhat.com" <pbonzini@redhat.com>,
	"mst@redhat.com" <mst@redhat.com>,
	"will@kernel.org" <will@kernel.org>,
	"gshan@redhat.com" <gshan@redhat.com>,
	"rafael@kernel.org" <rafael@kernel.org>,
	"alex.bennee@linaro.org" <alex.bennee@linaro.org>,
	"linux@armlinux.org.uk" <linux@armlinux.org.uk>,
	"darren@os.amperecomputing.com" <darren@os.amperecomputing.com>,
	"ilkka@os.amperecomputing.com" <ilkka@os.amperecomputing.com>,
	"vishnu@os.amperecomputing.com" <vishnu@os.amperecomputing.com>,
	"karl.heubaum@oracle.com" <karl.heubaum@oracle.com>,
	"miguel.luis@oracle.com" <miguel.luis@oracle.com>,
	"salil.mehta@opnsrc.net" <salil.mehta@opnsrc.net>,
	zhukeqian <zhukeqian1@huawei.com>,
	"wangxiongfeng (C)" <wangxiongfeng2@huawei.com>,
	"wangyanan (Y)" <wangyanan55@huawei.com>,
	"jiakernel2@gmail.com" <jiakernel2@gmail.com>,
	"maobibo@loongson.cn" <maobibo@loongson.cn>,
	"lixianglai@loongson.cn" <lixianglai@loongson.cn>,
	"harshpb@linux.ibm.com" <harshpb@linux.ibm.com>,
	Linuxarm <linuxarm@huawei.com>,
	Shaoqin Huang <shahuang@redhat.com>
Subject: RE: [PATCH V9 6/8] physmem: Add helper function to destroy CPU AddressSpace
Date: Mon, 20 May 2024 10:55:13 +0000	[thread overview]
Message-ID: <46d1577dc27a4012bb90d9871d03fabe@huawei.com> (raw)
In-Reply-To: <D1EBOZTRN8SZ.Q9F4CFDZ8VYW@gmail.com>

>  From: Nicholas Piggin <npiggin@gmail.com>
>  Sent: Monday, May 20, 2024 9:19 AM
>  
>  On Mon May 20, 2024 at 7:06 AM AEST, Salil Mehta wrote:
>  > Virtual CPU Hot-unplug leads to unrealization of a CPU object. This
>  > also involves destruction of the CPU AddressSpace. Add common function
>  > to help destroy the CPU AddressSpace.
>  
>  Patches 6,7 seem okay to me. I would like to see a series where they get
>  used. Would it be possible to include at least a TCG user in your series? That
>  would make it easier to review and test.


If you wish to play then you can use below repository which is V8 version of
architecture agnostic patch-set plus architecture dependent patch-set
stacked up together. This effectively RFC V2 rearranged with clear separation
of arch agnostic and specific parts and some review comments incorporated.

https://github.com/salil-mehta/qemu/commits/virt-cpuhp-armv8/rfc-v3-rc1/


It has TCG but it has some issues and hence we have disabled it for now. We are
working to fix it.


Thanks
Salil.

>  
>  Thanks,
>  Nick

WARNING: multiple messages have this Message-ID (diff)
From: Salil Mehta via <qemu-devel@nongnu.org>
To: Nicholas Piggin <npiggin@gmail.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"qemu-arm@nongnu.org" <qemu-arm@nongnu.org>
Cc: "maz@kernel.org" <maz@kernel.org>,
	"jean-philippe@linaro.org" <jean-philippe@linaro.org>,
	Jonathan Cameron <jonathan.cameron@huawei.com>,
	"lpieralisi@kernel.org" <lpieralisi@kernel.org>,
	"peter.maydell@linaro.org" <peter.maydell@linaro.org>,
	"richard.henderson@linaro.org" <richard.henderson@linaro.org>,
	"imammedo@redhat.com" <imammedo@redhat.com>,
	"andrew.jones@linux.dev" <andrew.jones@linux.dev>,
	"david@redhat.com" <david@redhat.com>,
	"philmd@linaro.org" <philmd@linaro.org>,
	"eric.auger@redhat.com" <eric.auger@redhat.com>,
	"oliver.upton@linux.dev" <oliver.upton@linux.dev>,
	"pbonzini@redhat.com" <pbonzini@redhat.com>,
	"mst@redhat.com" <mst@redhat.com>,
	"will@kernel.org" <will@kernel.org>,
	"gshan@redhat.com" <gshan@redhat.com>,
	"rafael@kernel.org" <rafael@kernel.org>,
	"alex.bennee@linaro.org" <alex.bennee@linaro.org>,
	"linux@armlinux.org.uk" <linux@armlinux.org.uk>,
	"darren@os.amperecomputing.com" <darren@os.amperecomputing.com>,
	"ilkka@os.amperecomputing.com" <ilkka@os.amperecomputing.com>,
	"vishnu@os.amperecomputing.com" <vishnu@os.amperecomputing.com>,
	"karl.heubaum@oracle.com" <karl.heubaum@oracle.com>,
	"miguel.luis@oracle.com" <miguel.luis@oracle.com>,
	"salil.mehta@opnsrc.net" <salil.mehta@opnsrc.net>,
	zhukeqian <zhukeqian1@huawei.com>,
	"wangxiongfeng (C)" <wangxiongfeng2@huawei.com>,
	"wangyanan (Y)" <wangyanan55@huawei.com>,
	"jiakernel2@gmail.com" <jiakernel2@gmail.com>,
	"maobibo@loongson.cn" <maobibo@loongson.cn>,
	"lixianglai@loongson.cn" <lixianglai@loongson.cn>,
	"harshpb@linux.ibm.com" <harshpb@linux.ibm.com>,
	Linuxarm <linuxarm@huawei.com>,
	Shaoqin Huang <shahuang@redhat.com>
Subject: RE: [PATCH V9 6/8] physmem: Add helper function to destroy CPU AddressSpace
Date: Mon, 20 May 2024 10:55:13 +0000	[thread overview]
Message-ID: <46d1577dc27a4012bb90d9871d03fabe@huawei.com> (raw)
In-Reply-To: <D1EBOZTRN8SZ.Q9F4CFDZ8VYW@gmail.com>

>  From: Nicholas Piggin <npiggin@gmail.com>
>  Sent: Monday, May 20, 2024 9:19 AM
>  
>  On Mon May 20, 2024 at 7:06 AM AEST, Salil Mehta wrote:
>  > Virtual CPU Hot-unplug leads to unrealization of a CPU object. This
>  > also involves destruction of the CPU AddressSpace. Add common function
>  > to help destroy the CPU AddressSpace.
>  
>  Patches 6,7 seem okay to me. I would like to see a series where they get
>  used. Would it be possible to include at least a TCG user in your series? That
>  would make it easier to review and test.


If you wish to play then you can use below repository which is V8 version of
architecture agnostic patch-set plus architecture dependent patch-set
stacked up together. This effectively RFC V2 rearranged with clear separation
of arch agnostic and specific parts and some review comments incorporated.

https://github.com/salil-mehta/qemu/commits/virt-cpuhp-armv8/rfc-v3-rc1/


It has TCG but it has some issues and hence we have disabled it for now. We are
working to fix it.


Thanks
Salil.

>  
>  Thanks,
>  Nick

  reply	other threads:[~2024-05-20 10:55 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-19 21:06 [PATCH V9 0/8] Add architecture agnostic code to support vCPU Hotplug Salil Mehta
2024-05-19 21:06 ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 1/8] accel/kvm: Extract common KVM vCPU {creation,parking} code Salil Mehta
2024-05-19 21:06   ` [PATCH V9 1/8] accel/kvm: Extract common KVM vCPU {creation, parking} code Salil Mehta via
2024-05-20  8:04   ` [PATCH V9 1/8] accel/kvm: Extract common KVM vCPU {creation,parking} code Nicholas Piggin
2024-05-20 10:48     ` Salil Mehta
2024-05-20 10:48       ` Salil Mehta via
2024-05-20 11:27     ` Salil Mehta
2024-05-20 11:27       ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 2/8] hw/acpi: Move CPU ctrl-dev MMIO region len macro to common header file Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 3/8] hw/acpi: Update ACPI GED framework to support vCPU Hotplug Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 4/8] hw/acpi: Update GED _EVT method AML with CPU scan Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 5/8] hw/acpi: Update CPUs AML with cpu-(ctrl)dev change Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 6/8] physmem: Add helper function to destroy CPU AddressSpace Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-20  8:18   ` Nicholas Piggin
2024-05-20 10:55     ` Salil Mehta [this message]
2024-05-20 10:55       ` Salil Mehta via
2024-05-22  1:28       ` Nicholas Piggin
2024-05-19 21:06 ` [PATCH V9 7/8] gdbstub: Add helper function to unregister GDB register space Salil Mehta
2024-05-19 21:06   ` Salil Mehta via
2024-05-19 21:06 ` [PATCH V9 8/8] docs/specs/acpi_hw_reduced_hotplug: Add the CPU Hotplug Event Bit Salil Mehta
2024-05-19 21:06   ` Salil Mehta via

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=46d1577dc27a4012bb90d9871d03fabe@huawei.com \
    --to=salil.mehta@huawei.com \
    --cc=alex.bennee@linaro.org \
    --cc=andrew.jones@linux.dev \
    --cc=darren@os.amperecomputing.com \
    --cc=david@redhat.com \
    --cc=eric.auger@redhat.com \
    --cc=gshan@redhat.com \
    --cc=harshpb@linux.ibm.com \
    --cc=ilkka@os.amperecomputing.com \
    --cc=imammedo@redhat.com \
    --cc=jean-philippe@linaro.org \
    --cc=jiakernel2@gmail.com \
    --cc=jonathan.cameron@huawei.com \
    --cc=karl.heubaum@oracle.com \
    --cc=linux@armlinux.org.uk \
    --cc=linuxarm@huawei.com \
    --cc=lixianglai@loongson.cn \
    --cc=lpieralisi@kernel.org \
    --cc=maobibo@loongson.cn \
    --cc=maz@kernel.org \
    --cc=miguel.luis@oracle.com \
    --cc=mst@redhat.com \
    --cc=npiggin@gmail.com \
    --cc=oliver.upton@linux.dev \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@linaro.org \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rafael@kernel.org \
    --cc=richard.henderson@linaro.org \
    --cc=salil.mehta@opnsrc.net \
    --cc=shahuang@redhat.com \
    --cc=vishnu@os.amperecomputing.com \
    --cc=wangxiongfeng2@huawei.com \
    --cc=wangyanan55@huawei.com \
    --cc=will@kernel.org \
    --cc=zhukeqian1@huawei.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.