Received: by 10.192.165.148 with SMTP id m20csp3691053imm; Mon, 30 Apr 2018 04:53:31 -0700 (PDT) X-Google-Smtp-Source: AB8JxZpXnpGHMLZzK3PynW4NJQRiZgTJML3Qm3/zpOYn+2MlfljCCdrd9gX48qUKCKsG6KlmR+Pk X-Received: by 2002:a65:5d0f:: with SMTP id e15-v6mr10070932pgr.119.1525089211486; Mon, 30 Apr 2018 04:53:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1525089211; cv=none; d=google.com; s=arc-20160816; b=G073bbUEMumrRJGjCFxB1z5powdFX/MyCjl1Ds9G1PnnRyYRGDjm2kOcGa6F6b3Ucj PTkKNInFGot9JSYg24eh+ECNOuhvIorg5qwgmolLHIHPxRmFYAVFFnddxbTlWkN+BwFE VF6vmy1h8tVh8LJOH7NchOfiBOmdI+dlx0jyHUar7rukvyJsNBej6Smo+m2PEDFVhxLs j4nXhpuG4m81Efi5SR4u+N0pPIERNF4nQHt5h9++PpnTfujLBSYIAnqbm8t1blRX28OI +BzsHdRrYP+3zokXAUddQSIpDN3gA8Wh9Nb99uEaXsjB0gcKGrv617sA5mL9GdHJB3kM U6wA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:to:references:message-id :content-transfer-encoding:cc:date:in-reply-to:from:subject :mime-version:dkim-signature:arc-authentication-results; bh=ThYN6a8KQnJdL/F+ToRtvdpiYdIa7mUt8n5Ui/LbjYE=; b=r8xjGE345Qp8hGlfuN+1fw5ss595WZi/bQahPArkp4URvKqHeTTAS0i76ore0fg3/N HnuRG0yQW39wNLqvVABt9sg+8gebnVPMF+JBcERVXRwP4g3BzYd0A9hTKoq68s4e/fPL AI/XnMcFLRnxnGBnEfMvnwOM8eXfs+/54dL1vmvqBuopwZIMVYCgWWF1XzA3J2PgAYTq THOnzTMc2jrjPXmGLC6R5VBE8HNtcie0xyYjSR6/IlOlyCH6ga1zgE901Sld1FogdeZX l3LqlyGeB6WuCPhx4Ces+PHvAyY8dGd51z0G3rsoit5EFpb2Vm5XiXEIH49EtVmOw+tF /wdA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2017-10-26 header.b=s/7+e6mx; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l18si7271008pfe.299.2018.04.30.04.53.17; Mon, 30 Apr 2018 04:53:31 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2017-10-26 header.b=s/7+e6mx; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754248AbeD3Lv0 (ORCPT + 99 others); Mon, 30 Apr 2018 07:51:26 -0400 Received: from aserp2120.oracle.com ([141.146.126.78]:37050 "EHLO aserp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753414AbeD3LvX (ORCPT ); Mon, 30 Apr 2018 07:51:23 -0400 Received: from pps.filterd (aserp2120.oracle.com [127.0.0.1]) by aserp2120.oracle.com (8.16.0.22/8.16.0.22) with SMTP id w3UBkat8010329; Mon, 30 Apr 2018 11:51:04 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=content-type : mime-version : subject : from : in-reply-to : date : cc : content-transfer-encoding : message-id : references : to; s=corp-2017-10-26; bh=ThYN6a8KQnJdL/F+ToRtvdpiYdIa7mUt8n5Ui/LbjYE=; b=s/7+e6mxbSmeejRBufTzcxHrV/2xPyhsZaZrhz4h6u76fTICNzeaumcZPEGyQKcQmZm+ 5M3ivou9Smfav26oHVAsngfkicXwZZn5dD4fIwa7ZIP+3voQBLONY626h5Y33FKYlcFt sqD9S1Ikc6qBd/e5olKXQSI+IWmaj8RVFS0Yqp3fZGkvlg3IXP2QkGuoHlhlBgi1/3M3 fqhXqpf/Np8FWtFs6SHUIlJ16t7Rpg4qjwRYkHmuJV6wp+oYQtpABaSf2aDjGGJ/wDPn 6XqXVbz467hjDyKousGQaiZWvTTOPkheFhoLoGol4me3ILaCUmfgJWt/EwdXmeIk52rz fw== Received: from userv0022.oracle.com (userv0022.oracle.com [156.151.31.74]) by aserp2120.oracle.com with ESMTP id 2hmgxfkqh6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 30 Apr 2018 11:51:03 +0000 Received: from aserv0121.oracle.com (aserv0121.oracle.com [141.146.126.235]) by userv0022.oracle.com (8.14.4/8.14.4) with ESMTP id w3UBp2nu027698 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Mon, 30 Apr 2018 11:51:02 GMT Received: from abhmp0004.oracle.com (abhmp0004.oracle.com [141.146.116.10]) by aserv0121.oracle.com (8.14.4/8.13.8) with ESMTP id w3UBp1WN022038; Mon, 30 Apr 2018 11:51:01 GMT Received: from dhcp-10-172-157-170.no.oracle.com (/10.172.157.170) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Mon, 30 Apr 2018 04:51:01 -0700 Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 11.3 \(3445.6.18\)) Subject: Re: [PATCH] IB/core: Make ib_mad_client_id atomic From: =?utf-8?Q?H=C3=A5kon_Bugge?= In-Reply-To: <1524856116.11756.85.camel@redhat.com> Date: Mon, 30 Apr 2018 13:50:57 +0200 Cc: jackm , Jason Gunthorpe , Don Hiatt , Ira Weiny , Sean Hefty , OFED mailing list , linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Message-Id: References: <20180418142450.15581-1-Haakon.Bugge@oracle.com> <1524196555.11756.30.camel@redhat.com> <20180420153451.GF30433@ziepe.ca> <50E3FE4A-2C6E-4493-92E1-383C349FAE3A@oracle.com> <20180423221618.0000795d@dev.mellanox.co.il> <20180426213250.0000515d@dev.mellanox.co.il> <9fdd3ec4-ee91-5442-e753-25d2ecd27ea9@xsintricity.com> <1524856116.11756.85.camel@redhat.com> To: Doug Ledford X-Mailer: Apple Mail (2.3445.6.18) X-Proofpoint-Virus-Version: vendor=nai engine=5900 definitions=8878 signatures=668698 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=3 malwarescore=0 phishscore=0 bulkscore=0 spamscore=0 mlxscore=0 mlxlogscore=965 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1711220000 definitions=main-1804300115 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > On 27 Apr 2018, at 21:08, Doug Ledford wrote: >=20 > On Thu, 2018-04-26 at 20:51 +0200, H=C3=A5kon Bugge wrote: >>> Jason is out this week. I'll end up processing this one (probably = later >>> today). But I=E2=80=99ll fix up the commit message to suit my = tastes when I do. >>=20 >> Thank you, Doug and Jack, >=20 > I reworded the commit message, let me know if you think I worded it > wrong: >=20 > commit 69f01b81539c62f3dd96f9f02138ad7b839a0c70 (HEAD -> = k.o/wip/dl-for-rc) > Author: H=C3=A5kon Bugge > Date: Wed Apr 18 16:24:50 2018 +0200 >=20 > IB/core: Make ib_mad_client_id atomic >=20 > Currently, kernel protects access to the agent ID allocator on a = per > port basis using a spinlock, so it is impossible for two = apps/threads on > the same port to get the same TID, but it is entirely possible for = two > threads on different ports to end up with the same TID. As this = can be > confusing (regardless of it being legal according to the IB Spec = 1.3, > C13-18.1.1, in section 13.4.6.4 - TransactionID usage: "Then = initiating > a new operation, MADHeader:TransactionID shall be set to such a = value > that within that MAD the combination of TIG, SGID, and MgmtClass is > different from that of any other currently executing operation. If = the > MAD does not have a GRH, its SLID is used in the combination in = place of > an SGID." which guarantees we are legal because our different ports = will > have different SGID/SLID creating a unique tuple even if the TIDs = are > identical), and as we might want to open the TID allocator up to = more > parallel usage later, make the TID an atomic type so that no two > allocations, regardless of port number, will be the same. The wording is OK per se, but the last sentence spans 12 lines - and as = such - a little bit hard to comprehend. I suggest just to refer to IB = Spec 1.3, clause C13-18.1.1 and not quote it and make the last = sub-sentence (starting with =E2=80=9Cand as we might want to open=E2=80=9D= ) a sentence by its own. Thxs, H=C3=A5kon >=20 > Signed-off-by: H=C3=A5kon Bugge > Reviewed-by: Jack Morgenstein > Reviewed-by: Ira Weiny > Reviewed-by: Zhu Yanjun > Signed-off-by: Doug Ledford >=20 > --=20 > Doug Ledford > GPG KeyID: B826A3330E572FDD > Key fingerprint =3D AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 = 2FDD