Re: [Xen-devel] [PATCH v2 2/5] x86/pv: map_ldt_shadow_page() cleanup

2017-08-30 Thread Jan Beulich
>>> On 29.08.17 at 17:57, wrote: > Switch the return value from int to bool, to match its semantics. Switch its > parameter from a frame offset to a byte offset (simplifying the sole caller) > and allowing for an extra sanity check that the fault is within the LDT > limit. > > Drop the unnecess

[Xen-devel] [PATCH v2 2/5] x86/pv: map_ldt_shadow_page() cleanup

2017-08-29 Thread Andrew Cooper
Switch the return value from int to bool, to match its semantics. Switch its parameter from a frame offset to a byte offset (simplifying the sole caller) and allowing for an extra sanity check that the fault is within the LDT limit. Drop the unnecessary gmfn and okay local variables, and correct