aboutsummaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorThanos Makatos <thanos.makatos@nutanix.com>2021-01-25 17:03:28 +0000
committerGitHub <noreply@github.com>2021-01-25 17:03:28 +0000
commit9dcc46fee15f11724e3e3d7ccef2619f48501f41 (patch)
treefc62e3fac1466ba5dc92d172e928139f4bd3cc05 /lib
parentef17a8af3e7ad2681705d817b31c2d167fd50c14 (diff)
downloadlibvfio-user-9dcc46fee15f11724e3e3d7ccef2619f48501f41.zip
libvfio-user-9dcc46fee15f11724e3e3d7ccef2619f48501f41.tar.gz
libvfio-user-9dcc46fee15f11724e3e3d7ccef2619f48501f41.tar.bz2
minor comments (#262)
Signed-off-by: Thanos Makatos <thanos.makatos@nutanix.com>
Diffstat (limited to 'lib')
-rw-r--r--lib/migration.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/lib/migration.c b/lib/migration.c
index 573feaf..02d6141 100644
--- a/lib/migration.c
+++ b/lib/migration.c
@@ -212,7 +212,7 @@ handle_pending_bytes(vfu_ctx_t *vfu_ctx, struct migration *migr,
case VFIO_USER_MIGR_ITER_STATE_DATA_PREPARED:
/*
* FIXME what happens if data haven't been consumed in the previous
- * iteration? Ask on LKML.
+ * iteration? Check https://www.spinics.net/lists/kvm/msg228608.html.
*/
if (*pending_bytes == 0) {
migr->iter.state = VFIO_USER_MIGR_ITER_STATE_FINISHED;
@@ -450,6 +450,14 @@ migration_region_access(vfu_ctx_t *vfu_ctx, char *buf, size_t count,
if (is_write) {
ret = migr->callbacks.write_data(vfu_ctx, buf, count, pos);
} else {
+ /*
+ * FIXME <linux/vfio.h> says:
+ *
+ * d. Read data_size bytes of data from (region + data_offset) from the
+ * migration region.
+ *
+ * Does this mean that partial reads are not allowed?
+ */
ret = migr->callbacks.read_data(vfu_ctx, buf, count, pos);
}
}