From nobody Sun Mar 20 09:09:32 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id E85831A2AB00; Sun, 20 Mar 2022 09:09:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4KLsQ46BpMz3Hjl; Sun, 20 Mar 2022 09:09:32 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1647767372; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=d9vi92hYYWa7PrvOggQmdihOCmLrGKP6CpPTfka+KT0=; b=isjGqxeZUBjixUn/wZjh8x/P/ojDaAX3ljX6Q5kMxpEucZyY8Rfka1Eh6fyli7QoTZtHJ9 K5WC9TKlGYLAbSO3KyC6XO3He9OxArseOpM660dRa6GwOR5BriiVU5T+e3JuUL0MRdXOx0 THfkS8ybkA4/aOJLZkQKFhP5ZXc8LEJuv+O9u/GfTiRbEAizmDOqtU66EVpND4v7DXyKfu eAXEhOJouByTN+7L4fJq7lqGkVzu6ACP4EgdKhZINqzMiCgGhoztWUO1TLBLZfjmBDR3bE ZUaSXm9F8AHTKwA4N+Kg+Dsto4/VCPXV5c5rncdD6du2dGSSTWWdtECcF878RQ== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id B492F2470C; Sun, 20 Mar 2022 09:09:32 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 22K99WpD029451; Sun, 20 Mar 2022 09:09:32 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 22K99WA5029450; Sun, 20 Mar 2022 09:09:32 GMT (envelope-from git) Date: Sun, 20 Mar 2022 09:09:32 GMT Message-Id: <202203200909.22K99WA5029450@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Vincenzo Maffione Subject: git: 931c75bf01d6 - stable/11 - netmap: add a tunable for the maximum number of VALE switches List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: vmaffione X-Git-Repository: src X-Git-Refname: refs/heads/stable/11 X-Git-Reftype: branch X-Git-Commit: 931c75bf01d6132e0b3baa54c052822a325c6830 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1647767372; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=d9vi92hYYWa7PrvOggQmdihOCmLrGKP6CpPTfka+KT0=; b=c7Denmom5BVaA3PGkeXz0WJ8SxWKtFh+Ym2EzFCWSyO0/a2VoKNta+JMz7DSR0uQhSLdzy qOBG3K6UDYpEKwPrud3HMPhvo+XYLln6rPqiUQnY5L3nNy2E/QD40/aoBFIMJulHp9dm6G eeH8kt0CRAwYLFl19oJ1gA5Pznx/Gbp3VRHa+Q0nQICtmmz/KebfKufFsHgY8KzlCQK9jj KbpMGTuOBG+7ccJrsVlxud1q+KZIyaXWyFjJVi14l1svkpExWhRJijtgMutg+QyoaQLNxC 8ywvsLNZlo/JeU2s3Pz1w5hWE2/bBk/G9U+1q/WJMitsFy9sOI1vl0CcMWISNg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1647767372; a=rsa-sha256; cv=none; b=P925pjAoLOFGWoyKertkC3t1aKZzW6OPG3FhurGIlco1EunJdefnHRttcNGzgzSm6vSuc0 q7NzFdYnFnnqUBjf3Xvm2h6QE5gzKmyx9BJK/cDecFwpxIpPqiXPb0beFZiJg02Y+ruAZa Sw3mkJ/ID6nOrnmR5L5mh7+iMceeYEpxon2Fa4OZ/J8Y+6WuNvu/AYWg1Q5LYL6nhPWTlA lA49qigaxSYxWuytvuiAcdn/7Xwg7HdJhYLQpXufwHzfEzlDxuZQqOefpcYt/4ToLLHPNc 4wYVv7X+U7LKiJ+YXDjRL10r5KgBMSdPJxhfF/46YJ9AnRtJso3MJHLeBc8UgA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch stable/11 has been updated by vmaffione: URL: https://cgit.FreeBSD.org/src/commit/?id=931c75bf01d6132e0b3baa54c052822a325c6830 commit 931c75bf01d6132e0b3baa54c052822a325c6830 Author: Vincenzo Maffione AuthorDate: 2022-03-06 17:28:44 +0000 Commit: Vincenzo Maffione CommitDate: 2022-03-20 09:08:36 +0000 netmap: add a tunable for the maximum number of VALE switches The new dev.netmap.max_bridges sysctl tunable can be set in loader.conf(5) to change the default maximum number of VALE switches that can be created. Current defaults is 8. MFC after: 2 weeks (cherry picked from commit dd6ab49a9aebb5fbb85973d9a859d24daa5e2165) --- share/man/man4/netmap.4 | 7 ++++++- share/man/man4/vale.4 | 5 +++-- sys/dev/netmap/netmap_bdg.c | 4 ++-- sys/dev/netmap/netmap_kern.h | 4 +++- sys/dev/netmap/netmap_vale.c | 13 +++++++++---- 5 files changed, 23 insertions(+), 10 deletions(-) diff --git a/share/man/man4/netmap.4 b/share/man/man4/netmap.4 index e6331fb9bdca..e20e7099724d 100644 --- a/share/man/man4/netmap.4 +++ b/share/man/man4/netmap.4 @@ -27,7 +27,7 @@ .\" .\" $FreeBSD$ .\" -.Dd February 6, 2020 +.Dd March 6, 2022 .Dt NETMAP 4 .Os .Sh NAME @@ -933,6 +933,11 @@ Batch size used when moving packets across a switch. Values above 64 generally guarantee good performance. +.It Va dev.netmap.max_bridges: 8 +Max number of +.Nm VALE +switches that can be created. This tunable can be specified +at loader time. .It Va dev.netmap.ptnet_vnet_hdr: 1 Allow ptnet devices to use virtio-net headers .El diff --git a/share/man/man4/vale.4 b/share/man/man4/vale.4 index 9128ce68e794..004a57a7e476 100644 --- a/share/man/man4/vale.4 +++ b/share/man/man4/vale.4 @@ -28,7 +28,7 @@ .\" $FreeBSD$ .\" $Id: $ .\" -.Dd February 6, 2020 +.Dd March 6, 2022 .Dt VALE 4 .Os .Sh NAME @@ -77,7 +77,8 @@ See for details on the API. .Ss LIMITS .Nm -currently supports up to 8 switches, with 254 ports per switch. +currently supports up to 254 ports per switch. The maximum +number of switches is provided by the max_bridges sysctl variable. .Sh SYSCTL VARIABLES See .Xr netmap 4 diff --git a/sys/dev/netmap/netmap_bdg.c b/sys/dev/netmap/netmap_bdg.c index 41faa1b5b56f..2535bd4aeaa9 100644 --- a/sys/dev/netmap/netmap_bdg.c +++ b/sys/dev/netmap/netmap_bdg.c @@ -1631,7 +1631,7 @@ netmap_init_bridges(void) #ifdef CONFIG_NET_NS return netmap_bns_register(); #else - nm_bridges = netmap_init_bridges2(NM_BRIDGES); + nm_bridges = netmap_init_bridges2(vale_max_bridges); if (nm_bridges == NULL) return ENOMEM; return 0; @@ -1644,6 +1644,6 @@ netmap_uninit_bridges(void) #ifdef CONFIG_NET_NS netmap_bns_unregister(); #else - netmap_uninit_bridges2(nm_bridges, NM_BRIDGES); + netmap_uninit_bridges2(nm_bridges, vale_max_bridges); #endif } diff --git a/sys/dev/netmap/netmap_kern.h b/sys/dev/netmap/netmap_kern.h index 0e20f6908f9e..adf26a89fb5a 100644 --- a/sys/dev/netmap/netmap_kern.h +++ b/sys/dev/netmap/netmap_kern.h @@ -1485,6 +1485,8 @@ int netmap_get_vale_na(struct nmreq_header *hdr, struct netmap_adapter **na, void *netmap_vale_create(const char *bdg_name, int *return_status); int netmap_vale_destroy(const char *bdg_name, void *auth_token); +extern unsigned int vale_max_bridges; + #else /* !WITH_VALE */ #define netmap_bdg_learning(_1, _2, _3, _4) 0 #define netmap_get_vale_na(_1, _2, _3, _4) 0 @@ -1532,7 +1534,7 @@ extern struct nm_bridge *nm_bridges; #define netmap_bns_get() #define netmap_bns_put(_1) #define netmap_bns_getbridges(b, n) \ - do { *b = nm_bridges; *n = NM_BRIDGES; } while (0) + do { *b = nm_bridges; *n = vale_max_bridges; } while (0) #endif /* Various prototypes */ diff --git a/sys/dev/netmap/netmap_vale.c b/sys/dev/netmap/netmap_vale.c index 77340acaf079..ca647fb3da92 100644 --- a/sys/dev/netmap/netmap_vale.c +++ b/sys/dev/netmap/netmap_vale.c @@ -82,10 +82,9 @@ __FBSDID("$FreeBSD$"); /* * system parameters (most of them in netmap_kern.h) - * NM_BDG_NAME prefix for switch port names, default "vale" + * NM_BDG_NAME prefix for switch port names, default "vale" * NM_BDG_MAXPORTS number of ports - * NM_BRIDGES max number of switches in the system. - * XXX should become a sysctl or tunable + * NM_BRIDGES max number of switches in the system. * * Switch ports are named valeX:Y where X is the switch name and Y * is the port. If Y matches a physical interface name, the port is @@ -113,10 +112,16 @@ __FBSDID("$FreeBSD$"); * last packet in the block may overflow the size. */ static int bridge_batch = NM_BDG_BATCH; /* bridge batch size */ + +/* Max number of vale bridges (loader tunable). */ +unsigned int vale_max_bridges = NM_BRIDGES; + SYSBEGIN(vars_vale); SYSCTL_DECL(_dev_netmap); SYSCTL_INT(_dev_netmap, OID_AUTO, bridge_batch, CTLFLAG_RW, &bridge_batch, 0, "Max batch size to be used in the bridge"); +SYSCTL_UINT(_dev_netmap, OID_AUTO, max_bridges, CTLFLAG_RDTUN, &vale_max_bridges, 0, + "Max number of vale bridges"); SYSEND; static int netmap_vale_vp_create(struct nmreq_header *hdr, struct ifnet *, @@ -362,7 +367,7 @@ netmap_vale_list(struct nmreq_header *hdr) j = req->nr_port_idx; NMG_LOCK(); - for (error = ENOENT; i < NM_BRIDGES; i++) { + for (error = ENOENT; i < vale_max_bridges; i++) { b = bridges + i; for ( ; j < NM_BDG_MAXPORTS; j++) { if (b->bdg_ports[j] == NULL)