Re: [PATCH 31/34] misc: don't put nr_cpus into the fsstress -n argument

[Date Prev] [Date Next] [Thread Prev] [Thread Next] [Date Index] [Thread Index]



On Tue, Feb 04, 2025 at 01:30:23PM -0800, Darrick J. Wong wrote:
> From: Darrick J. Wong <djwong@xxxxxxxxxx>
> 
> fsstress -n is the number of fs operations per process, not the total
> number of operations.  There's no need to factor nr_cpus into the -n
> argument because that causes excess runtime as core count increases.
> 
> Signed-off-by: "Darrick J. Wong" <djwong@xxxxxxxxxx>
> ---
>  tests/generic/476 |    2 +-
>  tests/generic/642 |    2 +-
>  tests/generic/750 |    2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)

Looks good. That'll certainly help keep runtime down on large
machines.

Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx>

-- 
Dave Chinner
david@xxxxxxxxxxxxx




[Index of Archives]     [Linux Filesystems Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux