On Fri, May 5, 2017 at 10:25 AM, Amir Goldstein <amir73il@xxxxxxxxx> wrote: > > You also forgot to mention in changes since v6: > > - store 'null' fh instead of 'invalid' fh And w.r.t. that change, the following comment in ovl_get_origin() is a bit confusing now, because the comment referring to 'invalid' file handles was removed from the function. /* Treat empty origin as "invalid" */