Received: by 2002:a05:7412:b10a:b0:f3:1519:9f41 with SMTP id az10csp956221rdb; Fri, 1 Dec 2023 03:20:14 -0800 (PST) X-Google-Smtp-Source: AGHT+IFEYSE3GLJRiuW5gv1JW84k1JLWmAkisWAhEj5Wjlafcb/Dz6nP2IDExxwWxAw6oL4AK+Dk X-Received: by 2002:a17:902:cf43:b0:1cf:c2dc:1c2c with SMTP id e3-20020a170902cf4300b001cfc2dc1c2cmr22291662plg.52.1701429614149; Fri, 01 Dec 2023 03:20:14 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701429614; cv=none; d=google.com; s=arc-20160816; b=SeFny/ZNOPZQQd/dsrPT2aUaWxtcSVBLFiE5+if1RDqxWcILioquGWuDO+gbdxPAmt 7b/3FAF9hNCgInDbqGGG+B3zmJ2DWQbqCI6KjuCyYqg9Vhopz00l0QbPo1luTHW886aL vq0yAqu6aJJh6Cba1A8ziRTESbu3Vb9Nj4qhgHEZ0l/0twHby3gPouYD/J34Zlp1jeN4 q816DcqIaB2UN13w+SiXWbs9DpQGQGY2WrPBzlhdiyxcsw7kj5OESBpg75RFaFdtgs2Q xlV38jwx+TGm9DL1rGeogleAabmKCYgyhmKsZFS2ieawbyCiTRkwH6lT+bKrwi28UFam ZyiA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=SQeMsnkSmvQHWc9TP1DATsAWIlnBwjDXrz6OTIEeqsQ=; fh=L2dZ3vo+sL7V47T3m5n5PglltCanschjvGCixJo9Cn0=; b=tq1hD3AoZ7HmygqyT+kJNVAsfRdMTv6Be1mAuRjZLrJn25dSO9x1Vk4CV/IKhAkvUZ MUaaacCfRKgtwzEhKVWZzXaHnjbyAXaQfsdL6tWCHE1iZrQAZFQOuI/yNxYIweuot69Z 4qxqQNmvPCpEBL6G1t1QkApxlx6EhydCQ7xXlZXWEWBN2oIfToXGNCh03RFVMCT8mzSJ P+RtwNXiw2iWO3AuIqA4p2qoUSXgAgVWoRONp+NS9qKm5ACcCgiU33P1CEIZiGN7v0Yp ldoWyPfM6BoqWxLd6DHZKI8B42rTz+iMpmdKtUpKTFcKas52MHKsRai7rgsh68hZKTVO SvSA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=kAQsN+K3; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=NONE dis=NONE) header.from=ibm.com Return-Path: Received: from howler.vger.email (howler.vger.email. [23.128.96.34]) by mx.google.com with ESMTPS id t5-20020a170902a5c500b001cf68dcabc2si2965394plq.545.2023.12.01.03.20.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Dec 2023 03:20:14 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34; Authentication-Results: mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=kAQsN+K3; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=NONE dis=NONE) header.from=ibm.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by howler.vger.email (Postfix) with ESMTP id 1605D80B9501; Fri, 1 Dec 2023 03:20:00 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at howler.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1378496AbjLALTm (ORCPT + 99 others); Fri, 1 Dec 2023 06:19:42 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47392 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1378535AbjLALTc (ORCPT ); Fri, 1 Dec 2023 06:19:32 -0500 Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7C73910DF; Fri, 1 Dec 2023 03:18:56 -0800 (PST) Received: from pps.filterd (m0353725.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3B1B8FZv018133; Fri, 1 Dec 2023 11:18:49 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=message-id : date : mime-version : subject : to : cc : references : from : in-reply-to : content-type : content-transfer-encoding; s=pp1; bh=SQeMsnkSmvQHWc9TP1DATsAWIlnBwjDXrz6OTIEeqsQ=; b=kAQsN+K3EGsgSamBNfuGLzBbEgW0jSGh5sp+407hkH+878d1dKL/Z2jB3xqC4Q0XLr9i sLYq1+Glf+acycicHlvSzXz4XV9GLCL78miJUaycDbi7bMp1+ACUnvlxlTI9bdK9Uw7P JIOVsG6AVJn4SS8o3FZrUIRTCH5Vf29PqBNFiqukw91BRkn2bX/va62bGnTnvcEvZued uSJ+9euorxY57lTe0x8mHxTsygEvPCFVGuENo+om+BoTw+3XBK2b/32Lq/fI9gADlUXB TigdGzpRXl02uypV1pK6jyCaUUHjzIYknEXWpJ8wcSDMpraGjJ3TiNs7bkTVbgt8BEXL kw== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 3uqebmrab6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 01 Dec 2023 11:18:49 +0000 Received: from m0353725.ppops.net (m0353725.ppops.net [127.0.0.1]) by pps.reinject (8.17.1.5/8.17.1.5) with ESMTP id 3B1B8vZ4021075; Fri, 1 Dec 2023 11:18:48 GMT Received: from ppma23.wdc07v.mail.ibm.com (5d.69.3da9.ip4.static.sl-reverse.com [169.61.105.93]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 3uqebmraas-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 01 Dec 2023 11:18:48 +0000 Received: from pps.filterd (ppma23.wdc07v.mail.ibm.com [127.0.0.1]) by ppma23.wdc07v.mail.ibm.com (8.17.1.19/8.17.1.19) with ESMTP id 3B1AY0SY020438; Fri, 1 Dec 2023 11:18:47 GMT Received: from smtprelay03.fra02v.mail.ibm.com ([9.218.2.224]) by ppma23.wdc07v.mail.ibm.com (PPS) with ESMTPS id 3ukvrm4gy0-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 01 Dec 2023 11:18:47 +0000 Received: from smtpav04.fra02v.mail.ibm.com (smtpav04.fra02v.mail.ibm.com [10.20.54.103]) by smtprelay03.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 3B1BIiUu20906498 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Fri, 1 Dec 2023 11:18:44 GMT Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 88C4420043; Fri, 1 Dec 2023 11:18:44 +0000 (GMT) Received: from smtpav04.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 4FFC420040; Fri, 1 Dec 2023 11:18:43 +0000 (GMT) Received: from [9.179.28.5] (unknown [9.179.28.5]) by smtpav04.fra02v.mail.ibm.com (Postfix) with ESMTP; Fri, 1 Dec 2023 11:18:43 +0000 (GMT) Message-ID: Date: Fri, 1 Dec 2023 12:18:42 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next v3 7/7] net/smc: manage system EID in SMC stack instead of ISM driver To: Wen Gu , wenjia@linux.ibm.com, hca@linux.ibm.com, gor@linux.ibm.com, agordeev@linux.ibm.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, kgraul@linux.ibm.com, jaka@linux.ibm.com Cc: borntraeger@linux.ibm.com, svens@linux.ibm.com, alibuda@linux.alibaba.com, tonylu@linux.alibaba.com, raspl@linux.ibm.com, schnelle@linux.ibm.com, linux-s390@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org References: <1701343695-122657-1-git-send-email-guwen@linux.alibaba.com> <1701343695-122657-8-git-send-email-guwen@linux.alibaba.com> Content-Language: en-US From: Alexandra Winter In-Reply-To: <1701343695-122657-8-git-send-email-guwen@linux.alibaba.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 X-Proofpoint-ORIG-GUID: FhuwSygUbyqQmJDW3VqesCgWrJ_W82Es X-Proofpoint-GUID: J4vhBy5WdwyS8I8f6ghFE4d9QqdS_khM X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2023-12-01_09,2023-11-30_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 spamscore=0 impostorscore=0 lowpriorityscore=0 clxscore=1015 mlxscore=0 bulkscore=0 phishscore=0 mlxlogscore=999 adultscore=0 malwarescore=0 suspectscore=0 priorityscore=1501 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2311060000 definitions=main-2312010074 X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Fri, 01 Dec 2023 03:20:00 -0800 (PST) On 30.11.23 12:28, Wen Gu wrote: > The System EID (SEID) is an internal EID that is used by the SMCv2 > software stack that has a predefined and constant value representing > the s390 physical machine that the OS is executing on. So it should > be managed by SMC stack instead of ISM driver and be consistent for > all ISMv2 device (including virtual ISM devices) on s390 architecture. > > Suggested-by: Alexandra Winter > Signed-off-by: Wen Gu > --- Reviewed-by: Alexandra Winter [...] > diff --git a/net/smc/smc_ism.c b/net/smc/smc_ism.c > index a33f861..ac88de2 100644 > --- a/net/smc/smc_ism.c > +++ b/net/smc/smc_ism.c [...] > @@ -431,14 +452,8 @@ static void smcd_register_dev(struct ism_dev *ism) > > mutex_lock(&smcd_dev_list.mutex); > if (list_empty(&smcd_dev_list.list)) { > - u8 *system_eid = NULL; > - > - system_eid = smcd->ops->get_system_eid(); > - if (smcd->ops->supports_v2()) { > + if (smcd->ops->supports_v2()) > smc_ism_v2_capable = true; > - memcpy(smc_ism_v2_system_eid, system_eid, > - SMC_MAX_EID_LEN); > - } > } Just a comment: Here we only check the first smcd device to determine whether we support v2. Which is ok, for today's platform firmware ISM devices, as they are always the same version. When you add virtual ISM devices (loopback-ism, virtio-ism) then this needs to be changed. IMO the logic then needs to be "if all smcd devices support v2, then smc_ism_v2_capable = true; else smc_ism_v2_capable = false;" I don't know if you would like to change that now in this patch, or later when you add when you add the support for loopback.