On 5/22/19 2:13 PM, Paul Durrant wrote: >> -----Original Message----- >> From: Alexandru Stefan ISAILA [mailto:aisa...@bitdefender.com] >> Sent: 20 May 2019 13:55 >> To: xen-devel@lists.xenproject.org >> Cc: Paul Durrant <paul.durr...@citrix.com>; jbeul...@suse.com; Andrew Cooper >> <andrew.coop...@citrix.com>; Wei Liu <wei.l...@citrix.com>; Roger Pau Monne >> <roger....@citrix.com>; >> rcojoc...@bitdefender.com; ta...@tklengyel.com; George Dunlap >> <george.dun...@citrix.com>; Alexandru >> Stefan ISAILA <aisa...@bitdefender.com> >> Subject: [PATCH v4 1/2] x86/emulate: Move hvmemul_linear_to_phys >> >> Thiis is done so hvmemul_linear_to_phys() can be called from >> hvmemul_send_vm_event(). >> >> Signed-off-by: Alexandru Isaila <aisa...@bitdefender.com> >> --- >> xen/arch/x86/hvm/emulate.c | 181 ++++++++++++++++++------------------- >> 1 file changed, 90 insertions(+), 91 deletions(-) >> >> diff --git a/xen/arch/x86/hvm/emulate.c b/xen/arch/x86/hvm/emulate.c >> index 8659c89862..254ff6515d 100644 >> --- a/xen/arch/x86/hvm/emulate.c >> +++ b/xen/arch/x86/hvm/emulate.c >> @@ -530,6 +530,95 @@ static int hvmemul_do_mmio_addr(paddr_t mmio_gpa, >> return hvmemul_do_io_addr(1, mmio_gpa, reps, size, dir, df, ram_gpa); >> } >> > > I know this is code movement, but it would probably good to a do a bit of > tidying...
I think there are different minds on this; I *generally* prefer pure code movement to be with as few changes as possible, to make sure actual changes are easy to call out. The changes you've asked for are pretty minor (and you're the maintainer of the file it's being moved to), so I won't argue about it in this particular case. Just want to counter the idea that move + change is the norm. :-) -George _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel