Re: [PATCH 5/5] migration: Move the yank unregister of channel_close out

2021-07-21 Thread Peter Xu
On Wed, Jul 21, 2021 at 11:39:37AM +0100, Dr. David Alan Gilbert wrote: > > +/* > > + * Return the ioc object if it's a migration channel. Note: it can return > > NULL > > + * for callers passing in a non-migration qemufile. E.g. see > > qemu_fopen_bdrv() > > + * and its usage in e.g. load_snap

Re: [PATCH 5/5] migration: Move the yank unregister of channel_close out

2021-07-21 Thread Dr. David Alan Gilbert
* Peter Xu (pet...@redhat.com) wrote: > It's efficient, but hackish to call yank unregister calls in channel_close(), > especially it'll be hard to debug when qemu crashed with some yank function > leaked. > > Remove that hack, but instead explicitly unregister yank functions at the > places where

[PATCH 5/5] migration: Move the yank unregister of channel_close out

2021-07-20 Thread Peter Xu
It's efficient, but hackish to call yank unregister calls in channel_close(), especially it'll be hard to debug when qemu crashed with some yank function leaked. Remove that hack, but instead explicitly unregister yank functions at the places where needed, they are: (on src) - migrate_fd_clea