On Mon, 11 Mar 2024 at 11:50, Peter Maydell <[email protected]> wrote: > > On Mon, 4 Mar 2024 at 01:28, <[email protected]> wrote: > > > > From: Fabiano Rosas <[email protected]> > > > > If we receive a file descriptor that points to a regular file, there's > > nothing stopping us from doing multifd migration with mapped-ram to > > that file. > > > > Enable the fd: URI to work with multifd + mapped-ram. > > > > Note that the fds passed into multifd are duplicated because we want > > to avoid cross-thread effects when doing cleanup (i.e. close(fd)). The > > original fd doesn't need to be duplicated because monitor_get_fd() > > transfers ownership to the caller. > > > > Signed-off-by: Fabiano Rosas <[email protected]> > > Reviewed-by: Peter Xu <[email protected]> > > Link: https://lore.kernel.org/r/[email protected] > > Signed-off-by: Peter Xu <[email protected]> > > Hi; Coverity points out some issues with this commit (CID 1539961, 1539965):
Oh, and also CID 1539960: there's a third instance of dup() with no error checking in another commit. thanks -- PMM
