Received: by 2002:a05:7412:b10a:b0:f3:1519:9f41 with SMTP id az10csp2445239rdb; Sun, 3 Dec 2023 18:01:25 -0800 (PST) X-Google-Smtp-Source: AGHT+IGr0zTXemOHAQP+X8U1GLcyiW1ZoFYWRFCvryFkvplE9mbhASL0xSrd/+WfoEUx90o3IVfi X-Received: by 2002:a05:6358:914d:b0:170:2838:e843 with SMTP id r13-20020a056358914d00b001702838e843mr145726rwr.64.1701655285142; Sun, 03 Dec 2023 18:01:25 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701655285; cv=none; d=google.com; s=arc-20160816; b=FQbP7U0DfsKKn5B/RsCdZaG7QZh1qKCDN+Q7wYzM4dUK5TlbnJhYhpb+D6ei3qPHAB CSkUL1kOFNvCJg3NaGQ4QjpVijqlfmJgWoa15LsabH1Lg1kOPD2jQyKbT2g20sJBuVWF /ccoH6Vmj4VuqqhmbwrMppSL55B4LPLvHfGz1493HuQvPVs1IC3fLJ3Pbq0fDOweb9ik U8L8clonDgL2jXVaPrvqmAA/vliTCLL2Eo7GO7d+8A/KVMVOu0yDwSuT621XkugeDeNS KLfodQ7T3QXTqSZpz24v3P0qomK8J5HZnj1b+LFl9ixIrCjN2NfGIjIUvII6Em7abCVS RCdA== 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 :references:cc:to:subject:user-agent:mime-version:date:message-id; bh=ysHojecPHbajxQGy44B/X7vNWnYm9jIIdHcRqyKRMzo=; fh=sEMXM8EIRon7mOSJlyzssyc8ZI4OrKolZNyYtAw3uRE=; b=s4k5pp3ki4UTHx/y14o+JJt9aXpRqWEGV75zvwzwDglKVFFn7yVbgOA9mtN5qlH+b7 yFiAkDS8ao9uT/mY4H6AhtpQchQECQU7QCBJW5WxKl/HzuYPs9hIIqbuy9xXDG21KYH1 l7Ns2zVqM8kEmFQ3hO4Vsok78lI6YjXK6pJVHRqtns6BPSj0kGYmHKv3ryYOixXWagIy 9Vct5R44TS8qi9J0awCO6nOiKnNcPmCvkDHkEAjeHHBOLwFiVA8g0Bli1Yu/syjTp82B T1i4zTmpDzVV08IOm7Pf30KWmvvHNwQmK7uRHfBwjr2g0r5OVkFZc9pOpMGXtom9CoSv b/yA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=alibaba.com Return-Path: Received: from snail.vger.email (snail.vger.email. [2620:137:e000::3:7]) by mx.google.com with ESMTPS id i23-20020a632217000000b005b8ee1c0c68si3093589pgi.605.2023.12.03.18.01.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 03 Dec 2023 18:01:25 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=alibaba.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by snail.vger.email (Postfix) with ESMTP id 454118097898; Sun, 3 Dec 2023 18:01:23 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at snail.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229918AbjLDCBN (ORCPT + 99 others); Sun, 3 Dec 2023 21:01:13 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44024 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234264AbjLDCBM (ORCPT ); Sun, 3 Dec 2023 21:01:12 -0500 Received: from out30-111.freemail.mail.aliyun.com (out30-111.freemail.mail.aliyun.com [115.124.30.111]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A12E7FE; Sun, 3 Dec 2023 18:01:17 -0800 (PST) X-Alimail-AntiSpam: AC=PASS;BC=-1|-1;BR=01201311R111e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=ay29a033018046049;MF=guwen@linux.alibaba.com;NM=1;PH=DS;RN=20;SR=0;TI=SMTPD_---0Vxh9rK-_1701655273; Received: from 30.221.130.147(mailfrom:guwen@linux.alibaba.com fp:SMTPD_---0Vxh9rK-_1701655273) by smtp.aliyun-inc.com; Mon, 04 Dec 2023 10:01:15 +0800 Message-ID: Date: Mon, 4 Dec 2023 10:01:09 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.15.1 Subject: Re: [PATCH net-next v3 0/7] net/smc: implement SMCv2.1 virtual ISM device support To: Alexandra Winter , 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> From: Wen Gu In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-11.7 required=5.0 tests=BAYES_00, ENV_AND_HDR_SPF_MATCH,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE,UNPARSEABLE_RELAY,USER_IN_DEF_SPF_WL 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 X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Sun, 03 Dec 2023 18:01:23 -0800 (PST) On 2023/12/2 00:32, Alexandra Winter wrote: > > > On 30.11.23 12:28, Wen Gu wrote: >> The fourth edition of SMCv2 adds the SMC version 2.1 feature updates for >> SMC-Dv2 with virtual ISM. Virtual ISM are created and supported mainly by >> OS or hypervisor software, comparable to IBM ISM which is based on platform >> firmware or hardware. >> >> With the introduction of virtual ISM, SMCv2.1 makes some updates: >> >> - Introduce feature bitmask to indicate supplemental features. >> - Reserve a range of CHIDs for virtual ISM. >> - Support extended GIDs (128 bits) in CLC handshake. >> >> So this patch set aims to implement these updates in Linux kernel. And it >> acts as the first part of SMC-D virtual ISM extension & loopback-ism [1]. >> >> [1] https://lore.kernel.org/netdev/1695568613-125057-1-git-send-email-guwen@linux.alibaba.com/ >> >> v3->v2: >> - Rename smc_clc_fill_fce as smc_clc_fill_fce_v2x; >> - Remove ISM_IDENT_MASK from drivers/s390/net/ism.h; >> - Add explicitly assigning 'false' to ism_v2_capable in ism_dev_init(); >> - Remove smc_ism_set_v2_capable() helper for now, and introduce it in >> later loopback-ism implementation; >> >> v2->v1: >> - Fix sparse complaint; >> - Rebase to the latest net-next; >> >> Wen Gu (7): >> net/smc: Rename some variable 'fce' to 'fce_v2x' for clarity >> net/smc: support SMCv2.x supplemental features negotiation >> net/smc: introduce virtual ISM device support feature >> net/smc: define a reserved CHID range for virtual ISM devices >> net/smc: compatible with 128-bits extend GID of virtual ISM device >> net/smc: disable SEID on non-s390 archs where virtual ISM may be used >> net/smc: manage system EID in SMC stack instead of ISM driver >> > > Wen Gu, > as you can see in [1] your patches 5/7 and 1/7 still have formatting issues. > In this case they need to pass > > scripts/checkpatch.pl --strict --max-line-length=80 > > (see linux/Documentation/process/coding-style.rst) > > [1] https://patchwork.kernel.org/project/netdevbpf/list/?series=&submitter=&state=&q=net%2Fsmc&archive=&delegate= Thanks. They will be fixed and I will check the patches with tests provided by https://github.com/kuba-moo/nipa/tree/master/tests/patch