Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-06-03 Thread Daniel P . Berrangé
On Thu, May 23, 2024 at 04:05:31PM -0300, Fabiano Rosas wrote: > When the "file:" migration support was added we missed the special > case in the qemu_open_old implementation that allows for a particular > file name format to be used to refer to a set of file descriptors that > have been previously

Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-31 Thread Fabiano Rosas
Peter Xu writes: > On Thu, May 23, 2024 at 04:05:31PM -0300, Fabiano Rosas wrote: >> When the "file:" migration support was added we missed the special >> case in the qemu_open_old implementation that allows for a particular >> file name format to be used to refer to a set of file descriptors tha

Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-30 Thread Peter Xu
On Thu, May 23, 2024 at 04:05:31PM -0300, Fabiano Rosas wrote: > When the "file:" migration support was added we missed the special > case in the qemu_open_old implementation that allows for a particular > file name format to be used to refer to a set of file descriptors that > have been previously

Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-24 Thread Prasad Pandit
On Fri, 24 May 2024 at 18:00, Fabiano Rosas wrote: > That's the point. If offset==0 we truncate all the way, if not, we truncate > to the offset. * Yes, I was wondering if the migration file has some data, but still 'offset' ends up being zero(0). If that's unlikely to happen, then we are good.

Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-24 Thread Fabiano Rosas
Prasad Pandit writes: > On Fri, 24 May 2024 at 00:38, Fabiano Rosas wrote: >> This is further indicated by the presence of the 'offset' >> argument, which indicates the start of the region where QEMU is >> allowed to write. >> >> Fix the issue by replacing the O_TRUNC flag on open by an ftruncat

Re: [PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-24 Thread Prasad Pandit
On Fri, 24 May 2024 at 00:38, Fabiano Rosas wrote: > This is further indicated by the presence of the 'offset' > argument, which indicates the start of the region where QEMU is > allowed to write. > > Fix the issue by replacing the O_TRUNC flag on open by an ftruncate > call, which will take the o

[PATCH v2 01/18] migration: Fix file migration with fdset

2024-05-23 Thread Fabiano Rosas
When the "file:" migration support was added we missed the special case in the qemu_open_old implementation that allows for a particular file name format to be used to refer to a set of file descriptors that have been previously provided to QEMU via the add-fd QMP command. When using this fdset fe