> On Dec 17, 2022, at 8:08 PM, Khem Raj <raj.k...@gmail.com> wrote:
>
> 64bit off_t is already in use since build uses _FILE_OFFSET_BITS=64
> already. Using lseek/off_t also makes it work with latest must without
s/must/musl/ ?
> using _LARGEFILE64_SOURCE macro. This macro is implied with _GNU_SOURCE
> when using glibc but not with musl.
>
Other than the nit LGTM
Reviewed-by: Raphael Norwitz <raphael.norw...@nutanix.com>
> Signed-off-by: Khem Raj <raj.k...@gmail.com>
> Cc: Michael S. Tsirkin <m...@redhat.com>
> CC: Raphael Norwitz <raphael.norw...@nutanix.com>
> ---
> contrib/vhost-user-blk/vhost-user-blk.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/contrib/vhost-user-blk/vhost-user-blk.c
> b/contrib/vhost-user-blk/vhost-user-blk.c
> index aa99877fcd..7941694e53 100644
> --- a/contrib/vhost-user-blk/vhost-user-blk.c
> +++ b/contrib/vhost-user-blk/vhost-user-blk.c
> @@ -532,9 +532,9 @@ vub_get_blocksize(int fd)
> static void
> vub_initialize_config(int fd, struct virtio_blk_config *config)
> {
> - off64_t capacity;
> + off_t capacity;
>
> - capacity = lseek64(fd, 0, SEEK_END);
> + capacity = lseek(fd, 0, SEEK_END);
> config->capacity = capacity >> 9;
> config->blk_size = vub_get_blocksize(fd);
> config->size_max = 65536;
> --
> 2.39.0
>
