Received: by 2002:a05:7412:1492:b0:e2:908c:2ebd with SMTP id s18csp365966rdh; Wed, 23 Aug 2023 02:10:45 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGuZhus2hWaMlTtvFCwQ8HGkurHrljzReUxIwWTOVCEidnQnUqeEzIVxkoW5qjMcsWD7Zie X-Received: by 2002:a05:6a20:ceaf:b0:131:5995:340 with SMTP id if47-20020a056a20ceaf00b0013159950340mr9536557pzb.20.1692781845106; Wed, 23 Aug 2023 02:10:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1692781845; cv=none; d=google.com; s=arc-20160816; b=QaiwXh1CcTpQHb3k2JoDXj13XMzK8W2RlOfy3GBxpCSXUQLXTm+ye+EMhbuZqevshu NZDGXPbrIbKmp7UeX6xQ+lxGHNnMXKBrZ/3IshWF2mGozge2aAOw4+zsDIea9oeyIpze mL6LT3AlNYW7ryIIFPapFJOKTUTnV6YIrRg5/fBrUOwSqAULXp/IqCU8A2ODIVQ2mhuY Jec4Q71L3w7yhYzW3xXZ8AGrzE/Sh9gp504E1Yup4GIEH7ZC4dFGDuxSGmpMj7hjEHX1 xShQvliRS85rNUZQb8vtMnYueWNQm23O7gVmXky9ZHYSgIt7JTtcfphxTa6C2DhnVVzU z2Tg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=nFscSl+9Hb0Red5yg0BS4fabypR8KMp0W4tGw5r1qZc=; fh=eeKO3VAjWyY9Y+Xe5jAIx55gMih/TM2J6zSfKLvlQ0M=; b=NC5l7VkdCTORfIukNVc4w6bn+IPm+3qJ/u8E7oH+c+uuxC9+/YTptQhEubc0OrxTBl WgOpXT5+OZ33QyBIJfI2tX8SXUNR4swMO3McCqbBtWmGefvwQx74iwo7QkLJE5ONXdvp xvZ6+Hx5bqlYAmuHgZfMZcLt0TX4Bd/UKzgYRnhttyC1MBinZ01WbFkKMGeVf6Pf+IRO /Ab5g2ufJDAvCrTXeMfvun5+spDPJTdh8BMbRM8UGnjgKuWdHDyvXXnw/cUL7GthDMTu vsg0tY5d35Zi8243DEmrVyO4lrbOznAig7XyIKpAUa6PTn8XITYpQhXRNDnm08PWlv2r 129Q== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=fujitsu.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id cw4-20020a056a00450400b0068a685bf30fsi3168264pfb.271.2023.08.23.02.10.32; Wed, 23 Aug 2023 02:10:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=fujitsu.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232837AbjHWGMM (ORCPT + 99 others); Wed, 23 Aug 2023 02:12:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:54438 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229491AbjHWGML (ORCPT ); Wed, 23 Aug 2023 02:12:11 -0400 Received: from esa6.hc1455-7.c3s2.iphmx.com (esa6.hc1455-7.c3s2.iphmx.com [68.232.139.139]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 30256FB; Tue, 22 Aug 2023 23:12:09 -0700 (PDT) X-IronPort-AV: E=McAfee;i="6600,9927,10810"; a="130427829" X-IronPort-AV: E=Sophos;i="6.01,195,1684767600"; d="scan'208";a="130427829" Received: from unknown (HELO yto-r1.gw.nic.fujitsu.com) ([218.44.52.217]) by esa6.hc1455-7.c3s2.iphmx.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 23 Aug 2023 15:12:03 +0900 Received: from yto-m1.gw.nic.fujitsu.com (yto-nat-yto-m1.gw.nic.fujitsu.com [192.168.83.64]) by yto-r1.gw.nic.fujitsu.com (Postfix) with ESMTP id 965ABDB3C1; Wed, 23 Aug 2023 15:12:01 +0900 (JST) Received: from kws-ab4.gw.nic.fujitsu.com (kws-ab4.gw.nic.fujitsu.com [192.51.206.22]) by yto-m1.gw.nic.fujitsu.com (Postfix) with ESMTP id D2E06CFBC1; Wed, 23 Aug 2023 15:12:00 +0900 (JST) Received: from localhost.localdomain (unknown [10.167.234.230]) by kws-ab4.gw.nic.fujitsu.com (Postfix) with ESMTP id EB5E034786; Wed, 23 Aug 2023 15:11:58 +0900 (JST) From: Li Zhijian To: linux-rdma@vger.kernel.org Cc: zyjzyj2000@gmail.com, jgg@ziepe.ca, leon@kernel.org, linux-kernel@vger.kernel.org, rpearsonhpe@gmail.com, Li Zhijian , Daisuke Matsuda Subject: [PATCH v2 2/2] RDMA/rxe: Call rxe_set_mtu after rxe_register_device Date: Wed, 23 Aug 2023 14:11:41 +0800 Message-Id: <20230823061141.258864-2-lizhijian@fujitsu.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20230823061141.258864-1-lizhijian@fujitsu.com> References: <20230823061141.258864-1-lizhijian@fujitsu.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-TM-AS-GCONF: 00 X-TM-AS-Product-Ver: IMSS-9.1.0.1417-9.0.0.1002-27830.004 X-TM-AS-User-Approved-Sender: Yes X-TMASE-Version: IMSS-9.1.0.1417-9.0.1002-27830.004 X-TMASE-Result: 10--4.000400-10.000000 X-TMASE-MatchedRID: 5PeO86FyjNKIYY2hRSylnMQ4mpKyfkqZ0MQw+++ihy/qFRQsrMdGBLbS c9DZ5nsJJCq8bCHd+Z97L4dx4z+0QP9HU1IxzaoJSs47mbT7SAQh9mNF8ZPJ2J4Q+L3BXIWucwh qloOj04py+ROQfAr1jeaffHI8kAmiHY/bzRmIaZGqFx2c/3V5cf3l3BKhUmN/myiLZetSf8nyb6 HMFK1qe11j5mhaIsibJ0RPnyOnrZL5HE9BR4AjwlDK8Q+H/9loAyA/oPUrd9CnyISwAhUK7va+4 xdGh4Bc0fjan/nyiwMw3dACUxgEQ58xxltjNeXm0lMxy4Ho/gPAYLx7rnbR8rDQ8m3TqgloelpC XnG+JjvDGBZ1G8r1Sf2D6gx/0ozp X-TMASE-SNAP-Result: 1.821001.0001-0-1-22:0,33:0,34:0-0 X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_MED, SPF_HELO_PASS,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org rxe_set_mtu() will call rxe_info_dev() to print message, and rxe_info_dev() expects dev_name(rxe->ib_dev->dev) has been assigned. Previously since dev_name() is not set, when a new rxe link is being added, 'null' will be used as the dev_name like: "(null): rxe_set_mtu: Set mtu to 1024" Move rxe_register_device() earlier to assign the correct dev_name so that it can be read by rxe_set_mtu() later. And it's safe to do such change since mtu will not be used during the rxe_register_device() After this change, message becomes: "rxe_eth0: rxe_set_mtu: Set mtu to 4096" Reviewed-by: Daisuke Matsuda Signed-off-by: Li Zhijian --- drivers/infiniband/sw/rxe/rxe.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/sw/rxe/rxe.c b/drivers/infiniband/sw/rxe/rxe.c index a086d588e159..8a43c0c4f8d8 100644 --- a/drivers/infiniband/sw/rxe/rxe.c +++ b/drivers/infiniband/sw/rxe/rxe.c @@ -169,10 +169,13 @@ void rxe_set_mtu(struct rxe_dev *rxe, unsigned int ndev_mtu) */ int rxe_add(struct rxe_dev *rxe, unsigned int mtu, const char *ibdev_name) { + int ret; + rxe_init(rxe); + ret = rxe_register_device(rxe, ibdev_name); rxe_set_mtu(rxe, mtu); - return rxe_register_device(rxe, ibdev_name); + return ret; } static int rxe_newlink(const char *ibdev_name, struct net_device *ndev) -- 2.29.2