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 A007C41BAE; Thu, 2 Feb 2023 16:56:42 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 83D7C42D20; Thu, 2 Feb 2023 16:56:42 +0100 (CET) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id 1182340EDC for ; Thu, 2 Feb 2023 16:56:41 +0100 (CET) Received: by linux.microsoft.com (Postfix, from userid 1086) id 5D8D720B74F7; Thu, 2 Feb 2023 07:56:40 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 5D8D720B74F7 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1675353400; bh=IbW1XQrZMNppFuwJm8K9ZwFLFFR3uEXV3Lm0JN9XS0Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=V6KIeUk3YsCD1j1804QoWYPKiOTdxj8mYQcOfeLXxHuTxYkRUF1QmWhUh/mSUZLvO MKnV9sXnKORFroCA8N6YCplWLFpJBDHQwdfnGW7CsLkluPaasbd4cAcYBd87acg0Xw vugPnDD8xQlyOZlI/Xkx2qH8k/kLja5VjswkPsPQ= Date: Thu, 2 Feb 2023 07:56:40 -0800 From: Tyler Retzlaff To: David Marchand Cc: dev@dpdk.org, thomas@monjalon.net, mb@smartsharesystems.com, bruce.richardson@intel.com, ferruh.yigit@amd.com Subject: Re: [PATCH v6 0/2] eal: provide leading and trailing zero bit count abstraction Message-ID: <20230202155640.GA20115@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> References: <1669241687-18810-1-git-send-email-roretzla@linux.microsoft.com> <1673380001-16558-1-git-send-email-roretzla@linux.microsoft.com> <20230120221418.GA9231@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) 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 On Thu, Feb 02, 2023 at 10:14:41AM +0100, David Marchand wrote: > Hello Tyler, > > On Fri, Jan 20, 2023 at 11:14 PM Tyler Retzlaff > wrote: > > > > hi folks, > > > > i think this one can probably be merged? > > > > Series-acked-by: Morten Brørup > > Series-acked-by: Bruce Richardson > > patch 1/2 Acked-by: Ferruh Yigit > > I like the cleanup of rte_common.h and additional unit tests, but the > MSVC bits don't belong here. > Please move them in your MSVC enablement series hm, the way i'm approaching this is to keep specific features together similar to if i added new platform functionality for threads i add windows or linux in the same series. similarly, in this case i'm adding msvc and gcc in the same series. the msvc enablement series introduces the changes for the build system so it's about enabling a compiler not adding functionality to dpdk so this change/API really aren't related to the other, but i agree the other series is part of converging on the platform, toolchain combination being enabled overall. reconsider? if not i guess i'll just withdraw the actual API for now and have to resubmit and review later since i don't think it belongs mixed in with the compiler enablement. > https://patchwork.dpdk.org/project/dpdk/list/?series=26662&state=%2A&archive=both. if the msvc series were just merged, i think the above discussion would be moot no? anyway, i'll abide by whatever decision you go with. thanks!