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

Re: [Xen-devel] [PATCH v3 7/8] minor fixes (formatting, comments, unused includes etc.)



On 7/8/2016 10:56 AM, Jan Beulich wrote:
On 06.07.16 at 17:55, <czuzu@xxxxxxxxxxxxxxx> wrote:
--- a/xen/arch/x86/hvm/hvm.c
+++ b/xen/arch/x86/hvm/hvm.c
@@ -31,7 +31,6 @@
  #include <xen/hypercall.h>
  #include <xen/guest_access.h>
  #include <xen/event.h>
-#include <xen/paging.h>
  #include <xen/cpu.h>
  #include <xen/wait.h>
  #include <xen/mem_access.h>
@@ -63,11 +62,9 @@
  #include <asm/hvm/nestedhvm.h>
  #include <asm/hvm/monitor.h>
  #include <asm/hvm/ioreq.h>
-#include <asm/hvm/vmx/vmx.h>
  #include <asm/altp2m.h>
  #include <asm/mtrr.h>
  #include <asm/apic.h>
-#include <asm/vm_event.h>
  #include <public/sched.h>
  #include <public/hvm/ioreq.h>
  #include <public/version.h>
The inclusion of asm/vm_event.h should really be removed in patch 2,
I had to drop it here, to get the patch to apply ahead of the earlier
ones in this series.

--- a/xen/arch/x86/monitor.c
+++ b/xen/arch/x86/monitor.c
@@ -21,7 +21,6 @@
#include <asm/hvm/vmx/vmx.h>
  #include <asm/monitor.h>
-#include <asm/vm_event.h>
  #include <public/vm_event.h>
static inline void write_ctrlreg_adjust_traps(struct domain *d, uint8_t index)
Similarly I had to drop this hunk, which suggests that one of the
earlier patches needlessly adds that #include (or it gets validly
added and then a later patch makes it unnecessary again).

Jan
Thanks for applying these ahead, it spares me having to include them in 
future series.
As for the issue, yeah that's probably what happened along the way 
although it was to be expected (applying patches from a series 
'unsequentially') so again, thanks for the effort.
Will keep in mind having to take care of the changes that didn't make it 
in a next series.
Corneliu.

_______________________________________________
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®.