From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 23B3CA0A03; Sun, 17 Jan 2021 22:48:59 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id A04D4140D0E; Sun, 17 Jan 2021 22:48:58 +0100 (CET) Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by mails.dpdk.org (Postfix) with ESMTP id 68CDF140D0D for ; Sun, 17 Jan 2021 22:48:57 +0100 (CET) Received: from compute2.internal (compute2.nyi.internal [10.202.2.42]) by mailout.nyi.internal (Postfix) with ESMTP id 638945C0062; Sun, 17 Jan 2021 16:48:54 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute2.internal (MEProxy); Sun, 17 Jan 2021 16:48:54 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=fm3; bh= mCxU2XgFLWuB1F4Ic74ckySlyIbeWsdeAybOg00m8uE=; b=DrzUOjDx01wn1M/m KthMz1TOq70jPNNr5srFqVJ33mVeYKiRu0G70hn8ioT8tMhReVHDU3HHkoNBm6Y7 HtEkhpzOHJVnOBShTYGWUHBtwPLZOpmzTRxR6DKmwNSFHBFr3ZpaQlGidMhq7vRm 0rIDIkBvaOzgBCeXTTs8DOIwHtdAiqkmXgOSgWtu61B40tCAU1Ht3vVU13bHEwfK VXTW4lQF0G6NknZ9g621bX13evsIiVm9oa2aAbUa7/1WNN4NAcjIqfnfvsdJ1S6D TNSiV9WHw4jSNABs/qtciM/PkT0ysBR08ZPdK33vouHYWrgU+ej0AWQ8E5yaz4tz fYUCbQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=mCxU2XgFLWuB1F4Ic74ckySlyIbeWsdeAybOg00m8 uE=; b=kxOG/lA/S/swjeyHohNRX02yXccFCkG8DKRpcUGfqjqBoSJd3uK3RbtL8 qnsTMF5oVNHOiE1WacjcNXb8JT4Sy1Gc7OzCqAU99kyN0d75EjBzg3ZPXpe8deQr USZYBFVlJnQNwJhEG+wqvbWKQPLeA7z9PFhM0B1ZKKZExjQAqh0564Ef/jCKqZ6L KJISjL444fd1If7AtLEUZVhsLJJNLwngm2FGYn9p1jsP/vgnJGpNw5eGMhcXTPx9 qMiJkBN3fNQHod24FDMk7+a4/5GB1+PHcacYUtPFks7NTeoUF31zDe377wWrXb9A Nk1aoAhUSEATDBe8ePMAb9fwsGfZQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduledrtdeigdduheeiucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhephffvufffkfgjfhgggfgtsehtufertddttddvnecuhfhrohhmpefvhhhomhgr shcuofhonhhjrghlohhnuceothhhohhmrghssehmohhnjhgrlhhonhdrnhgvtheqnecugg ftrfgrthhtvghrnhepudeggfdvfeduffdtfeeglefghfeukefgfffhueejtdetuedtjeeu ieeivdffgeehnecukfhppeejjedrudefgedrvddtfedrudekgeenucevlhhushhtvghruf hiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpehthhhomhgrshesmhhonhhjrghl ohhnrdhnvght X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 8CB53108005C; Sun, 17 Jan 2021 16:48:53 -0500 (EST) From: Thomas Monjalon To: Nithin Dabilpuram Cc: Cristian Dumitrescu , jerinj@marvell.com, dev@dpdk.org Date: Sun, 17 Jan 2021 22:48:52 +0100 Message-ID: <11602414.S2x50zxl5J@thomas> In-Reply-To: <20210105085729.2814-2-ndabilpuram@marvell.com> References: <20201214102835.19980-1-ndabilpuram@marvell.com> <20210105085729.2814-1-ndabilpuram@marvell.com> <20210105085729.2814-2-ndabilpuram@marvell.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v2 2/2] bitmap: modify bitmap not empty API for 128B cacheline X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 05/01/2021 09:57, Nithin Dabilpuram: > Currently bitmap line not empty check API assumes cache line > of 64B and only checks 8 slabs. Since in 128B cacheline, we > have 16 slabs per cacheline, rte_bitmap_clear() will mark > complete line as empty as soon as 8 slabs are empty thereby > breaking bitmap scan functionality. Fix it by defining new > __rte_bitmap_line_not_empty() for 128B cacheline platform. > > Signed-off-by: Nithin Dabilpuram > Acked-by: Cristian Dumitrescu > --- > > v2: > - Fix issue with commit message and add acked-by from Cristian. It seems you don't want to consider this patch as a fix, so no backport request. Applied with title "bitmap: support 128-byte cacheline in empty check", thanks.