On 3/24/22 07:50, Cornelia Huck wrote:
On Wed, Mar 23 2022, Eric Auger <eric.au...@redhat.com> wrote:

The CRB command buffer currently is a RAM MemoryRegion and given
its base address alignment, it causes an error report on
vfio_listener_region_add(). This region could have been a RAM device
region, easing the detection of such safe situation but this option
was not well received. So let's add a helper function that uses the
memory region owner type to detect the situation is safe wrt
the assignment. Other device types can be checked here if such kind
of problem occurs again.

Signed-off-by: Eric Auger <eric.au...@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org>

---

v2 -> v3:
- Use TPM_IS_CRB()

v1 -> v2:
- do not check the MR name but rather the owner type
---
  hw/vfio/common.c     | 27 ++++++++++++++++++++++++++-
  hw/vfio/trace-events |  1 +
  2 files changed, 27 insertions(+), 1 deletion(-)

Reviewed-by: Cornelia Huck <coh...@redhat.com>

Acked-by: Stefan Berger <stef...@linux.ibm.com>

Reply via email to