[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] plugins: Expose physical addresses instead of device offsets
From: |
Aaron Lindsay |
Subject: |
Re: [PATCH] plugins: Expose physical addresses instead of device offsets |
Date: |
Mon, 8 Mar 2021 15:22:36 -0500 |
Alex,
I've now tested this change, and it is giving what appear to be valid
and correct physical addresses for both RAM and IO accesses in all the
cases I've thrown at it. My main concern with this patch at this point
is that I am concerned I may be breaking your new plugin here:
> +++ b/contrib/plugins/hwprofile.c
> @@ -201,7 +201,7 @@ static void vcpu_haddr(unsigned int cpu_index,
> qemu_plugin_meminfo_t meminfo,
> return;
> } else {
> const char *name = qemu_plugin_hwaddr_device_name(hwaddr);
> - uint64_t off = qemu_plugin_hwaddr_device_offset(hwaddr);
> + uint64_t off = qemu_plugin_hwaddr_phys_addr(hwaddr);
How angry is the plugin going to be that these are now physical
addresses instead of offsets?
-Aaron