From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-12.1 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 92793C35247 for ; Wed, 5 Feb 2020 19:23:00 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6661921741 for ; Wed, 5 Feb 2020 19:23:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1580930580; bh=Evt0EDSSw6OwMomJAB6cZdqBW/+iDAZiR03Ckl7UDkU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=y5yTOs+X5C6846RQvKKc/0vc+00db/hc+OZ3kz5DiSrg+atjhe+cL4n/OWv39eDX4 mPEjuFi+0FfMbejbC1xNMxS6PyY2T8XyIPUyMuuMIZfPVyZh9YO96fjAVTUIagmFMp WDVfjyglog/XLnT7tl51n7o33gtxD47kh2j/s494= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727199AbgBETW7 (ORCPT ); Wed, 5 Feb 2020 14:22:59 -0500 Received: from mail.kernel.org ([198.145.29.99]:39880 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727116AbgBETW7 (ORCPT ); Wed, 5 Feb 2020 14:22:59 -0500 Received: from localhost (unknown [213.57.247.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8E8582072B; Wed, 5 Feb 2020 19:22:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1580930579; bh=Evt0EDSSw6OwMomJAB6cZdqBW/+iDAZiR03Ckl7UDkU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=RLLkhOU66n1mK7t5MnUNS37BRe1n988IvnjDqHpV7hf4MTRev2j5i7ZRPT2rVdop+ dPP+//xJHgIuFDlMVKJQFsypMee3oV7nTRmQ3QoJN8tGhhMFrnnLS1BLCRUdroLCAg 6+j9lnCTB4FrY9X1v1rcmnT+K6QsVehO5OB/fK9U= Date: Wed, 5 Feb 2020 21:22:55 +0200 From: Leon Romanovsky To: Jason Gunthorpe Cc: "Goldman, Adam" , linux-rdma@vger.kernel.org, mike.marciniszyn@intel.com, dennis.dalessandro@intel.com Subject: Re: [PATCH] kernel-boot: Do not perform device rename on OPA devices Message-ID: <20200205192255.GB414821@unreal> References: <1580824520-38122-1-git-send-email-adam.goldman@intel.com> <20200205191227.GE28298@ziepe.ca> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200205191227.GE28298@ziepe.ca> Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rdma@vger.kernel.org On Wed, Feb 05, 2020 at 03:12:27PM -0400, Jason Gunthorpe wrote: > On Tue, Feb 04, 2020 at 08:55:20AM -0500, Goldman, Adam wrote: > > From: "Goldman, Adam" > > > > PSM2 will not run with recent rdma-core releases. Several tools and > > libraries like PSM2, require the hfi1 name to be present. > > > > Recent rdma-core releases added a new feature to rename kernel devices, > > but the default configuration will not work with hfi1 fabrics. > > > > Related opa-psm2 github issue: > > https://github.com/intel/opa-psm2/issues/43 > > > > Fixes: 5b4099d47be3 ("kernel-boot: Perform device rename to make stable names") > > Reviewed-by: Mike Marciniszyn > > Signed-off-by: Goldman, Adam > > kernel-boot/rdma-persistent-naming.rules | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/kernel-boot/rdma-persistent-naming.rules b/kernel-boot/rdma-persistent-naming.rules > > index 9b61e16..95d6851 100644 > > +++ b/kernel-boot/rdma-persistent-naming.rules > > @@ -25,4 +25,4 @@ > > # Device type = RoCE > > # mlx5_0 -> rocex525400c0fe123455 > > # > > -ACTION=="add", SUBSYSTEM=="infiniband", PROGRAM="rdma_rename %k NAME_FALLBACK" > > +ACTION=="add", SUBSYSTEM=="infiniband", KERNEL!="hfi1*", PROGRAM="rdma_rename %k NAME_FALLBACK" > > We are moving to the new names by default slowly, when wrong > assumptions are found in other packages they need to be updated and > their fixes pushed out. > > At some point the major distros will default this to On. People using > leading edge distros can turn it off with the global switch Leon > mentioned. > > This is the same process netdev went through when they introduced > persistent names. > > If I recall, hfi was one of the reason this work was done. HFI has > problems generating consistent names for its multi-function devices in > various cases and I NAK'd the kernel hack to try and 'fix' that. You remember correctly, it was related to the bug in production where physical ports were not aligned with their physical location and the module parameter was supposed to "reverse" the enumeration order. Something like that. Thanks > > Jason