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

Re: [Xen-devel] [PATCH v2] x86/hvm: Add MSR old value

>>> On 22.12.17 at 10:09, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
> On 12/05/2017 11:07 AM, Jan Beulich wrote:
>>>>> On 04.12.17 at 16:45, <rcojocaru@xxxxxxxxxxxxxxx> wrote:
>>>>  On Fri, Oct 13, 2017 at 03:50:57PM +0300, Alexandru Isaila wrote:
>>>>> This patch adds the old value param and the onchangeonly option
>>>>> to the VM_EVENT_REASON_MOV_TO_MSR event.
>>>>> The param was added to the vm_event_mov_to_msr struct and to the
>>>>> hvm_monitor_msr function. Finally I've changed the bool_t param
>>>>> to a bool for the hvm_msr_write_intercept function.
>>>>> Signed-off-by: Alexandru Isaila <aisaila@xxxxxxxxxxxxxxx>
>>>>> Acked-by: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
>>>>> ---
>>>>> Changes since V1:
>>>>>   - Removed Stray blanks inside the inner parentheses
>>>>>   - Added space after the if statement
>>>>>   - Added * 8 to the set/clear/test_bit statements
>>>>>   - Removed the blank line after monitored_msr.
>>>>> ---
>>>>>  tools/libxc/include/xenctrl.h     |  2 +-
>>>>>  tools/libxc/xc_monitor.c          |  3 ++-
>>>> Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>
>>> Ping - AFAICT this patch has all the required acks?
>> Sure, and it is not being forgotten. But the tree is not fully open
>> yet, so it is subject to the judgment of committers whether any
>> patch (or series) is suitable of committing at this point. I did think
>> about committing this patch yesterday, but decided against
>> doing so. Please be a little more patient - even once the tree is
>> fully open, a few working days should be allowed for everyone
>> to deal with backlog, before sending pings.
> Sorry for the previous early ping. Just to check: is there action
> required on our part for this patch post-4.10 (rebasing, etc.)?

I think when I had last looked I did get the (wrong) impression
that there was a missing ack. I'll try to remember to commit this
with the next batch.


Xen-devel mailing list



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