Hi,
On 26 October 2017 at 16:47, Andrew Cooper <andrew.coop...@citrix.com> wrote:
> On 26/10/17 12:13, Wei Liu wrote:
>> On Wed, Oct 25, 2017 at 02:57:05PM +0530, Bhupinder Thakur wrote:
>>> Currently the type of console mfn is unsigned long in libxl. This may be
>>> an issue for 32-bit toolstack running on 64-bit Xen, where the pfn are
>>> 64 bit. To ensure that console_mfn can hold any valid 64-bit pfn, the
>>> type of console_mfn is changed to xen_pfn_t.
>>>
>>> Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org>
>>> ---
>>> CC: Ian Jackson <ian.jack...@eu.citrix.com>
>>> CC: Wei Liu <wei.l...@citrix.com>
>>> CC: Stefano Stabellini <sstabell...@kernel.org>
>>> CC: Julien Grall <julien.gr...@arm.com>
>>>
>>> This patch is as per the review of commit fa1f157
>>> libxl: Fix the bug introduced in commit "libxl: use correct type
>>>
>>> tools/libxl/libxl_console.c | 2 +-
>>> tools/libxl/libxl_dom.c | 2 +-
>>> tools/libxl/libxl_internal.h | 2 +-
>>> 3 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/tools/libxl/libxl_console.c b/tools/libxl/libxl_console.c
>>> index 6bfc0e5..f2ca689 100644
>>> --- a/tools/libxl/libxl_console.c
>>> +++ b/tools/libxl/libxl_console.c
>>> @@ -329,7 +329,7 @@ int libxl__device_console_add(libxl__gc *gc, uint32_t
>>> domid,
>>> flexarray_append(ro_front, "port");
>>> flexarray_append(ro_front, GCSPRINTF("%"PRIu32,
>>> state->console_port));
>>> flexarray_append(ro_front, "ring-ref");
>>> - flexarray_append(ro_front, GCSPRINTF("%lu", state->console_mfn));
>>> + flexarray_append(ro_front, GCSPRINTF("%"PRIu_xen_pfn,
>>> state->console_mfn));
>> Actually, please consider changing console_mfn to console_pfn.
>
> If you are going to make this change, then it is a gfn, not a pfn.
> (console_pfn would be as equally wrong for PV guests as console_mfn is
> currently wrong for HVM guest.)
Changing console_mfn to console_gfn will require changes in many
files. Should I go ahead and change all the files?
Regards,
Bhupinder
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
