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

Re: [Xen-devel] [Qemu-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)



Anthony, I've split the reorganization and code style issues in a separate
patch series which I've posted today:

http://markmail.org/message/feu6u3f7py5vt77v

Thanks


On Mon, Jul 25, 2016 at 5:09 PM, Anthony PERARD
<anthony.perard@xxxxxxxxxx> wrote:
> On Sun, Jul 10, 2016 at 02:47:31PM +0300, Emil Condrea wrote:
>> Emil Condrea (19):
>>   xen: Create a new file xen_pvdev.c
>>   xen: Create a new file xen_frontend.c
>>   xen: Move xenstore_update to xen_pvdev.c
>>   xen: Move evtchn functions to xen_pvdev.c
>>   xen: Prepare xendev qtail to be shared with frontends
>>   xen: Rename xen_be_printf to xen_pv_printf
>>   xen: Rename xen_be_unbind_evtchn
>>   xen: Rename xen_be_send_notify
>>   xen: Rename xen_be_evtchn_event
>>   xen: Rename xen_be_find_xendev
>>   xen: Rename xen_be_del_xendev
>>   xen: Rename xen_be_frontend_changed
>
> Patches from 1 to 12 looks fine but ./script/checkpatch.pl reveal some
> coding style issue in the code that is moved. Could you fix those issues
> in separate patches?  (Having separate patches for code movement and for
> coding style fix makes it easier to review.)
>
> I'll start reviewing the more complicated patches now.
>
> Thanks,
>
> --
> Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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