[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 00/52] migration/rdma: Error handling fixes
From: |
Juan Quintela |
Subject: |
Re: [PATCH 00/52] migration/rdma: Error handling fixes |
Date: |
Tue, 31 Oct 2023 11:25:42 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.3 (gnu/linux) |
Daniel P. Berrangé <berrange@redhat.com> wrote:
> On Wed, Oct 04, 2023 at 08:00:34PM +0200, Juan Quintela wrote:
>> Daniel P. Berrangé <berrange@redhat.com> 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 refraining from
>> >> > saying more; see the commit messages instead.
>> >> >
>> >> > Issues remaining after this series include:
>> >> >
>> >> > * Terrible error messages
>> >> >
>> >> > * Some error message cascades remain
>> >> >
>> >> > * There is no written contract for QEMUFileHooks, and the
>> >> > responsibility for reporting errors is unclear
>> >>
>> >> Even being removed.. because no one is really extending that..
>> >>
>> >> https://lore.kernel.org/all/20230509120700.78359-1-quintela@redhat.com/#t
>> >
>> > One day (in another 5-10 years) I still hope we'll get to
>> > the point where QEMUFile itself is obsolete :-)
>>
>> If you see the patches on list, I have move rate limit check outside of
>> QEMUFile, so right now it is only a buffer to write in the main
>> migration thread.
>
> Can you point me to that patch(es) as I'm not identifying
> them yet.
Yet another set of counters.
They are on today PULL request.
Later, Juan.