Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp3078783rdb; Tue, 26 Dec 2023 15:35:28 -0800 (PST) X-Google-Smtp-Source: AGHT+IG/YpiDVR8WgAvayxeNRip33oZoEl/rSxUVfnnwEXxoWiGnagyN2oR+Wvj+F/wEOZ2qoY70 X-Received: by 2002:a05:6870:414e:b0:203:b29b:bdd4 with SMTP id r14-20020a056870414e00b00203b29bbdd4mr7151173oad.45.1703633728386; Tue, 26 Dec 2023 15:35:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703633728; cv=none; d=google.com; s=arc-20160816; b=XlFKHX8TkXn4UArvCryPG4Jy5Z6y00TQB5o9MxZJXMiBlcCA4lCmr04zKNOR8v7YfO LsCGG+MUSIPv/Jx8/OnaqGPit7zwZCC+ct/alieeCU36uyM7jdCp55G5634kgB/WOQ/l bJJElK+yfz3jikdUP0BAXcnWSPoVwn1/yBjOUKHF735f8287o9m9G/GwBOBg+aNQWFv9 hjOskAbSszpymNJpzei9xLugPTeoZDE7VjxWQyzWHQaVlhSpEevHYBy+UUl/WTzy3oAw eOMo3xYnjQ7djGDoEZ4jZ1M0sw4FqIP8xfNS4B6gUUfKNrjI3yDZR0FC0mrUXvzESAuf IBCg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:dkim-signature; bh=eZ+VyiPmwTPUzp7gonixlKN6VYglweSWOvX0B9fYuA0=; fh=yIq4MnqvYLF1YcnZmLkvVHLJ/nkrJtZNHwbvM4PrN3U=; b=hx2JRdnJMw5UBaiAlopIPqucDBloEF+xlJJxONtToGDpjqWJMTydmweJmCkgCT4Le0 C49vurI/CzJUXOErsty286JFZL4H34FUOYw5fhJeKM5Fcxdx5oeabhkKfMfh4twdkK2k nppeyhWRdGb7I26E3e/oAMkuYkGPKWRNHjsBwunSVbVDbKNYSxwCwTQtI1sWT+UtWCNI JBbP1uS8ih/mAAVKos3Uq9WBSE+FQrLEK2wgqL59laIipBq3OzNuhKXZSAB9nLh0hgT3 Wk/CmkoiNEyB9ya9nNWjf+PvHzVLSL5Ybsc+yop6F6sFvG9eC9d+mmMQmWKYyOSs40tx UoSQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@google.com header.s=20230601 header.b="Pig/ZVLX"; spf=pass (google.com: domain of linux-nfs+bounces-817-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-nfs+bounces-817-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id i26-20020a05620a0a1a00b007815562ee39si2187515qka.24.2023.12.26.15.35.28 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 26 Dec 2023 15:35:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-nfs+bounces-817-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20230601 header.b="Pig/ZVLX"; spf=pass (google.com: domain of linux-nfs+bounces-817-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-nfs+bounces-817-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 094A61C209D9 for ; Tue, 26 Dec 2023 23:35:28 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 9114B568A; Tue, 26 Dec 2023 23:35:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="Pig/ZVLX" X-Original-To: linux-nfs@vger.kernel.org Received: from mail-qt1-f174.google.com (mail-qt1-f174.google.com [209.85.160.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 34B2E5662 for ; Tue, 26 Dec 2023 23:35:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Received: by mail-qt1-f174.google.com with SMTP id d75a77b69052e-425928c24easo1089261cf.0 for ; Tue, 26 Dec 2023 15:35:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1703633724; x=1704238524; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=eZ+VyiPmwTPUzp7gonixlKN6VYglweSWOvX0B9fYuA0=; b=Pig/ZVLXD93Dxf0FQZVmWS4c3zfEJHp22y6aq3hJK6TJPCZpE3s5DIjYp1KMwVQWNA mzJ1x8DSPQ0e3fh+Yqr26fOlFTA0L0zOrJRz+aTvyybKMtOf2hMbn6foBaeN2BgRvl+n daQgvBS9hydoSM9+jxcY5SoKtMhQx5ShHkbVxKKQeKVUNKUKe2eH08G/RsdfWUDZos1u 98BZ9tiwy6fUEE20czQ/J1PCbQAv8Ebm0VlJ8aOTqe4O5EnpUry6Di5EsTty1wkaPVOx Ar1hX961iQlPaGEmPUQCeIKCLRJ7v/ZPOkxBptOOl3nMNiP0xiJTurmsgf1/dT2fJgrn 7rLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703633724; x=1704238524; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=eZ+VyiPmwTPUzp7gonixlKN6VYglweSWOvX0B9fYuA0=; b=U1y6O1eGBEjj+L2akZgDTUWRvo9dq08bMb5xMeRVMrRxVn0Ca5Zpw1i/54rTHTibAw ENIc50PP3lMJa4ZOj3uGqBwoI2vY3QfhVm5GTkG/rhjecN/n+96M2AERFiXklsOOnSWv ob+M7++OvgO52ZhtgFtB5auUhuSmY8HNnacNMVZP0KTBiHUFtrpybozy1DwXUEA64xH1 pdapr20VRWU92Y9t19EmhQC7S6h9ET0E2wxvJkWAMILef2H7UMeRDq7ZClDwZtGC9teJ +IOLQsnnQy10sFstLByI10DDnc7iOJ6rGc0vrh0d+5oBDSgMRkGlDBKaJujGAgTv9xM7 xP8A== X-Gm-Message-State: AOJu0YwgycYpOORHeRfJ9QTqjad/+FKdbFnUTc9WxNwKJm5Q9iqmC4XX O6PzEiTViqV6/QRsAN30BCMCsGiOEPODW/XQ0duMz1/Lrm+D X-Received: by 2002:ac8:590b:0:b0:427:e836:2549 with SMTP id 11-20020ac8590b000000b00427e8362549mr101794qty.29.1703633723983; Tue, 26 Dec 2023 15:35:23 -0800 (PST) Precedence: bulk X-Mailing-List: linux-nfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 References: <20231226-verbs-v1-1-3a2cecf11afd@google.com> In-Reply-To: From: Tanzir Hasan Date: Tue, 26 Dec 2023 15:35:11 -0800 Message-ID: Subject: Re: [PATCH] xprtrdma: removed unnecessary headers from verbs.c To: Randy Dunlap Cc: Chuck Lever , Jeff Layton , Neil Brown , Olga Kornievskaia , Dai Ngo , Tom Talpey , Trond Myklebust , Anna Schumaker , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , linux-nfs@vger.kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Nick Desaulniers , Al Viro Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Dec 26, 2023 at 3:20=E2=80=AFPM Randy Dunlap wrote: > > Hi, > > On 12/26/23 13:23, Tanzir Hasan wrote: > > asm-generic/barrier.h and asm/bitops.h are already brought into the > > header and the file can still be built with their removal. > > Brought into which header? Hi Randy, Sorry for the poor explanation. I see that I left out the specific header. The inclusion of linux/sunrpc/svc_rdma.h brings in linux/sunrpc/rpc_rdma.h This brings in linux/bitops.h which is preferred over asm/bitops.h > Does this conflict with Rule #1 in Documentation/process/submit-checklist= .rst ? Yes, this conflicts with Rule #1. A better version of this patch would be t= o add linux/bitops.h to this file directly. The main reason this patch exists is to clear out the asm-generic file since those are not preferred. I can do this by ei= ther including just linux/bitops.h or including both linux/bitops.h and asm/barrier.h. Would the second approach conform better with Rule #1? Thanks, Tanzir