On 29/08/17 15:14, Jan Beulich wrote:
On 29.08.17 at 13:19, wrote:
>> --- a/xen/arch/x86/mm.c
>> +++ b/xen/arch/x86/mm.c
>> @@ -667,45 +667,49 @@ static int alloc_segdesc_page(struct page_info *page)
>> }
>>
>>
>> -/* Map shadow page at offset @off. */
>> -int map_ldt_shadow_page(unsigne
>>> On 29.08.17 at 13:19, wrote:
> --- a/xen/arch/x86/mm.c
> +++ b/xen/arch/x86/mm.c
> @@ -667,45 +667,49 @@ static int alloc_segdesc_page(struct page_info *page)
> }
>
>
> -/* Map shadow page at offset @off. */
> -int map_ldt_shadow_page(unsigned int off)
> +/*
> + * Map a guests LDT page (a
On Tue, Aug 29, 2017 at 12:19:13PM +0100, Andrew Cooper 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 lim
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
