[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] Re: xenpm: provide core/package cstate residencies



On 13/07/2010 08:31, "Jan Beulich" <JBeulich@xxxxxxxxxx> wrote:

>>>> On 12.07.10 at 19:34, Keir Fraser <keir.fraser@xxxxxxxxxxxxx> wrote:
>> I applied the hypervisor component of this patch. I leave it to Ian Jackson
>> to deal with the tools part. In future please split patches that touch both
>> hypervisor and tools into a patch series in which each component patch
>> touches only one or the other.
> 
> Hmm, is that really a good mechanism, especially when a change
> modifies the hypervisor <-> tools interface in an incompatible way?
> The resulting inconsistency, besides being bad by itself, would likely
> make bisecting more difficult.

In this case the change isn't an API, or even an ABI, breakage. A consistent
build of hypervisor and tools will work, and continue to work, across the
hypervisor-only changeset.

I think there will rarely be interface-breaking patches. If there are we
might have to consider them as one unit. We'll see. A patch series will
usually be the correct way to go however.

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.