From nobody Fri May 15 19:42:20 2026 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 4gHHf84my8z6dt65 for ; Fri, 15 May 2026 19:42:24 +0000 (UTC) (envelope-from oliver.pntr@gmail.com) Received: from mail-yx1-xb134.google.com (mail-yx1-xb134.google.com [IPv6:2607:f8b0:4864:20::b134]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "smtp.gmail.com", Issuer "WR4" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4gHHf82gbNz41xG for ; Fri, 15 May 2026 19:42:24 +0000 (UTC) (envelope-from oliver.pntr@gmail.com) Authentication-Results: mx1.freebsd.org; none Received: by mail-yx1-xb134.google.com with SMTP id 956f58d0204a3-65dd9b25829so308403d50.3 for ; Fri, 15 May 2026 12:42:24 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1778874142; cv=none; d=google.com; s=arc-20240605; b=JHTbnSTdpwjCTT+AdjDuvg82lqBXH5UOAJIaLEPJQ6kIxEaRKG98OfhFWiO7p0+oHG pVFZgnJWLVmzNIigxwDd50vxz5OJz5U4F3zQLafWXzBfinft+Lm8V/NlQUMk9YpXSmST ZsPkFI0h0bjUcRe/1I3XaRDm56ebiAVwusPtedQ94D/2xYkajzW2LiSuI3r3yAC9Gxwr NV3iSHf4zuKl2X6lkhMtYDbnWa7cMDLkUFqH7gwWv3K5Ir4WEtD7AjtBPWZgppw00+Z0 GefhjxT/ebkOw3CiNkKFiezC0jc1s49e7a0dg4XXCC+FtlWNSma2X/3phYZrIJsNMmO/ eFZw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20240605; h=cc:to:subject:message-id:date:from:references:in-reply-to :mime-version:dkim-signature; bh=WGhiDaaL4v1Gsdz2nsFKiiqvR0PXQwS1OTLPeVdENjA=; fh=hBlj4sgAQWIVVA2MKtP0oEPiUEaCK5IGYZLcTLvGclE=; b=iM2mHtdnOu8j8NGYpPV7zQSp9XGsDW2QI3I8uopc61kMqZceiVVyp+2R46tqW/PETd +G2Qb7m8VoehsdhsEZ8gbfgvfk7xMIa/xpuaB8NbIp5lr5QRSZrxS8V+ycfbcEKQGncz JTQ1A1GRF92XsOeUeeu4DjrDRNxfokVBECprwboAoCHjblah7hm6dETBlhkB8tZtwY9T TOzdmnIL9On21nWQN3Yh3FYPFjno677Dlzwdi2uCUycJxE/DGolY7EjYpVgtK0cyQJ+L A9mnG81NmbLrhXKyDtGJG1W5ythHIv/PWv32ymeLwf6uoyKcMK9wX0fa43JGX8cWTthp 5hSg==; darn=freebsd.org ARC-Authentication-Results: i=1; mx.google.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1778874142; x=1779478942; darn=freebsd.org; h=cc:to:subject:message-id:date:from:references:in-reply-to :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=WGhiDaaL4v1Gsdz2nsFKiiqvR0PXQwS1OTLPeVdENjA=; b=X6Xq5N3A+6BOgt7I1KsAQLQLqQqOayF5M1clyRHpndqvqk0f8zNzjknyPl3jwWqmwt cRCPlDt+DNHTFS2vF10q7L8SPgwXeTGJFByLOfm0JQsu0Xz2/o0UesULuJOO8jA8HBVW A8/IS2O541kBk7C0LjUlLE9i+4fomWP0g3BgftM4TOcCKr4HDAe6ltqo+rBa1CTlLgIZ HioIsUPK6zZy3yI+u1J9QwAfpCeeh05pz2vjupX0avbjofSt4wGTStGN9jDmLBB4rO+g H6NQD8MUYcAWIMWaJx/EZJT0Vu9oDyXqZ8OGbc0vzKB2dQJhaFwYVkDfU3Zl/gbDfXx8 MFuQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1778874142; x=1779478942; h=cc:to:subject:message-id:date:from:references:in-reply-to :mime-version:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=WGhiDaaL4v1Gsdz2nsFKiiqvR0PXQwS1OTLPeVdENjA=; b=fxuYJja5WmR2vGjWBvdMExaEsWtqOhMAI3qPVl+bLNIKnIeHktlhcwJanvW8lHNxOy XQgSazWIevDt2OyH5ZglzdzHlWMpmeotL0eHtFeHhtqkfvHvElJ57WRT1lUgF/nEUch9 8bUzKoSv5HysZZ9p2+NMoPLeOAh5kFcXaU4I4JslAJZpqHCfvWYLtsvJurtzXIA/GXH0 jAUtU7z/7zLwRiHMufWgeEI41UKKtQ5Uiq+alisNnNhfSQ7mWx3Md9P+fWcRPsMJpQBj MfsVIChYSyz5V2++eY57By36vMsygbCj4aCm9iaLxNicHvSfXUgGMqueLg6lTKwdKeAr Ihpg== X-Forwarded-Encrypted: i=1; AFNElJ/K30+rj9AdEd2QS0VSFTaEWhmQz79xgJOFKjdu9s4xJFT/ltJ0+PsM93OfSAfVLr9s8p5t7MXoFhHYYxDEc7vmPOt8@freebsd.org X-Gm-Message-State: AOJu0Yx5UBjG2M4yQ5GXf8ADeUGgOP37y86q8aP/fv7hqcz6XnK6s1/Y k4rTtLp+Yiw3ZgWo0gYEsyB2ltxSeY9l6oqRAq5exCVHVg8osHjJGaLd2eBxIuQLI5zuS47g9er cyPO0di2U0682jFmxj8JKuQrkafGhuPA= X-Gm-Gg: Acq92OELTZVxBGSlXkbY+LTYFb352TPFVU+Ns3aAaD/gNUttAh5958jz0omLH3Mmfu5 RdRBBr4WFTN+ai4TM50pnktPDSY2YGZ+lCXvLiF25IZbpbMXPz7n6FYy7ghHeBdBETbY5ibXD4R zTqih7JkDbN0mF6J893AHdsvXLxpMvBIYneLH1OUcui6f9HYryoyWtiu/pngBR29MAnHuJHVRIj plFO6dtY0ujeQAOzyeMDz1AaeBjziQYtm5H6xHStXLdx1t00SdTMmzPhzhsOT5R5ApKHlvouLZG 2ZQwf6mGDCKNBu0s14DGrFyW7ZCLMl5+nQAAmsWtYBH5FdbrthU= X-Received: by 2002:a05:690e:1402:b0:65d:6f10:d42a with SMTP id 956f58d0204a3-65e227cfff6mr5395860d50.39.1778874142364; Fri, 15 May 2026 12:42:22 -0700 (PDT) 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: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Received: by 2002:a05:7011:c9a:b0:514:c31c:db89 with HTTP; Fri, 15 May 2026 12:42:20 -0700 (PDT) In-Reply-To: <6a064e15.44875.4253434b@gitrepo.freebsd.org> References: <6a064e15.44875.4253434b@gitrepo.freebsd.org> From: Oliver Pinter Date: Fri, 15 May 2026 20:42:20 +0100 X-Gm-Features: AVHnY4KHqD2ECw2L6wgkbEZgWe3CrVi5X3X0ChNljK0_PYUDWiepeylSSfHkyGA Message-ID: Subject: Re: git: b92b9da33006 - main - nfsd: Allow vfs.nfsd.srvmaxio to be up to 4Mbytes To: Rick Macklem Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Type: multipart/alternative; boundary="0000000000005b16ab0651e0699c" X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[]; TAGGED_FROM(0.00)[]; ASN(0.00)[asn:15169, ipnet:2607:f8b0::/32, country:US] X-Rspamd-Queue-Id: 4gHHf82gbNz41xG X-Spamd-Bar: ---- X-Rspamd-Pre-Result: action=no action; module=replies; Message is reply to one we originated --0000000000005b16ab0651e0699c Content-Type: text/plain; charset="UTF-8" Hi! On Friday, May 15, 2026, Rick Macklem wrote: > The branch main has been updated by rmacklem: > > URL: https://cgit.FreeBSD.org/src/commit/?id= > b92b9da3300655c86dcd42ea8a5ba45badd90847 > > commit b92b9da3300655c86dcd42ea8a5ba45badd90847 > Author: Rick Macklem > AuthorDate: 2026-05-14 22:33:23 +0000 > Commit: Rick Macklem > CommitDate: 2026-05-14 22:33:23 +0000 > > nfsd: Allow vfs.nfsd.srvmaxio to be up to 4Mbytes > > Without this patch, the maximum setting for > vfs.nfsd.srvmaxio was 1Mbyte. This patch increases > that to 4Mbytes. > > The same as for any setting above 128Kbytes, settings up to > 4Mbytes require that kern.ipc.maxsockbuf be increased. > (A message generated after setting vfs.nfsd.srvmaxio via > the /etc/rc.conf variable nfs_server_maxio will indicate > the minimum setting, which will be somewhat greater than > four times the setting of vfs.nfsd.srvmaxio.) > > Requested by: Cedric Blancher > MFC after: 2 weeks > Fixes: 13d3bd165e22 ("subr_uio.c: Remove a KASSERT() for large NFS > server I/O") Wouldn't be better to introduce an MFC with tag, rater than abusing the Fixes tag? This seems to me like a new "feature", rather than a bug. And it will may be added to releasenotes also. > --- > sys/fs/nfs/nfsproto.h | 3 +++ > sys/fs/nfsserver/nfs_nfsdport.c | 5 +++-- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/sys/fs/nfs/nfsproto.h b/sys/fs/nfs/nfsproto.h > index 6e67c5be3416..1054c85d8e8f 100644 > --- a/sys/fs/nfs/nfsproto.h > +++ b/sys/fs/nfs/nfsproto.h > @@ -89,8 +89,11 @@ > * It used to be called NFS_MAXDATA, but has been renamed to clarify that > * it refers to server side only and doesn't conflict with the NFS_MAXDATA > * defined in rpcsvc/nfs_prot.h for userland. > + * NFS_SRVMAXIO is the default setting for vfs.nfsd.srvmaxio > + * NFS_SRVLIMITIO is the maximum setting allowed for vfs.nfsd.srvmaxio > */ > #define NFS_SRVMAXIO (128 * 1024) > +#define NFS_SRVLIMITIO (4 * 1024 * 1024) > > /* Stat numbers for rpc returns (version 2, 3 and 4) */ > /* > diff --git a/sys/fs/nfsserver/nfs_nfsdport.c b/sys/fs/nfsserver/nfs_ > nfsdport.c > index e42d5ca1e0a3..1e6936ed351c 100644 > --- a/sys/fs/nfsserver/nfs_nfsdport.c > +++ b/sys/fs/nfsserver/nfs_nfsdport.c > @@ -241,8 +241,9 @@ sysctl_srvmaxio(SYSCTL_HANDLER_ARGS) > printf("nfsd: vfs.nfsd.srvmaxio can only be increased\n"); > return (EINVAL); > } > - if (newsrvmaxio > 1048576) { > - printf("nfsd: vfs.nfsd.srvmaxio cannot be > 1Mbyte\n"); > + if (newsrvmaxio > NFS_SRVLIMITIO) { > + printf("nfsd: vfs.nfsd.srvmaxio cannot be > %d\n", > + NFS_SRVLIMITIO); > return (EINVAL); > } > if ((newsrvmaxio & (newsrvmaxio - 1)) != 0) { > > --0000000000005b16ab0651e0699c Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi!

On Friday, May 15, 2026, Rick Macklem <rmacklem@freebsd.org> wrote:
The branch main has been updated by rmacklem:

URL: https://cgit.FreeBSD.org/src/commit/?id=3Db92b9da3300655c86dcd42ea8a5ba45badd90847

commit b92b9da3300655c86dcd42ea8a5ba45badd90847
Author:=C2=A0 =C2=A0 =C2=A0Rick Macklem <rmacklem@FreeBSD.org>
AuthorDate: 2026-05-14 22:33:23 +0000
Commit:=C2=A0 =C2=A0 =C2=A0Rick Macklem <rmacklem@FreeBSD.org>
CommitDate: 2026-05-14 22:33:23 +0000

=C2=A0 =C2=A0 nfsd: Allow vfs.nfsd.srvmaxio to be up to 4Mbytes

=C2=A0 =C2=A0 Without this patch, the maximum setting for
=C2=A0 =C2=A0 vfs.nfsd.srvmaxio was 1Mbyte.=C2=A0 This patch increases
=C2=A0 =C2=A0 that to 4Mbytes.

=C2=A0 =C2=A0 The same as for any setting above 128Kbytes, settings up to =C2=A0 =C2=A0 4Mbytes require that kern.ipc.maxsockbuf be increased.
=C2=A0 =C2=A0 (A message generated after setting vfs.nfsd.srvmaxio via
=C2=A0 =C2=A0 the /etc/rc.conf variable nfs_server_maxio will indicate
=C2=A0 =C2=A0 the minimum setting, which will be somewhat greater than
=C2=A0 =C2=A0 four times the setting of vfs.nfsd.srvmaxio.)

=C2=A0 =C2=A0 Requested by:=C2=A0 =C2=A0Cedric Blancher <cedric.blancher@gmail.com>
=C2=A0 =C2=A0 MFC after:=C2=A0 =C2=A0 =C2=A0 2 weeks
=C2=A0 =C2=A0 Fixes:=C2=A0 13d3bd165e22 ("subr_uio.c: Remove a KASSERT= () for large NFS server I/O")

Wouldn&#= 39;t be better to introduce an MFC with tag, rater than abusing the Fixes t= ag? This seems to me like a new "feature", rather than a bug. And= it will may be added to releasenotes also.
=C2=A0
---
=C2=A0sys/fs/nfs/nfsproto.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| 3 +++=
=C2=A0sys/fs/nfsserver/nfs_nfsdport.c | 5 +++--
=C2=A02 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/sys/fs/nfs/nfsproto.h b/sys/fs/nfs/nfsproto.h
index 6e67c5be3416..1054c85d8e8f 100644
--- a/sys/fs/nfs/nfsproto.h
+++ b/sys/fs/nfs/nfsproto.h
@@ -89,8 +89,11 @@
=C2=A0 * It used to be called NFS_MAXDATA, but has been renamed to clarify = that
=C2=A0 * it refers to server side only and doesn't conflict with the NF= S_MAXDATA
=C2=A0 * defined in rpcsvc/nfs_prot.h for userland.
+ * NFS_SRVMAXIO is the default setting for vfs.nfsd.srvmaxio
+ * NFS_SRVLIMITIO is the maximum setting allowed for vfs.nfsd.srvmaxio
=C2=A0 */
=C2=A0#define=C2=A0 =C2=A0 =C2=A0 =C2=A0 NFS_SRVMAXIO=C2=A0 =C2=A0 (128 * 1= 024)
+#define=C2=A0 =C2=A0 =C2=A0 =C2=A0 NFS_SRVLIMITIO=C2=A0 (4 * 1024 * 1024)<= br>
=C2=A0/* Stat numbers for rpc returns (version 2, 3 and 4) */
=C2=A0/*
diff --git a/sys/fs/nfsserver/nfs_nfsdport.c b/sys/fs/nfsserver/nfs_nfsdport.c
index e42d5ca1e0a3..1e6936ed351c 100644
--- a/sys/fs/nfsserver/nfs_nfsdport.c
+++ b/sys/fs/nfsserver/nfs_nfsdport.c
@@ -241,8 +241,9 @@ sysctl_srvmaxio(SYSCTL_HANDLER_ARGS)
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 printf("nfsd: = vfs.nfsd.srvmaxio can only be increased\n");
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 return (EINVAL); =C2=A0 =C2=A0 =C2=A0 =C2=A0 }
-=C2=A0 =C2=A0 =C2=A0 =C2=A0if (newsrvmaxio > 1048576) {
-=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0printf("nfsd: = vfs.nfsd.srvmaxio cannot be > 1Mbyte\n");
+=C2=A0 =C2=A0 =C2=A0 =C2=A0if (newsrvmaxio > NFS_SRVLIMITIO) {
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0printf("nfsd: = vfs.nfsd.srvmaxio cannot be > %d\n",
+=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0NFS_S= RVLIMITIO);
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 return (EINVAL); =C2=A0 =C2=A0 =C2=A0 =C2=A0 }
=C2=A0 =C2=A0 =C2=A0 =C2=A0 if ((newsrvmaxio & (newsrvmaxio - 1)) !=3D = 0) {

--0000000000005b16ab0651e0699c--