From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f44.google.com (mail-ej1-f44.google.com [209.85.218.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 54E123E2745 for ; Mon, 15 Jun 2026 11:17:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781522225; cv=none; b=P4GbU21ifRflcpwL2M7x1xJBR7LVk9V1VJY47T5iFgIvwViWd1mpro4ceGzu4ay6qfa6VinyPA2+pb5lrvFLpVoaOnS9eJ7t9MnIUnFxir/ZM8gqco6zp498Yh5M5Pqh1JjmDDH42uOMd6NuNfS2z/TmAuR11jPJci2gy8FNhOM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781522225; c=relaxed/simple; bh=d4RFhTaPxFFCmH6AlRCJafNoHTeDvIseqfsOBH9V82I=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=L+oE+hYkUtmryphhQWXGy75EgkW9/54v6cj9TONTxElvEuHfR9rex8irZ0SCdQNCPxQG8sKSoujB593cNVQTpQYotWthn8f2MPp08ddnFCFONALx2qzrZXaUpTKObIUujQgypB5cTEZ+1/or1hTf2kPEKOSb61jbe0xtKas9hLg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=LKDTr7fE; arc=none smtp.client-ip=209.85.218.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="LKDTr7fE" Received: by mail-ej1-f44.google.com with SMTP id a640c23a62f3a-bebac79fff8so391219766b.0 for ; Mon, 15 Jun 2026 04:17:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1781522223; x=1782127023; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=sga08RS68cA7NsGgPks/diWwawk9O5eve5G9GV8WMYQ=; b=LKDTr7fEWxwmvA5GG6mIW3g8D/lRx7LfQF0YMWaRO3gpE/eSSI3oxDUOpRZwTyBPYA s4h6W/P5PeEN+/69Z1OHqsQNrnoeuJCD8aDSzSgUfreJbQi9t1LN3gYP2/ps3xEFP6Va Auc8CPdrAuZYG7SE4gwjCUKPSdCg2nSH3rBIHlvHkJWPvDx/EePNjOBr4Ym9Lzgy4lpi LVT1iyXxLv4P+lXRBxZjW8VLfGJSWx3YgYMvPHIoz7XdRhmLFzYgP2zZxKiXEBHzvc/X OcLmHQwFX7435NRiJIhXj1q1noDzr2zwkOE1GkTqMYiE/VhCeI/BEBqSgr37G2IjT56K jQ6Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1781522223; x=1782127023; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=sga08RS68cA7NsGgPks/diWwawk9O5eve5G9GV8WMYQ=; b=eu9VC+W6gavsa07/NFdiqWotOr20z2oXpuv6d1uB9qhrLn02hHnA8E8z2HFy8l0MHD WnZR54RghIvJaXljoTmI16K9hydrMls7E7PIGcMcaeuSXH1otlKRA9LXBVsrKfvDbgru uxqzAOf134gY/gPRviY9CLugV80smIRY7cgwf4QXaZch+px+aiwZyCzZrQJzqeE2AwkY rQFVLCQPUW3y5HSWs8kNVGhuzMWBPFgRuhUJoJdGygA6AD98ooiTnG6sgajT+UhwrVVH bI4K5oj9ZM315lu4QdNX3ch3MLEbJWat3Y5fDfyXK2Xp/lmoX5ZvYB8YC/7IaTJy4VYl XsBw== X-Forwarded-Encrypted: i=1; AFNElJ/eann9TZZ1WFQn9f2p2yFyITqdKFpeknSjQmEcbX8z67kbUoHG3cgCkSVIEWC2LMkVt4tQz5c=@vger.kernel.org X-Gm-Message-State: AOJu0YyoNAV/KK9FXg0b4JfSRZboiHlQy22vvlWptVtOjnJBf+Bu9Gif xf1m/UrS5pNiIWCutawHyEVkCNsoziwPTVTIPtftGVaDFhMmAv7pW12t X-Gm-Gg: Acq92OFzrVB2cWvm2pnJ4hM2gb5TGyO0nEAL/T8xdDr8+sf650meqqwdYNcKpCFyJrO IvNO8ckmnt76+MuLF1Fa0Nb3veBvhNHZ1VvhnkBFQMFRwr4NBCFA5ikltSORhwOouOHfNLr7G78 aVY9Bs+/B+awqp4ymMiEVLzavbCYhGRWrkxCaNdIVqy4uQdOGOUsCfvQYNNJbBrj3/vC6WnXHVq a92hi98ltG6JdlPx7TAEywuSXLWi3ruhAfDcRawgyg90iE5fecjtiTNB5kurQPXM5+e87Xv8wS/ fV0RYcnWEYrfF+6ZoXHU1tXd0R5Wi3lZpUvmgAzd9FcuJHMakM4gTiOkNhUEALKXre4wZ9jR/G7 pnGRSMt8Th7nOcv7BkJ/T4R8aGx0E4NIvIYC85mtMmbiICOApiDZ0t1EVFJ5w2iGNwQyq0mPbwJ 16IuHzhFjNeS0tdTfdtBQy74SVxeFkhCiY42vnYkRCsnqvC3MxFAGT0iYYnyIpGaZaR7fVkIHfG CRANlFb80KAaCt8yY/n6NopgtewANH7fjY4gRvEv3TWDu2/k6gFO7ZNi1w= X-Received: by 2002:a17:907:c305:b0:bec:7661:6397 with SMTP id a640c23a62f3a-bfe2a222b62mr581935066b.29.1781522222408; Mon, 15 Jun 2026 04:17:02 -0700 (PDT) Received: from ?IPV6:2620:10d:c096:325:77fd:1068:74c8:af87? ([2620:10d:c092:600::1:2bf9]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-bfdb4b2276fsm452438866b.17.2026.06.15.04.17.01 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 15 Jun 2026 04:17:01 -0700 (PDT) Message-ID: Date: Mon, 15 Jun 2026 12:16:47 +0100 Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next v2 1/2] netdev: expose io_uring rx_page_order order via netlink To: Jakub Kicinski , Dragos Tatulea Cc: Donald Hunter , "David S. Miller" , Eric Dumazet , Paolo Abeni , Simon Horman , Andrew Lunn , Jens Axboe , Yael Chemla , Tariq Toukan , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, io-uring@vger.kernel.org References: <20260612211709.1456966-2-dtatulea@nvidia.com> <20260612211709.1456966-3-dtatulea@nvidia.com> <20260613170232.6f9e72ba@kernel.org> Content-Language: en-US From: Pavel Begunkov In-Reply-To: <20260613170232.6f9e72ba@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 6/14/26 01:02, Jakub Kicinski wrote: > On Sat, 13 Jun 2026 16:09:03 +0200 Dragos Tatulea wrote: >> On 13.06.26 11:53, Pavel Begunkov wrote: >>> On 6/12/26 22:17, Dragos Tatulea wrote: >>>> This adds observability for the io_uring zcrx rx-buf-len configuration. >>> >>> It might be nicer to look it up in the queue, e.g. rxq->mp_params, >>> and make it a queue attribute instead of zcrx specific one. In either >>> case, no objections. >> >> In io_pp_nl_fill() or in page_pool_nl_fill() as it was done in v1 for order? > > It's fine. We decided to make the "page size" a memory provider > property, now we're going back to making it a queue level param? > Like my RFC had that everyone hated so much? Sigh. TBH, I never cared much how nl would show it, so not opposing either version. My idea is that even without plumbing in per-queue non-mp size configuration, it'd be nice to have a common way to check it b/w providers. From the semantics and observability perspective, zcrx is probably not that interesting as the parameter is basically just a hint with no affect on uapi, and I'd assume people would rather see the page pool size or even the NIC's page size. But I guess it depends on what Dragos is really after with this patch. -- Pavel Begunkov