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

Re: [Xen-devel] [PATCH 1/6] x86/boot: Check for E820_RAM earlier when searching the E820

To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
From: Jan Beulich <jbeulich@xxxxxxxx>
Date: Tue, 7 Jan 2020 16:12:16 +0100
Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>
Delivery-date: Tue, 07 Jan 2020 15:12:28 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 06.01.2020 16:54, Andrew Cooper wrote:
> There is no point performing the masking calculations if we are going to
> throw the result away.

A reasonably optimizing compiler ought to do so. It's slightly less
source code the original way. Nevertheless I don't really mind the
change, so ...

> No functional change.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

References:
- [Xen-devel] [PATCH 0/6] x86/boot: Remove mappings at 0
  - From: Andrew Cooper
- [Xen-devel] [PATCH 1/6] x86/boot: Check for E820_RAM earlier when searching the E820
  - From: Andrew Cooper

Prev by Date: Re: [Xen-devel] [PATCH] xen/arm: vgic-v3: Fix the typo of GICD IRQ active status range
Next by Date: Re: [Xen-devel] [PATCH 2/6] x86/boot: Map the trampoline as read-only
Previous by thread: [Xen-devel] [PATCH 1/6] x86/boot: Check for E820_RAM earlier when searching the E820
Next by thread: [Xen-devel] [PATCH 3/6] x86/boot: Remove the preconstructed low 16M superpage mappings
Index(es):
- Date
- Thread

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