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 ECF9643F76; Fri, 3 May 2024 15:15:11 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 66A85402D0; Fri, 3 May 2024 15:15:11 +0200 (CEST) Received: from fhigh3-smtp.messagingengine.com (fhigh3-smtp.messagingengine.com [103.168.172.154]) by mails.dpdk.org (Postfix) with ESMTP id E5D71402C5 for ; Fri, 3 May 2024 15:13:19 +0200 (CEST) Received: from compute3.internal (compute3.nyi.internal [10.202.2.43]) by mailfhigh.nyi.internal (Postfix) with ESMTP id 0877611401C9; Fri, 3 May 2024 09:13:19 -0400 (EDT) Received: from imap47 ([10.202.2.97]) by compute3.internal (MEProxy); Fri, 03 May 2024 09:13:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:subject:subject:to :to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s= fm3; t=1714741999; x=1714828399; bh=fr4uikQB6X2n57S+UpPEaOZN4VkZ sh0jE8tOkRne9Jw=; b=Fzz0oEZKmjaUQVzgwxwZ83WWShJDnckbMZlB0fbC/xxn rSmJ1T/BNMwTPA/79Ece6PYnlJNG9/OQBtNpHB8Kq1JIUh/fDpby5+9WKxLSbFyd 1m9IFFT3iHApPtdW6s9kR28b6U8S/jfNDdeCjmOjDja9gka3qjethdhF8csMp6at GQVDRsNHmB08lizEfgKNtr7oi0KD3ptdaGs/ThGfgC9js9NTHdho63UXdP8ahRF/ ZgJ/VTODK2uynK3Gph9IM8gRCpdzhNz9mNtWfvUlHh8K/XbiuvmmyP6zl2MPmRIi va+G98+DhPAWDWg6jKAjEytIIy2FD+S6yA0Kv7WzMA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvledrvddvtddgiedtucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepofgfggfkjghffffhvfevufgtsehttdertderredtnecuhfhrohhmpedfvfho mhculfhonhgvshdfuceothhhjhesfhhrvggvsghsugdrohhrgheqnecuggftrfgrthhtvg hrnhepveekieejjeehfeehudekueetvdfgfeegiefgueffieelffefueefjeeugfdutdeg necuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepthhhjh esfhhrvggvsghsugdrohhrgh X-ME-Proxy: Feedback-ID: ib75146ab:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id B44A0A60079; Fri, 3 May 2024 09:13:18 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.11.0-alpha0-417-gddc99d37d-fm-hotfix-20240424.001-g2c179674 MIME-Version: 1.0 Message-Id: <005a46b7-e86e-4530-80e8-955bc0cb5411@app.fastmail.com> In-Reply-To: References: <20240502135541.47142-1-thj@freebsd.org> <20240503094615.1427-1-thj@freebsd.org> Date: Fri, 03 May 2024 14:12:58 +0100 From: "Tom Jones" To: "Bruce Richardson" Cc: dev@dpdk.org Subject: Re: [PATCH] freebsd: Add support for multiple dpdk instances on FreeBSD Content-Type: text/plain X-Mailman-Approved-At: Fri, 03 May 2024 15:15:10 +0200 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 Hi Bruce, thanks for letting me know I'm not tied to anything particularly. This change isn't compatible with the previous API, but I'm not against making it so if that is really the best thing to do. As is, the dpdk changes and the contigmem changes need to come together because the API changes for getting the physical addresses. It is just the sysctl paths that differ. I'm not sure what the compatibility needs to be for DPDK, for all of my usage I have built the kernel module with the package - making API changes easy. I'm happy to follow which ever path you think is best. Sorry for the patch confusion, I'll try to keep the sequence obvious going forward. Tom On Fri, May 3, 2024, at 14:03, Bruce Richardson wrote: > On Fri, May 03, 2024 at 09:46:15AM +0000, Tom Jones wrote: >> Add support to the contigmem module on FreeBSD for multiple concurrent >> files, this enables running multiple dpdk instances with the nic_uio >> driver. >> >> Add relevant parts in dpdk to support this. >> >> Signed-off-by: Tom Jones --- > > Thanks for these patches, I hope to review and test them soon. From an > initial quick scan through, I think the changes may be easier reviewed if > split up into more than one patch. For example, could the changes to move > the "SYSCTL_INT" declarations from being in global to local scope be > separated out into one patch? Also, if the kernel module changes are > backward compatible, they could be in a separate patch to the userspace > changes to take advantage of the kernel module enhancements. > > Is such a split possible or are all the changes tightly tied together? > > /Bruce > > PS: For sending new versions of this patch, or as a patchset, please add a > version number so we can track what version is the latest one in patchwork > and on the mailing list. [Use -v flag to git format-patch] Thanks. -- - Tom