On 8/13/25 16:17, Steve Sistare wrote:
Set an error message if vfio_cpr_ram_discard_register_listener fails so
the fail label gets a valid error object.

Reported-by: Cédric Le Goater <[email protected]>
Fixes: eba1f657cbb1 ("vfio/container: recover from unmap-all-vaddr failure")
Signed-off-by: Steve Sistare <[email protected]>
---
  hw/vfio/listener.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/hw/vfio/listener.c b/hw/vfio/listener.c
index f498e23..5ebafaa 100644
--- a/hw/vfio/listener.c
+++ b/hw/vfio/listener.c
@@ -574,6 +574,9 @@ void vfio_container_region_add(VFIOContainerBase 
*bcontainer,
              vfio_ram_discard_register_listener(bcontainer, section);
          } else if (!vfio_cpr_ram_discard_register_listener(bcontainer,
                                                             section)) {
+            error_setg(&err,
+                       "vfio_cpr_ram_discard_register_listener for %s failed",
+                       memory_region_name(section->mr));
              goto fail;
          }
          return;


Applied to vfio-next.

Thanks,

C.



Reply via email to