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

Re: [Xen-devel] [PATCH] x86/hvm: Unify hvm_event_pending()'s API with the !CONFIG_HVM version

>>> On 12.11.18 at 11:07, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 12/11/2018 09:57, Jan Beulich wrote:
>>>>> On 09.11.18 at 15:42, <andrew.cooper3@xxxxxxxxxx> wrote:
>>> No functional change.
>> So I understand that "unify" means converting the return type to
>> bool. You could have said so.
> Perhaps, but that misses the point of making the change.
> The patch introducing the !CONFIG_HVM version should have used
> int/non-const so the API was same, or had this patch as a prerequisite
> to update the API to our new standards, and then duplicate it for the
> !CONFIG_HVM case.

All fine, just that it would have been nice if the description here
said slightly more than just "No functional change."


Xen-devel mailing list



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