refactor migration tests and small test fixes

Signed-off-by: William Henderson <[email protected]>

test/py/libvfio_user.py

@@ -1287,4 +1287,20 @@ def get_bitmap_size(size: int, pgsize: int) -> int:
     return ((nr_pages + 63) & ~63) // 8
 
 
+get_errno_loc = libc.__errno_location
+get_errno_loc.restype = c.POINTER(c.c_int)
+
+
+def set_real_errno(errno: int):
+    """
+    ctypes's errno is an internal value that only updates the real value when
+    the foreign function call returns. In callbacks, however, this doesn't
+    happen, so `c.set_errno` doesn't propagate in time. In this case we need to
+    manually set the real errno.
+    """
+
+    c.set_errno(errno)  # set internal errno so `c.get_errno` gives right value
+    get_errno_loc()[0] = errno  # set real errno
+
+
 # ex: set tabstop=4 shiftwidth=4 softtabstop=4 expandtab: #

test/py/test_dirty_pages.py

@@ -157,8 +157,8 @@ def get_dirty_page_bitmap(addr=0x10 << PAGE_SHIFT, length=0x10 << PAGE_SHIFT,
                           page_size=PAGE_SIZE, expect=0):
     """
     Get the dirty page bitmap from the server for the given region and page
-    size as a 64-bit integer. This only works for bitmaps that fit within a
-    64-bit integer.
+    size as a 64-bit integer. This function only works for bitmaps that fit
+    within a 64-bit integer because that's what it returns.
     """
 
     bitmap_size = get_bitmap_size(length, page_size)