Re: [PATCH 2/2] seek_sanity_test: drop unused pagesz

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



On Tue, Oct 11, 2022 at 11:23:40AM +0200, David Disseldorp wrote:
> b91889d7 ("seek_sanity_test: use XFS ioctls to determine file allocation
> unit size") left this unused, resulting in:
> 
> seek_sanity_test.c: In function 'get_io_sizes':
> seek_sanity_test.c:70:6: warning: unused variable 'pagesz'
>   int pagesz = sysconf(_SC_PAGE_SIZE);
>       ^~~~~~
> 
> Signed-off-by: David Disseldorp <ddiss@xxxxxxx>
> ---

OK, finally we can't stand this warning :)

Reviewed-by: Zorro Lang <zlang@xxxxxxxxxx>

>  src/seek_sanity_test.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/src/seek_sanity_test.c b/src/seek_sanity_test.c
> index 78f835e8..8a586f74 100644
> --- a/src/seek_sanity_test.c
> +++ b/src/seek_sanity_test.c
> @@ -67,7 +67,6 @@ static int get_io_sizes(int fd)
>  	off_t pos = 0, offset = 1;
>  	struct stat buf;
>  	int shift, ret;
> -	int pagesz = sysconf(_SC_PAGE_SIZE);
>  
>  	ret = detect_xfs_alloc_unit(fd);
>  	if (!ret)
> -- 
> 2.35.3
> 




[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