From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32124C4361B for ; Thu, 10 Dec 2020 19:59:20 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id B064F22D5A for ; Thu, 10 Dec 2020 19:59:19 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org B064F22D5A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=2phO9dwFig9Oyph5o454cQfc3c4D8vkArXd5U3zO13w=; b=UF1JLtR4zKhUq2D+UVy6hXIZc FVz2hzV9gUTJkksfYf/5J63pkJbJKa23OVJe0qbL0Rfq77XY4+YtU3H16infToXOMsgWsmMZoxfzg kwMgZ1l2qfBqWFekt0N77zZZdy2pRVINJ6Yh3L09u0EEhsCqjlQizE7e3d1u4lISQRGH3cjwBF40O bnhX74skO5RNwcIVAA4sTfFX5EtVA61k0e/BNKAgN/mtUkbRcTlYy/L4DD1YKSrEDU09zgMSYbrHX iw+7ZiUK8mRGe+umD2/CC/u3nui7XWSoUggcXYwT6qEzVy3Nbq7mcgJK5kyiqwWyCDUAFUOvTEbb5 +NX5rvD/g==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1knS5j-0001Bd-Uk; Thu, 10 Dec 2020 19:59:08 +0000 Received: from mail.kernel.org ([198.145.29.99]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1knS5f-00019O-2t; Thu, 10 Dec 2020 19:59:03 +0000 Date: Thu, 10 Dec 2020 20:58:55 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1607630342; bh=s2+iLf6qBKVf58ffYZN9l2zof1I9JLW9V6wDx6uBGB8=; h=From:To:Cc:Subject:References:In-Reply-To:From; b=eklBHcybGU7NaHOQQv++7WWXgTVIXfU4CnRYhDTvdmMh828hU1HhwGuy262A7mXl3 UVfMlgSR9hjmxiHEEmyAKdwE8N24GronEa9YX/3xByG12lhH5sLlYjOzWtARATmbU+ gbjs//eYTsEVpjwIHW5m8/FwAch5WCZ79eUruF3gO37OfRXISugUzkUxi/tczLL5H1 C81E6xkV7hUWEVha0XOmiWm7bl+CmN06kwTj6Wpcq+BBF8+02d6kP4fL05muxPEfXb pZ3KGXhlBNtOeEDWP9EZLwn/4dPMZS2smZLYs42jw9BUNf2bClh/cggGRtfKBSWABt n3IVdl/kI4rwg== From: Wolfram Sang To: Heiner Kallweit Subject: Re: [PATCH] dma-mapping: move hint unlikely for dma_mapping_error from drivers to core Message-ID: <20201210195855.GA11120@kunai> Mail-Followup-To: Wolfram Sang , Heiner Kallweit , George Cherian , Herbert Xu , "David S. Miller" , Zaibo Xu , Boris Brezillon , Arnaud Ebalard , Srujana Challa , Matthias Brugger , Giovanni Cabiddu , Elie Morisse , Nehal Shah , Shyam Sundar S K , Mike Marciniszyn , Dennis Dalessandro , Doug Ledford , Jason Gunthorpe , Andreas Larsson , Jakub Kicinski , Netanel Belgazal , Arthur Kiyanovski , Guy Tzalik , Saeed Bishara , Zorik Machulsky , Iyappan Subramanian , Keyur Chudgar , Quan Nguyen , Igor Russkikh , Jay Cliburn , Chris Snook , Ariel Elior , Sudarsana Kalluru , GR-everest-linux-l2@marvell.com, Michael Chan , Raju Rangoju , Madalin Bucur , Ioana Ciornei , Ioana Radulescu , Claudiu Manoil , Catherine Sullivan , Sagi Shahar , Jon Olson , Yisen Zhuang , Salil Mehta , Hauke Mehrtens , Sebastian Hesselbarth , Thomas Petazzoni , Sunil Goutham , Geetha sowjanya , Subbaraya Sundeep , hariprasad , Felix Fietkau , John Crispin , Sean Wang , Mark Lee , Tariq Toukan , Saeed Mahameed , Leon Romanovsky , Alexei Starovoitov , Daniel Borkmann , Jesper Dangaard Brouer , John Fastabend , Boris Pismenny , Jon Mason , Rain River , Zhu Yanjun , Shannon Nelson , Pensando Drivers , Jiri Pirko , Edward Cree , Martin Habets , Daniele Venzano , Kunihiko Hayashi , Wingman Kwok , Murali Karicheri , Kevin Brace , Radhey Shyam Pandey , Michal Simek , Kalle Valo , Jiri Slaby , Nick Kossifidis , Luis Chamberlain , Larry Finger , Luca Coelho , Lorenzo Bianconi , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, qat-linux@intel.com, linux-i2c@vger.kernel.org, linux-rdma@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org, ath10k@lists.infradead.org, linux-wireless@vger.kernel.org, ath11k@lists.infradead.org, wil6210@qti.qualcomm.com, b43-dev@lists.infradead.org, iommu@lists.linux-foundation.org References: <5d08af46-5897-b827-dcfb-181d869c8f71@gmail.com> MIME-Version: 1.0 In-Reply-To: <5d08af46-5897-b827-dcfb-181d869c8f71@gmail.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20201210_145903_341994_20B9B539 X-CRM114-Status: GOOD ( 13.62 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Kevin Brace , Shyam Sundar S K , Igor Russkikh , Iyappan Subramanian , GR-everest-linux-l2@marvell.com, Ioana Radulescu , Madalin Bucur , Michal Simek , Doug Ledford , Murali Karicheri , Mark Lee , Jiri Pirko , Jesper Dangaard Brouer , Hauke Mehrtens , Arnaud Ebalard , Lorenzo Bianconi , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org, Guy Tzalik , Luis Chamberlain , linux-crypto@vger.kernel.org, Jon Mason , Shannon Nelson , Kunihiko Hayashi , Chris Snook , Daniele Venzano , Zhu Yanjun , linux-i2c@vger.kernel.org, Ioana Ciornei , Jon Olson , Arthur Kiyanovski , Leon Romanovsky , "David S. Miller" , Boris Pismenny , John Fastabend , Rain River , Martin Habets , Yisen Zhuang , Wingman Kwok , Sean Wang , Claudiu Manoil , Pensando Drivers , Kalle Valo , Boris Brezillon , Sagi Shahar , Tariq Toukan , Sudarsana Kalluru , Netanel Belgazal , Andreas Larsson , Catherine Sullivan , Zaibo Xu , Mike Marciniszyn , Alexei Starovoitov , Elie Morisse , Luca Coelho , Quan Nguyen , Subbaraya Sundeep , Jiri Slaby , Daniel Borkmann , Dennis Dalessandro , linux-rdma@vger.kernel.org, Srujana Challa , ath10k@lists.infradead.org, linux-arm-kernel@lists.infradead.org, ath11k@lists.infradead.org, b43-dev@lists.infradead.org, Jay Cliburn , Radhey Shyam Pandey , John Crispin , Michael Chan , Salil Mehta , Edward Cree , iommu@lists.linux-foundation.org, wil6210@qti.qualcomm.com, Saeed Bishara , bpf@vger.kernel.org, Geetha sowjanya , Giovanni Cabiddu , Thomas Petazzoni , Herbert Xu , Zorik Machulsky , qat-linux@intel.com, Jason Gunthorpe , Nick Kossifidis , Jakub Kicinski , Nehal Shah , Sunil Goutham , Sebastian Hesselbarth , Ariel Elior , Keyur Chudgar , George Cherian , linux-mediatek@lists.infradead.org, Matthias Brugger , netdev@vger.kernel.org, hariprasad , Raju Rangoju , Felix Fietkau , Saeed Mahameed , Larry Finger Content-Type: multipart/mixed; boundary="===============2170758860630855186==" Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org --===============2170758860630855186== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="k+w/mQv8wyuph6w0" Content-Disposition: inline --k+w/mQv8wyuph6w0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Dec 10, 2020 at 03:47:50PM +0100, Heiner Kallweit wrote: > Zillions of drivers use the unlikely() hint when checking the result of > dma_mapping_error(). This is an inline function anyway, so we can move > the hint into the function and remove it from drivers. > From time to time discussions pop up how effective unlikely() is, > and that it should be used only if something is really very unlikely. > I think that's the case here. >=20 > Patch was created with some help from coccinelle. >=20 > @@ > expression dev, dma_addr; > @@ >=20 > - unlikely(dma_mapping_error(dev, dma_addr)) > + dma_mapping_error(dev, dma_addr) >=20 > Signed-off-by: Heiner Kallweit Acked-by: Wolfram Sang # for I2C --k+w/mQv8wyuph6w0 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAl/SffoACgkQFA3kzBSg KbYcjxAAgOE4gHcgEP8+Oex1fposdP2Z4KiWFjIYYWG4fo/Ry9PjDbSGh9Nptht2 fnsCRcFXFj4oaSXaflBTq6ky4usgo2Gyp9puXbnpyj7P2uEjrqZs1zUFpAWdzMor UgiJkW/P2IZjCDfwxE8nn9L0fm8ZfcHWqVohAgDh/9SKsrQCdzlzwvd7vSQ94fXr qnYrmc6BF68dxVZx4TV18GddP5qFXYKytQ8pXL51XZEJTI05IGmc2l6hs/B4tKj6 muxiEFw5Ac0eseMimi4J5YDJJZxWe28onn69mMJYQDzVPqSZRyhSAqCv0EhMg6Vp sABbG/eShtxir8A5ZrVRgqCaVyBjPu6pHAxdccHkj4d/6hfvD6F2FDXXaWirAf3i A4gsMJAmxtBYV0Lyx0D+fzCFnvUSDDSOEayRJdzotQXVCbLvuWHTp6EXVJFD5mMU /o3LApTC1uYQTXfGh1HHanpSEXLXfVgzjuDHRUsVIwemk5JwUAl6fw4oXbMrHYdZ v9Inx4U81LGxayz1vGmzbE39AeE7YH/5lH4metjot96RpKa+Gg++mMxUHBPW6Jam AOz6I3cKYsn7mPkzAZfDNvhvfgz2vxXcGGULSCdaWnVCJY7FMqe8i98w1z/Ymo7U JSXmDUhFS43r0JqUfbR0sRGgLL+kHTEa6I4ZT8UNd9DmTtS0Jmo= =4X1P -----END PGP SIGNATURE----- --k+w/mQv8wyuph6w0-- --===============2170758860630855186== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek --===============2170758860630855186==--