From 94654419c5f8a9721de34ed4f7bacc80cc4a120f Mon Sep 17 00:00:00 2001 From: nick Date: Mon, 23 Sep 2013 10:53:55 +0100 Subject: [PATCH] serve: Add a comment clarifying that a behaviour is safe --- src/serve.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/serve.c b/src/serve.c index 90d0dea..0d52571 100644 --- a/src/serve.c +++ b/src/serve.c @@ -665,7 +665,11 @@ void* build_allocation_map_thread(void* serve_uncast) * *incomplete* record of writes. Nobody will use it, as * allocation_map_built == 0 for the lifetime of the process. * - * The stream functionality can still be relied on. + * The stream functionality can still be relied on. We don't need to + * worry about mirroring waiting for the allocation map to finish, + * because we already copy every byte at least once. If that changes in + * the future, we'll need to wait for the allocation map to finish or + * fail before we can complete the migration. */ warn( "Didn't build allocation map for %s", serve->filename ); }