On Thu, Nov 02, 2023 at 02:30:28PM +0000, James Morse wrote:
Hi Russell,
On 20/10/2023 18:35, Russell King (Oracle) wrote:
So I'm having to go through all the comments and update James' patches, otherwise we're going to end up with reviewers deeming the series not worth the effort of reviewing because they've been ignored.
I have no idea what is going on with James - and I haven't been asked to take on this work, but in order to move this thing forward, someone has to step up to the plate and take action... otherwise I don't see vcpu hotplug ever making it into the kernel.
I have too much work going on with MPAM. This was posted as an RFC to try and get the Kubernetes and Kata-containers people to test it, and tell us that it works for them.
Without that, there is the risk we've solved the wrong problem.
I think what you're essentially saying is that on the kernel side, we are waiting for confirmation from the Kubernetes and Kata-containers people that this patch is acceptable to them.
What do we do if they don't respond?
Also, I've had to add back some patches that James dropped between his trees - he dropped the IA64 changes, but that architecture is not going to be removed from the mainline kernel, so its necessary to update that as well. So that's added at least one extra patch to this 35 patch series.
Fair enough - it didn't even build from rc6 to rc1 of the last cycle, and Ard's series said there was prior agreement with the few (one?) person who still cared about that thing.
It appears that ia64 has now been dropped, so that's one less architecture we have to worry about for the vCPU hotplug changes. I've already rebased my v6.6 patch set against current Linus' tip and dropped the ia64 bits from all patches. Essentially:
ACPI: Move ACPI_HOTPLUG_CPU to be disabled on arm64 * Dropped ia64 changes ia64/topology: Switch over to GENERIC_CPU_DEVICES * Dropped ACPI: Rename ACPI_HOTPLUG_CPU to include 'present' * Dropped ia64 changes ACPI: Add _OSC bits to advertise OS support for toggling CPU present/enabled * Dropped ia64 changes * Updated James' comment to remove reference to ia64
The v6.6 patches are at:
git://git.armlinux.org.uk/~rmk/linux-arm.git aarch64/hotplug-vcpu/v6.6
Other branches can be found under aarch64/hotplug-vcpu/*. Note that the kernel build bot will be building these.
I will be pushing out aarch64/hotplug-vcpu/head - which will be the latest set of patches - thus it isn't expected that pulling this will always fast-forward (it probably never will.) Also may not be tested.
Please do. If you change more than 50% of a patch, feel free to change the author too.
Thanks. I would like your input on the patches I haven't updated from your RFC v2 changes. If you'd like a list, I can send one.