From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gregory Farnum Subject: Re: msgr2 protocol Date: Mon, 12 Sep 2016 17:03:50 -0700 Message-ID: References: <20160610190510.GA18999@degu.eng.arb.redhat.com> <20160611230503.GA18268@degu.eng.arb.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Return-path: Received: from mail-vk0-f49.google.com ([209.85.213.49]:33548 "EHLO mail-vk0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752105AbcIMADw (ORCPT ); Mon, 12 Sep 2016 20:03:52 -0400 Received: by mail-vk0-f49.google.com with SMTP id f76so159597614vke.0 for ; Mon, 12 Sep 2016 17:03:52 -0700 (PDT) In-Reply-To: Sender: ceph-devel-owner@vger.kernel.org List-ID: To: Sage Weil Cc: Haomai Wang , Marcus Watts , ceph-devel On Mon, Sep 12, 2016 at 6:21 AM, Sage Weil wrote: > On Mon, 12 Sep 2016, Haomai Wang wrote: >> This way is ok to me. So another change is double messenger >> instances(to v1 and v2) or let each messenger support multi binding >> addresses(this may need to refactor messenger interface). > > Yeah. I'm guessing we'll want to have an entity_addrvec_t with address > types mapped to different Messenger implementations (e.g., xio), so we'll > wan to allow multiple instance eventually. But we'll also just want to > allow multiple binding (v1 + v2, or ipv4 + ipv6). :/ Hmm, is that really necessary? It seems a fair bit more complicated and I'm not sure there's much payoff given the connection types. Long-term the only doubled connection I can see being needed is the client one; OSD cluster messengers will only be required to bind twice during the initial upgrade period. Put another way, what's the advantage of supporting two different protocols within one messenger? That just sounds like a disaster waiting to happen, and not one worth risking for slightly reducing the thread count on AsyncMessenger (especially with users coming from the SimpleMessenger). -Greg