Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-10-31 Thread Juan Quintela
Daniel P. Berrangé wrote: > On Wed, Oct 04, 2023 at 08:00:34PM +0200, Juan Quintela wrote: >> Daniel P. Berrangé wrote: >> > On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: >> >> On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: >> >> > Oh dear, where to start. There's

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-10-05 Thread Daniel P . Berrangé
On Wed, Oct 04, 2023 at 08:00:34PM +0200, Juan Quintela wrote: > Daniel P. Berrangé wrote: > > On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: > >> On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: > >> > Oh dear, where to start. There's so much wrong, and in pretty obv

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-10-04 Thread Juan Quintela
Daniel P. Berrangé wrote: > On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: >> On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: >> > Oh dear, where to start. There's so much wrong, and in pretty obvious >> > ways. This code should never have passed review. I'm refrai

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-28 Thread Markus Armbruster
Thank you for reviewing! Especially Li Zhijian, who went through pretty much all the patches. Big chunk of work, much appreciated. Will send v2 shortly.

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-25 Thread Zhijian Li (Fujitsu)
On 22/09/2023 23:21, Peter Xu wrote: > On Thu, Sep 21, 2023 at 08:27:24AM +, Zhijian Li (Fujitsu) wrote: >> I'm worried that I may not have enough time, ability, or environment to >> review/test >> the RDMA patches. but for this patch set, i will take a look later. > > That'll be helpful, t

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-22 Thread Peter Xu
On Thu, Sep 21, 2023 at 08:27:24AM +, Zhijian Li (Fujitsu) wrote: > I'm worried that I may not have enough time, ability, or environment to > review/test > the RDMA patches. but for this patch set, i will take a look later. That'll be helpful, thanks! So it seems maybe at least we should hav

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-21 Thread Zhijian Li (Fujitsu)
Perter, On 20/09/2023 00:49, Peter Xu wrote: > On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: >> Oh dear, where to start. There's so much wrong, and in pretty obvious >> ways. This code should never have passed review. I'm refraining from >> saying more; see the commit mess

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-20 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: >> On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: >> > Oh dear, where to start. There's so much wrong, and in pretty obvious >> > ways. This code should never have passed review. I'm refr

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-19 Thread Fabiano Rosas
Daniel P. Berrangé writes: > On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: >> On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: >> > Oh dear, where to start. There's so much wrong, and in pretty obvious >> > ways. This code should never have passed review. I'm refr

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-19 Thread Daniel P . Berrangé
On Tue, Sep 19, 2023 at 12:49:46PM -0400, Peter Xu wrote: > On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: > > Oh dear, where to start. There's so much wrong, and in pretty obvious > > ways. This code should never have passed review. I'm refraining from > > saying more; see t

Re: [PATCH 00/52] migration/rdma: Error handling fixes

2023-09-19 Thread Peter Xu
On Mon, Sep 18, 2023 at 04:41:14PM +0200, Markus Armbruster wrote: > Oh dear, where to start. There's so much wrong, and in pretty obvious > ways. This code should never have passed review. I'm refraining from > saying more; see the commit messages instead. > > Issues remaining after this serie

[PATCH 00/52] migration/rdma: Error handling fixes

2023-09-18 Thread Markus Armbruster
Oh dear, where to start. There's so much wrong, and in pretty obvious ways. This code should never have passed review. I'm refraining from saying more; see the commit messages instead. Issues remaining after this series include: * Terrible error messages * Some error message cascades remain