From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751541AbeCWE2H convert rfc822-to-8bit (ORCPT ); Fri, 23 Mar 2018 00:28:07 -0400 Received: from prv-mh.provo.novell.com ([137.65.248.74]:57828 "EHLO prv-mh.provo.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751175AbeCWE2G (ORCPT ); Fri, 23 Mar 2018 00:28:06 -0400 Message-Id: <5AB4F2CC020000F9000B145C@prv-mh.provo.novell.com> X-Mailer: Novell GroupWise Internet Agent 18.0.0 Date: Thu, 22 Mar 2018 22:27:56 -0600 From: "Gang He" To: , "Gang He" Cc: , Subject: Re: [PATCH] dlm: prompt the user SCTP is experimental References: <1521528327-882-1-git-send-email-ghe@suse.com> In-Reply-To: <1521528327-882-1-git-send-email-ghe@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8BIT Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello David, Do you agree to add this prompt to the user? Since sometimes customers attempted to setup SCTP protocol with two rings, but they could not get the expected result, then it maybe bring some concerns to the customer for DLM qualities. Thanks Gang >>> > As you know, DLM module can use TCP or SCTP protocols to > communicate among the cluster. > But, according to our testing, SCTP protocol is still considered > experimental, since not all aspects are working correctly and > it is not full tested. > e.g. SCTP connection channel switch needs about 5mins hang in case > one connection(ring) is broken. > Then, I suggest to add a kernel print, which prompts the user SCTP > protocol for DLM should be considered experimental, it is not > recommended in production environment. > > Signed-off-by: Gang He > --- > fs/dlm/lowcomms.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/fs/dlm/lowcomms.c b/fs/dlm/lowcomms.c > index cff79ea..18fd85d 100644 > --- a/fs/dlm/lowcomms.c > +++ b/fs/dlm/lowcomms.c > @@ -1307,6 +1307,7 @@ static int sctp_listen_for_all(void) > return -ENOMEM; > > log_print("Using SCTP for communications"); > + log_print("SCTP protocol is experimental, use at your own risk"); > > result = sock_create_kern(&init_net, dlm_local_addr[0]->ss_family, > SOCK_STREAM, IPPROTO_SCTP, &sock); > -- > 1.8.5.6