Message ID | 20240502103538.243789-1-yu.zhang@ionos.com |
---|---|
State | New |
Headers | show |
Series | [v2] MAINTAINERS: Update the reviewers for RDMA migration | expand |
On Thu, May 02, 2024 at 12:35:38PM +0200, Yu Zhang wrote: > As the links [1][2] below stated, QEMU development community is currently > having some difficulties in maintaining the RDMA migration subsystem due > to the lack of resources (maintainers, test cases, test environment etc.) > and considering to deprecate it. This is true. > > According to our user experience in the recent two years, we observed that > RDMA is capable of providing higher migration speed and lower performance > impact to a running VM, which can significantly improve the end-user's > experience during the VM live migration. We believe that RDMA still plays > a key role for the QoS and can't yet be replaced by TCP/IP for VM migration > at the moment. We need numbers, and that's also what I was requesting. Per my understanding this is not yet compared, so IMHO we shouldn't state that before we have some test results. Note that I'm not NIC provider, I don't sell cards, I don't earn anything out of it. However I want to make sure commit messages are true and fair when I review a patch. It also matters when I want to provide more information to future QEMU users to not misguide on how they should see RDMA under any doc of migration section. IMHO we can continue the discussion in the other thread to finish it, rather than rushing on a MAINTAINERS file update; nobody is going to remove the code without finishing the discussion so no worry. Besides, there's also what Dan used to ask on the possibility of wrapping RDMA channels with QIOChannels. I thought it won't easily work, but again definitely be nice if you can help to double check with your RDMA expertises. > > Jinpu Wang is the upstream maintainer of RNBD/RTRS. He is experienced in > RDMA programming, and Yu Zhang maintains the downstream QEMU for IONOS > cloud in production. > > With the consent and supports from Michael Galaxy, who has developed this > feature for QEMU, we would like to take over the maintainer's role and > create the necessary resources to maintain it further for the community. So I suppose this patch is not yet ready for merge, either we need to wait for a true test results coming, or we just need some refinement on the statements. I hope I'm not asking too much here, but would you mind also provide a RDMA documentation patch too? It's under docs/migration/ currently, and now any new feature is required to have such for migration: https://www.qemu.org/docs/master/devel/migration/features.html RDMA is not a new one, but if we want to keep it, let's make it serious. IMHO it'll be nice to provide suggestions to user on when an user should consider using RDMA migrations, or if it's only in maintenance stage we can also mention there so we suggest people stick with TCP. Basically we have a place to provide a summary of all our discussions, so somehow we can consolidate such discussion there. Last important thing: you may want to copy Fabiano, he's going to take whatever patch coming for 9.1. Thanks, > > [1] https://mail.gnu.org/archive/html/qemu-devel/2024-04/msg00001.html > [2] https://mail.gnu.org/archive/html/qemu-devel/2024-04/msg00228.html > > Signed-off-by: Yu Zhang <yu.zhang@ionos.com> > Signed-off-by: Jack Wang <jinpu.wang@ionos.com> > Reviewed-by: Michael Galaxy <mgalaxy@akamai.com> > Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org> > Cc: Li Zhijian <lizhijian@fujitsu.com> > Cc: Peter Xu <peterx@redhat.com> > --- > v2: > - fixed an email address > - added "Tested-by: " > MAINTAINERS | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index f1f692202519..fa9c62aae3e1 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3425,7 +3425,10 @@ F: util/userfaultfd.c > X: migration/rdma* > > RDMA Migration > +R: Yu Zhang <yu.zhang@ionos.com> > +R: Jack Wang <jinpu.wang@ionos.com> > R: Li Zhijian <lizhijian@fujitsu.com> > +R: Michael Galaxy <mgalaxy@akamai.com> > R: Peter Xu <peterx@redhat.com> > S: Odd Fixes > F: migration/rdma* > -- > 2.34.1 >
diff --git a/MAINTAINERS b/MAINTAINERS index f1f692202519..fa9c62aae3e1 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3425,7 +3425,10 @@ F: util/userfaultfd.c X: migration/rdma* RDMA Migration +R: Yu Zhang <yu.zhang@ionos.com> +R: Jack Wang <jinpu.wang@ionos.com> R: Li Zhijian <lizhijian@fujitsu.com> +R: Michael Galaxy <mgalaxy@akamai.com> R: Peter Xu <peterx@redhat.com> S: Odd Fixes F: migration/rdma*