From patchwork Tue Jan 23 22:17:45 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mina Almasry X-Patchwork-Id: 191230 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:2553:b0:103:945f:af90 with SMTP id p19csp640963dyi; Tue, 23 Jan 2024 14:30:44 -0800 (PST) X-Google-Smtp-Source: AGHT+IE2+8/W6UcAPU6YVNV9OZEezkjssMAYxUgr5kCb+Rk1QQS2esvPU6TuEXhzGZWtuXUDjTH+ X-Received: by 2002:a05:6a20:144b:b0:19c:5c8b:e779 with SMTP id a11-20020a056a20144b00b0019c5c8be779mr1480320pzi.84.1706049044512; Tue, 23 Jan 2024 14:30:44 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706049044; cv=pass; d=google.com; s=arc-20160816; b=ehnutVbsxqEFCoX63/LFJQOMU+fGe6hlC5Z8EQwUDRI5FjXhF5FfXIXCcM0voBW1S1 DzC6E4Ruw7dSJowZ4xClnqjr+yafMP5Ivb7PVtN/2XsB/SKsV2Zaexo/yoPyREr9ihRM SCKr1nrvREgjcTe7NnIcxBnFiFWbKrUoLSgtxijAi/B7yzayrefQtCkCfelOcz5SF7iu RKgE4h/VbArTwd1+3JQGMocgBU/uC+5ToplxwCeJP9+8Xt/WXmbk/B2xQpkNrnj4IT1J uPUk3uov75bFjOsidEt2WQdGnfoO5N0tki80R+c8XwJ38Z+OGForZZ/KcCVaMUkeJEFQ qh9w== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=cc:to:from:subject:message-id:references:mime-version :list-unsubscribe:list-subscribe:list-id:precedence:in-reply-to:date :dkim-signature; bh=sq2A5rdPoZzXe3JfsyjEi2DmgNcsMleYf8+7wJXU3oU=; fh=3zG4NHuhPxuhQ6Shw7Nt9BiRVbWd7elghbVYpfpS9LI=; b=wbkhzlpv+Jab6zk74Juv+5bKpsMblxrZpPXqQT/CCUl8h1dbE3JFHmXxKO/8YnyCN9 MeYXV2EBW+Purzq6gRLlIZhSREsKNeBvonddSJ+khUkemBryFDxBKFpMRqZEVjWXzwPb aQB5k7An21yQLCEva6QVOP5p+dE6dOHkE1m2VboLTghy3yqP4boWfUSLHB472cXb7ASc 0wb67KnpQy9w9tvg+9GZwtzX229uKW20JH2Tmk0FtjWTkPbpCTA4Z01kyqKQab3nkyy0 dg2mhB+GRvR2LAHMg/L0oSLkNCcimF3SrAoW0vmF5WE6thh+D5TUIwltkmJ7TydMSpEK Go7A== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@google.com header.s=20230601 header.b=bvAyWCgT; arc=pass (i=1 spf=pass spfdomain=flex--almasrymina.bounces.google.com dkim=pass dkdomain=google.com dmarc=pass fromdomain=google.com); spf=pass (google.com: domain of linux-kernel+bounces-36125-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-36125-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [2604:1380:40f1:3f00::1]) by mx.google.com with ESMTPS id j10-20020a170902690a00b001d453cffb92si10152888plk.620.2024.01.23.14.30.44 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Jan 2024 14:30:44 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-36125-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@google.com header.s=20230601 header.b=bvAyWCgT; arc=pass (i=1 spf=pass spfdomain=flex--almasrymina.bounces.google.com dkim=pass dkdomain=google.com dmarc=pass fromdomain=google.com); spf=pass (google.com: domain of linux-kernel+bounces-36125-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-36125-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=google.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 6423EB2A0E0 for ; Tue, 23 Jan 2024 22:18:52 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id DB1A050249; Tue, 23 Jan 2024 22:18:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="bvAyWCgT" Received: from mail-yb1-f202.google.com (mail-yb1-f202.google.com [209.85.219.202]) (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 361BF4F216 for ; Tue, 23 Jan 2024 22:17:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.202 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706048278; cv=none; b=F8dlxw+Iddv4TyCScTfMEXvM1N/0b8CdSBb0gEMp22VieQ2+bV+n2tyXsJHpzFzSdZbbTE6sIatxmYTbM2fcZxzgqiETXvdz2kOe8XGK2GCDzA2YP0Pc5lxmZrUcp8kFSssBXaMrExhEN5vqiIBf8nrcq+e7EsI3975oqp+Y8cQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706048278; c=relaxed/simple; bh=eQc/0XprESJCX8/SL8NWCcerd2+39PRmvZ4jwBgHHms=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=kcrzFSp2ONcdzHNZ4h3LXGyEJC6sgOb1CGMGpejlhdlgTZaIh+80MhTwmPkZAp3zaq2HGuuPihxGBPxAhiljNbm3igtB8O7eYlDFY3ppK3mSq3KrkXM6V/sHIHIlz64nnYZ8Dtsih74WkaTv95xSz+V998R7yz53v24j9eSXZcg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--almasrymina.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=bvAyWCgT; arc=none smtp.client-ip=209.85.219.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--almasrymina.bounces.google.com Received: by mail-yb1-f202.google.com with SMTP id 3f1490d57ef6-dc236791689so6612378276.3 for ; Tue, 23 Jan 2024 14:17:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1706048276; x=1706653076; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=sq2A5rdPoZzXe3JfsyjEi2DmgNcsMleYf8+7wJXU3oU=; b=bvAyWCgTdB7xqhlCHl4FccHzz0TK4HuT69XojP7Cihu+xUqrUNsjr6aNdFRlc0lsA6 oVP+JJe+KMh22It+JjF8vj3488kwKDj6QU20MRaBCLEZ1i8xL+HzBcyLfACMM/GAmzkc KTaARTv6Pgz+NXkYt8/JkL35I0bDegpG5Eug7Ry8RDHnbw34yfpkPqaeKOO7bph7hcCb Hz3IzdfhsiEoI7zRO/kpcKycv9QKpLW7Gqcl/msLocu02G6gQCaYgUYPyWAzNu5qRZUj 2Z7BEswmWXCYGl6FBdDwUgEIgTT8CGLjVoEGurlb5lE3NpvaAlXYiFfM8/E8QngSuMFR shoQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1706048276; x=1706653076; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=sq2A5rdPoZzXe3JfsyjEi2DmgNcsMleYf8+7wJXU3oU=; b=TDYwVqrGrPxeit3rtiGRwvDk8aPUV1uhvs+/2nqwaQtEet3VPTh7p/B5gXs5p2NBjT pvMc05IuGm8CdcS2I8i22ZQt2iE+tjcxlxuB7EDII2ksrIBPZ00SF17su1SYv0lPbtWH KMpjXQ9v1rbjDDWj3qN0p4B9vCc6/YS16PM8ZxJ+bmUE3j29U3wJ6ml0bTBHoryUgkea pof30gG2WSSIjKP/owAhdYfcPtL4nK/226FNdcMM4RNa9lroU7ygzYy2O5pvkeHalLWL 0bjrVhVnEXNxe20Tk25oZSjxtVQsoUqc23IfBjLgSCmgjoeUcOd1uDENLYY8sEA+Rrfz DYcg== X-Gm-Message-State: AOJu0YzXcWD3mTA6jcMeRC1WMzxYlJyt0UMTM590FbsD9vpcoxKcB25U vNX0sqklAfT6fTf/GtS//sG0Ui7ZZ1pS2Y+YKNhwXKvv4FHTw+QYZW+IJG+5IYH82XnT5HiV7ae IGzYxyxmGegQpm+elv6QUIKo8oE9/TcaYmDytiTwgOMYW7C8YS46Ruu29pT3sorQi5RmXvjepTJ QQNWr+VxPUpW81FwcPwnCnigeRWKxwYvxGG5WACnCfqooaGTfrcTI9g+hTFQ8qe4BM60o= X-Received: from almasrymina.svl.corp.google.com ([2620:15c:2c4:200:3608:aa16:52d:3594]) (user=almasrymina job=sendgmr) by 2002:a05:6902:86:b0:dbe:a214:888c with SMTP id h6-20020a056902008600b00dbea214888cmr3189553ybs.11.1706048276193; Tue, 23 Jan 2024 14:17:56 -0800 (PST) Date: Tue, 23 Jan 2024 14:17:45 -0800 In-Reply-To: <20240123221749.793069-1-almasrymina@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240123221749.793069-1-almasrymina@google.com> X-Mailer: git-send-email 2.43.0.429.g432eaa2c6b-goog Message-ID: <20240123221749.793069-2-almasrymina@google.com> Subject: [PATCH net-next v6 1/2] net: introduce abstraction for network memory From: Mina Almasry To: linux-kernel@vger.kernel.org, netdev@vger.kernel.org Cc: Mina Almasry , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Jason Gunthorpe , " =?utf-8?q?Christian_K=C3=B6nig?= " , Shakeel Butt , Yunsheng Lin , Willem de Bruijn X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1788922082931937443 X-GMAIL-MSGID: 1788922082931937443 Add the netmem_ref type, an abstraction for network memory. To add support for new memory types to the net stack, we must first abstract the current memory type. Currently parts of the net stack use struct page directly: - page_pool - drivers - skb_frag_t Originally the plan was to reuse struct page* for the new memory types, and to set the LSB on the page* to indicate it's not really a page. However, for compiler type checking we need to introduce a new type. netmem_ref is introduced to abstract the underlying memory type. Currently it's a no-op abstraction that is always a struct page underneath. In parallel there is an undergoing effort to add support for devmem to the net stack: https://lore.kernel.org/netdev/20231208005250.2910004-1-almasrymina@google.com/ netmem_ref can be pointers to different underlying memory types, and the low bits are set to indicate the memory type. Helpers are provided to convert netmem pointers to the underlying memory type (currently only struct page). In the devmem series helpers are provided so that calling code can use netmem without worrying about the underlying memory type unless absolutely necessary. Reviewed-by: Shakeel Butt Signed-off-by: Mina Almasry --- v6: - Applied Reviewed-by from Shakeel. rfc v5: - RFC due to merge window. - Change to 'typedef unsigned long __bitwise netmem_ref;' - Fixed commit message (Shakeel). - Did not apply Shakeel's reviewed-by since the code changed significantly. v4: - use 'struct netmem;' instead of 'typedef void *__bitwise netmem_ref;' Using __bitwise with a non-integer type was wrong and triggered many patchwork bot errors/warnings. Using an integer type causes the compiler to warn when casting NULL to the integer type. Attempt to use an empty struct for our opaque network memory. v3: - Modify struct netmem from a union of struct page + new types to an opaque netmem_ref type. I went with: +typedef void *__bitwise netmem_ref; rather than this that Jakub recommended: +typedef unsigned long __bitwise netmem_ref; Because with the latter the compiler issues warnings to cast NULL to netmem_ref. I hope that's ok. - Add some function docs. v2: - Use container_of instead of a type cast (David). --- include/net/netmem.h | 41 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 41 insertions(+) create mode 100644 include/net/netmem.h diff --git a/include/net/netmem.h b/include/net/netmem.h new file mode 100644 index 000000000000..9f327d964782 --- /dev/null +++ b/include/net/netmem.h @@ -0,0 +1,41 @@ +/* SPDX-License-Identifier: GPL-2.0 + * + * Network memory + * + * Author: Mina Almasry + */ + +#ifndef _NET_NETMEM_H +#define _NET_NETMEM_H + +/** + * netmem_ref - a nonexistent type marking a reference to generic network + * memory. + * + * A netmem_ref currently is always a reference to a struct page. This + * abstraction is introduced so support for new memory types can be added. + * + * Use the supplied helpers to obtain the underlying memory pointer and fields. + */ +typedef unsigned long __bitwise netmem_ref; + +/* This conversion fails (returns NULL) if the netmem_ref is not struct page + * backed. + * + * Currently struct page is the only possible netmem, and this helper never + * fails. + */ +static inline struct page *netmem_to_page(netmem_ref netmem) +{ + return (__force struct page *)netmem; +} + +/* Converting from page to netmem is always safe, because a page can always be + * a netmem. + */ +static inline netmem_ref page_to_netmem(struct page *page) +{ + return (__force netmem_ref)page; +} + +#endif /* _NET_NETMEM_H */