Received: by 2002:a05:7412:b995:b0:f9:9502:5bb8 with SMTP id it21csp3084824rdb; Tue, 26 Dec 2023 15:54:13 -0800 (PST) X-Google-Smtp-Source: AGHT+IGOKY/uMaySMFofMktR+6X8AKkF+fedbE1GfPw+p+VxpoJguTsx/b9YqmK0LF3s7Asj/SiB X-Received: by 2002:a9d:6399:0:b0:6dc:3d2:3c1 with SMTP id w25-20020a9d6399000000b006dc03d203c1mr93672otk.65.1703634853479; Tue, 26 Dec 2023 15:54:13 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703634853; cv=none; d=google.com; s=arc-20160816; b=OPfiOFDYlypqaylfAnVDsYk/9l3IW0A0foT+GlSZ5OTWL0yzUzUA5WVofwJPR8Ulc8 hg/M8l38Tz+uZ7zhMQrNvppqPb0IfDrjpVa8dXvL6jXBsQVkEYjQsN+qUsvGICIRACEw dqMhvxHdyFw6NMn5Rr6Ymq/ziZFz4EaudRuPBY4FeRO4JZV0torlribPL4JVNxNvkeJh Zn+5TIH1hMDFwARybfwldPSYw/DedjIwx7i+MLb82Y+zTja5f07tpCRB10yTHXGpd3Ld xY3Xb79Z2oAeWwDwAFqbATy7UXdXxe+DoLVJvFtGFZlYOmnnY2r4mgiGikSgkXUzKctM QMHA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=HVJev5bn/SXmUllhi1fZYGckmmp1VVLDmEqAwl2Z208=; fh=UfXLpf/uxkgCQAk0qGF30P+yZcnO/fR/I4OTaiMeYb8=; b=UZLLwMfpC7I1YLV4xY5j2vM/b6RtldtysLZ13Kjr/lmy34pApjHflYXFLA71m3t5Xa luz0WsZmIPK684r7yPi8RkvMuxaETFg8p4WWpUiBvlf52/BlI3rMg3zgmCFKzWcZ+78G K1FwvI8CHO9AFl+XpVQ3THVtC2HR9gIsqgPxmtlrgrXfXL9n1UhbS4uHyAzBgcZNDpmc nYmxwl9taRxRQOM1p7Eo/68g4PmE+FvtcsBgco2CnPEzg3pxzTLeSbAhzlg8lZM9VJhR NrwxOLmvGMCVvY1BuIPvTYAaMLsz4ZKEHiUH4Xos8/XzWpyaZTShTnTh8RkFttdX88dI QKRg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b=mNlcdX7Y; spf=pass (google.com: domain of linux-nfs+bounces-818-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-nfs+bounces-818-linux.lists.archive=gmail.com@vger.kernel.org" Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id d1-20020a654241000000b005c6bdab9e58si9854394pgq.668.2023.12.26.15.54.13 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 26 Dec 2023 15:54:13 -0800 (PST) Received-SPF: pass (google.com: domain of linux-nfs+bounces-818-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=bombadil.20210309 header.b=mNlcdX7Y; spf=pass (google.com: domain of linux-nfs+bounces-818-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-nfs+bounces-818-linux.lists.archive=gmail.com@vger.kernel.org" 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 sv.mirrors.kernel.org (Postfix) with ESMTPS id E291428314D for ; Tue, 26 Dec 2023 23:54:12 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E7C8D5687; Tue, 26 Dec 2023 23:54:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="mNlcdX7Y" X-Original-To: linux-nfs@vger.kernel.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 883F31078A; Tue, 26 Dec 2023 23:54:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Sender:Reply-To:Content-ID:Content-Description; bh=HVJev5bn/SXmUllhi1fZYGckmmp1VVLDmEqAwl2Z208=; b=mNlcdX7YAzTroaH2aOGFi3ncxR 2pNVzBaD/QlHN+u5vcyz9kJsWseAFVvJP5qwfpYJojtLD66OSY2Hdxbv705YjapFqRt9x649dxUap xmXLFccJ/U2UE3uG62d1IC6SlQSIu1tJuY9x19T9yNKI970Yt1H0l4mu4krvCTfg+Gr1Tz08eFp6t ueFWeDf0saiU6NwIdKiYcLLBTGa6uGwZUvOLtTmBUJ0pxig+BimyaCGwZtR5W20M59Zkh6V4MUwsv cr5PBKj3rAuFqFBJg7EQUi1SpzJFHMo/eGV0ZbkePXXw4/quDWjgFOd2XlTiIuqIk6lw3+2eL1nss MQfvPDeg==; Received: from [50.53.46.231] (helo=[192.168.254.15]) by bombadil.infradead.org with esmtpsa (Exim 4.96 #2 (Red Hat Linux)) id 1rIHFC-00DgKc-0t; Tue, 26 Dec 2023 23:53:54 +0000 Message-ID: <12958640-e6c0-43d3-a710-48ba7873c8f5@infradead.org> Date: Tue, 26 Dec 2023 15:53:53 -0800 Precedence: bulk X-Mailing-List: linux-nfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] xprtrdma: removed unnecessary headers from verbs.c Content-Language: en-US To: Tanzir Hasan 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 References: <20231226-verbs-v1-1-3a2cecf11afd@google.com> From: Randy Dunlap In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hi Tanzir, On 12/26/23 15:35, Tanzir Hasan wrote: > On Tue, Dec 26, 2023 at 3:20 PM 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 to 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 either > including just linux/bitops.h or including both linux/bitops.h and > asm/barrier.h. > Would the second approach conform better with Rule #1? Yes, it would IMO. Where can I find your current working list of what/how to #include? Thanks. -- #Randy