| Message ID | 20221217180844.3453267-1-raj.khem@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/2] contrib/vhost-user-blk: Replace lseek64 with lseek | expand |
> On Dec 17, 2022, at 8:08 PM, Khem Raj <raj.khem@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.norwitz@nutanix.com> > Signed-off-by: Khem Raj <raj.khem@gmail.com> > Cc: Michael S. Tsirkin <mst@redhat.com> > CC: Raphael Norwitz <raphael.norwitz@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 >
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;
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 using _LARGEFILE64_SOURCE macro. This macro is implied with _GNU_SOURCE when using glibc but not with musl. Signed-off-by: Khem Raj <raj.khem@gmail.com> Cc: Michael S. Tsirkin <mst@redhat.com> CC: Raphael Norwitz <raphael.norwitz@nutanix.com> --- contrib/vhost-user-blk/vhost-user-blk.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)