From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 5FEE837CD33 for ; Sat, 10 Oct 2026 06:11:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.14 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791612668; cv=none; b=O6h5cQSNPD7KMm4pIb38iLgTo9LcmMMr8bLvefv9+/7/x8geynKvHDHKCupLnecRb5PJitUtSNI2xZuDykyDMBwFdE2Bflzri9wMUhJO0b184NgYCg4strmcT+mkSV2m6AcWlx0tSwfC7xQNMgvlB0THWtnlnqgUBf3aKNBTHzs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1791612668; c=relaxed/simple; bh=wpblAJc6Odww8wRDHBiwoFtswZS0nDjQzcO+QlkNqKI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Yi7y5qzEyns/GPe/loM/wW9CDGnZGkP4Sn1LpR6NMISTywQ+OO+V/LAzhEnEx00peQ0pQbWOi7Ot0ur9ZhRFRbLBU4rnMc5vp93+yG8xbTm5M1TChlHy1roiW9FAz0+Y4YIQpnyPfgQ4sLLzhqC3I0qISu0Gr/VtlBDCN1IeyBo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=VDIwbMOz; arc=none smtp.client-ip=198.175.65.14 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="VDIwbMOz" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1791612666; x=1823148666; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=wpblAJc6Odww8wRDHBiwoFtswZS0nDjQzcO+QlkNqKI=; b=VDIwbMOzFsrCTwbpiPJGV1HAWjgnUR39qvLaVo4AZJUR3fwY7sbay1/t x88z+OW9sbnzoELPka7GW5+HThVEClC5XRvzTblcBkSBctZgphMeQUklG ETjSk8dTe6z162B3KQEAkjSXjBCchrEU8Xi0EFwV+TqTYHo/UFHHbLS8H RM1++bxnjjvTcH86IyGhH0PG+r3/+Y2Lh2m5ZVBhbdSGS22f6UARJrhH6 oCZKCAnDi9RzR5BFf7O0kAHWIWq28p0sEYmie3EyftNFvwBsnzoePrd31 mi6o7HnFAMd6FguXApcGPT/tMSJ5IihpPbWBqvM8MJWBIcs6vH2iwfuG+ g==; X-CSE-ConnectionGUID: ynM3Y8HZTDKrRkXjgPRr+g== X-CSE-MsgGUID: 5Lc2i/BNRCWxpJFpbdhcVQ== X-IronPort-AV: E=McAfee;i="6800,10657,11930"; a="285481" X-IronPort-AV: E=Sophos;i="6.27,149,1787036400"; d="scan'208";a="285481" Received: from orviesa008.jf.intel.com ([10.64.159.148]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 09 Oct 2026 23:11:05 -0700 X-CSE-ConnectionGUID: X++7SABlT7a8Xa0owoSNaQ== X-CSE-MsgGUID: VbcRJbsOQ/Sj9WhAAZxrjQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.27,149,1787036400"; d="scan'208";a="408167" Received: from lkp-server01.sh.intel.com (HELO 0caa9d2d175c) ([10.239.97.150]) by orviesa008.jf.intel.com with ESMTP; 09 Oct 2026 23:11:03 -0700 Received: from kbuild by 0caa9d2d175c with local (Exim 4.98.2) (envelope-from ) id 1xFQIN-000000006Yl-44fb; Sat, 10 Oct 2026 06:10:59 +0000 Date: Sat, 10 Oct 2026 14:10:41 +0800 From: kernel test robot To: Chuck Lever , NeilBrown , Jeff Layton , Olga Kornievskaia , Dai Ngo , Tom Talpey Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev, linux-nfs@vger.kernel.org, syzbot+9c7b9c5f6dd1d470b605@syzkaller.appspotmail.com Subject: Re: [PATCH v1] NFSD: Don't start nfsd threads from a pool_threads write Message-ID: <202610101401.uliqHtVY-lkp@intel.com> References: <20261009144017.1284586-1-cel@kernel.org> Precedence: bulk X-Mailing-List: linux-nfs@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20261009144017.1284586-1-cel@kernel.org> Hi Chuck, kernel test robot noticed the following build errors: [auto build test ERROR on brauner-vfs/vfs.all] [also build test ERROR on linus/master v7.3-rc6 next-20261008] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Chuck-Lever/NFSD-Don-t-start-nfsd-threads-from-a-pool_threads-write/20261009-104017 base: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git vfs.all patch link: https://lore.kernel.org/r/20261009144017.1284586-1-cel%40kernel.org patch subject: [PATCH v1] NFSD: Don't start nfsd threads from a pool_threads write config: loongarch-defconfig (https://download.01.org/0day-ci/archive/20261010/202610101401.uliqHtVY-lkp@intel.com/config) compiler: clang version 24.0.0git (https://github.com/llvm/llvm-project 242243a5814307de825db3428662d67b064e2cbc) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261010/202610101401.uliqHtVY-lkp@intel.com/reproduce) If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot | Closes: https://lore.kernel.org/oe-kbuild-all/202610101401.uliqHtVY-lkp@intel.com/ All errors (new ones prefixed by >>): >> fs/nfsd/nfsctl.c:504:41: error: use of undeclared identifier 'nn' 504 | if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { | ^~ >> fs/nfsd/nfsctl.c:504:41: error: use of undeclared identifier 'nn' 504 | if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { | ^~ >> fs/nfsd/nfsctl.c:504:41: error: use of undeclared identifier 'nn' 504 | if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { | ^~ >> fs/nfsd/nfsctl.c:504:41: error: use of undeclared identifier 'nn' 504 | if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { | ^~ >> fs/nfsd/nfsctl.c:504:41: error: use of undeclared identifier 'nn' 504 | if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { | ^~ 5 errors generated. vim +/nn +504 fs/nfsd/nfsctl.c 449 450 /* 451 * write_pool_threads - Set or report the configured number of threads per pool 452 * 453 * Input: 454 * buf: ignored 455 * size: zero 456 * 457 * OR 458 * 459 * Input: 460 * buf: C string containing whitespace- 461 * separated unsigned integer values 462 * representing the number of NFSD 463 * threads to start in each pool 464 * size: non-zero length of C string in @buf 465 * Output: 466 * On success: passed-in buffer filled with '\n'-terminated C 467 * string containing integer values representing the 468 * configured number of NFSD threads in each pool; 469 * return code is the size in bytes of the string 470 * On error: return code is zero or a negative errno value 471 */ 472 static ssize_t write_pool_threads(struct file *file, char *buf, size_t size) 473 { 474 /* if size > 0, look for an array of number of threads per node 475 * and apply them then write out number of threads per node as reply 476 */ 477 char *mesg = buf; 478 int i; 479 int rv; 480 int len; 481 int npools; 482 int *nthreads; 483 struct net *net = netns(file); 484 485 mutex_lock(&nfsd_mutex); 486 npools = nfsd_nrpools(net); 487 if (npools == 0) { 488 /* 489 * NFS is shut down. The admin can start it by 490 * writing to the threads file but NOT the pool_threads 491 * file, sorry. Report zero threads. 492 */ 493 mutex_unlock(&nfsd_mutex); 494 strcpy(buf, "0\n"); 495 return strlen(buf); 496 } 497 498 nthreads = kzalloc_objs(int, npools); 499 rv = -ENOMEM; 500 if (nthreads == NULL) 501 goto out_free; 502 503 /* Threads started now would run without NFSv4 state. */ > 504 if (size > 0 && test_bit(NFSD_NET_UP, &nn->flags)) { 505 for (i = 0; i < npools; i++) { 506 rv = get_int(&mesg, &nthreads[i]); 507 if (rv == -ENOENT) 508 break; /* fewer numbers than pools */ 509 if (rv) 510 goto out_free; /* syntax error */ 511 rv = -EINVAL; 512 if (nthreads[i] < 0) 513 goto out_free; 514 trace_nfsd_ctl_pool_threads(net, i, nthreads[i]); 515 } 516 517 /* 518 * There must always be a thread in pool 0; the admin 519 * can't shut down NFS completely using pool_threads. 520 */ 521 if (nthreads[0] == 0) 522 nthreads[0] = 1; 523 524 rv = nfsd_set_nrthreads(i, nthreads, net); 525 if (rv) 526 goto out_free; 527 } 528 529 rv = nfsd_get_nrthreads(npools, nthreads, net); 530 if (rv) 531 goto out_free; 532 533 mesg = buf; 534 size = SIMPLE_TRANSACTION_LIMIT; 535 for (i = 0; i < npools && size > 0; i++) { 536 snprintf(mesg, size, "%d%c", nthreads[i], (i == npools-1 ? '\n' : ' ')); 537 len = strlen(mesg); 538 size -= len; 539 mesg += len; 540 } 541 rv = mesg - buf; 542 out_free: 543 kfree(nthreads); 544 mutex_unlock(&nfsd_mutex); 545 return rv; 546 } 547 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki