[External] Re: [PATCH v2 1/4] API: introduce memory failure

[Michal Privoznik]

On 10/14/20 4:44 AM, zhenwei pi wrote:
> I described it in '[PATCH v2 4/4] virsh: implement memory failure event'
> 
> Notice:
> The full patch set includes 4 patches:
> virsh: implement memory failure event         (current patch)
> qemu: monitor: handle memory failure event
> qemu: process: implement domainMemoryFailure
> API: introduce memory failure
> 
> To avoid build/test errors, the 4 patches should be merged/removed
> together.
> 
> 
> Suggested by Peter, separate a 'all in one' patch into 4 patches 
> (described in cover letter '[PATCH v2 0/4] support memory failure').
> I forked a repo and pushed the 4 
> patches(https://gitlab.com/pacepi/libvirt/-/tree/memory-failure-v2), CI 
> worked fine.

To add to Peter's reply: the reason we require each patch to build on 
its own is backport. You are not doing it necessarily in your series, 
but if for instance you'd move some code then:

1) it's a standalone change that should be in a separate patch
2) if a distribution wants to backport the patch (e.g. because it is a 
prerequisite for some other patch), then the code base should build 
without having to backport irrelevant patches.

And to some extent your patches can be viewed as backportable. I can 
imagine that somebody writes the code for other driver (say libxl) and a 
distro might want to backport it. It will need to backport this patch 
which adds the RPC and client facing APIs. But at the current state it 
can't do so because it won't compile.

Looking forward to v3.

Michal