* [PATCH] xfsprogs: Fix mismatched return type of filesize() @ 2025-02-17 15:50 ` Pavel Reichl 2025-02-18 8:18 ` Carlos Maiolino 2025-02-21 18:57 ` [PATCH v2] " Pavel Reichl 0 siblings, 2 replies; 4+ messages in thread From: Pavel Reichl @ 2025-02-17 15:50 UTC (permalink / raw) To: aalbersh; +Cc: linux-xfs The function filesize() was declared with a return type of 'long' but defined with 'off_t'. This mismatch caused build issues due to type incompatibility. This commit updates the declaration to match the definition, ensuring consistency and preventing potential compilation errors. Signed-off-by: Pavel Reichl <preichl@redhat.com> --- mkfs/proto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkfs/proto.c b/mkfs/proto.c index 6dd3a200..981f5b11 100644 --- a/mkfs/proto.c +++ b/mkfs/proto.c @@ -20,7 +20,7 @@ static struct xfs_trans * getres(struct xfs_mount *mp, uint blocks); static void rsvfile(xfs_mount_t *mp, xfs_inode_t *ip, long long len); static int newregfile(char **pp, char **fname); static void rtinit(xfs_mount_t *mp); -static long filesize(int fd); +static off_t filesize(int fd); static int slashes_are_spaces; /* -- 2.48.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] xfsprogs: Fix mismatched return type of filesize() 2025-02-17 15:50 ` [PATCH] xfsprogs: Fix mismatched return type of filesize() Pavel Reichl @ 2025-02-18 8:18 ` Carlos Maiolino 2025-02-18 18:15 ` Darrick J. Wong 2025-02-21 18:57 ` [PATCH v2] " Pavel Reichl 1 sibling, 1 reply; 4+ messages in thread From: Carlos Maiolino @ 2025-02-18 8:18 UTC (permalink / raw) To: Pavel Reichl; +Cc: aalbersh, linux-xfs On Mon, Feb 17, 2025 at 04:50:43PM +0100, Pavel Reichl wrote: > The function filesize() was declared with a return type of 'long' but > defined with 'off_t'. This mismatch caused build issues due to type > incompatibility. > > This commit updates the declaration to match the definition, ensuring > consistency and preventing potential compilation errors. > > Signed-off-by: Pavel Reichl <preichl@redhat.com> Looks good, perhaps add: Fixes: 73fb78e5ee8 ("mkfs: support copying in large or sparse files") ? Otherwise, Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com> > --- > mkfs/proto.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mkfs/proto.c b/mkfs/proto.c > index 6dd3a200..981f5b11 100644 > --- a/mkfs/proto.c > +++ b/mkfs/proto.c > @@ -20,7 +20,7 @@ static struct xfs_trans * getres(struct xfs_mount *mp, uint blocks); > static void rsvfile(xfs_mount_t *mp, xfs_inode_t *ip, long long len); > static int newregfile(char **pp, char **fname); > static void rtinit(xfs_mount_t *mp); > -static long filesize(int fd); > +static off_t filesize(int fd); > static int slashes_are_spaces; > > /* > -- > 2.48.1 > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfsprogs: Fix mismatched return type of filesize() 2025-02-18 8:18 ` Carlos Maiolino @ 2025-02-18 18:15 ` Darrick J. Wong 0 siblings, 0 replies; 4+ messages in thread From: Darrick J. Wong @ 2025-02-18 18:15 UTC (permalink / raw) To: Carlos Maiolino; +Cc: Pavel Reichl, aalbersh, linux-xfs On Tue, Feb 18, 2025 at 09:18:46AM +0100, Carlos Maiolino wrote: > On Mon, Feb 17, 2025 at 04:50:43PM +0100, Pavel Reichl wrote: > > The function filesize() was declared with a return type of 'long' but > > defined with 'off_t'. This mismatch caused build issues due to type > > incompatibility. > > > > This commit updates the declaration to match the definition, ensuring > > consistency and preventing potential compilation errors. > > > > Signed-off-by: Pavel Reichl <preichl@redhat.com> > > Looks good, perhaps add: > > Fixes: 73fb78e5ee8 ("mkfs: support copying in large or sparse files") Yes please! Sorry about the discrepancy. With that fixed, Reviewed-by: "Darrick J. Wong" <djwong@kernel.org> --D > ? > > Otherwise, > Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com> > > > --- > > mkfs/proto.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/mkfs/proto.c b/mkfs/proto.c > > index 6dd3a200..981f5b11 100644 > > --- a/mkfs/proto.c > > +++ b/mkfs/proto.c > > @@ -20,7 +20,7 @@ static struct xfs_trans * getres(struct xfs_mount *mp, uint blocks); > > static void rsvfile(xfs_mount_t *mp, xfs_inode_t *ip, long long len); > > static int newregfile(char **pp, char **fname); > > static void rtinit(xfs_mount_t *mp); > > -static long filesize(int fd); > > +static off_t filesize(int fd); > > static int slashes_are_spaces; > > > > /* > > -- > > 2.48.1 > > > > > ^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2] xfsprogs: Fix mismatched return type of filesize() 2025-02-17 15:50 ` [PATCH] xfsprogs: Fix mismatched return type of filesize() Pavel Reichl 2025-02-18 8:18 ` Carlos Maiolino @ 2025-02-21 18:57 ` Pavel Reichl 1 sibling, 0 replies; 4+ messages in thread From: Pavel Reichl @ 2025-02-21 18:57 UTC (permalink / raw) To: aalbersh; +Cc: linux-xfs The function filesize() was declared with a return type of 'long' but defined with 'off_t'. This mismatch caused build issues due to type incompatibility. This commit updates the declaration to match the definition, ensuring consistency and preventing potential compilation errors. Fixes: 73fb78e5ee8 ("mkfs: support copying in large or sparse files") Signed-off-by: Pavel Reichl <preichl@redhat.com> Reviewed-by: Darrick J. Wong <djwong@kernel.org> Reviewed-by: Carlos Maiolino <cem@kernel.org> --- mkfs/proto.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mkfs/proto.c b/mkfs/proto.c index 6dd3a200..981f5b11 100644 --- a/mkfs/proto.c +++ b/mkfs/proto.c @@ -20,7 +20,7 @@ static struct xfs_trans * getres(struct xfs_mount *mp, uint blocks); static void rsvfile(xfs_mount_t *mp, xfs_inode_t *ip, long long len); static int newregfile(char **pp, char **fname); static void rtinit(xfs_mount_t *mp); -static long filesize(int fd); +static off_t filesize(int fd); static int slashes_are_spaces; /* -- 2.48.1 ^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-02-21 18:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <-uT7HOcTG_xe8v8U0_5OQg6ll9vJyYEFQeSs_2FwUHujx116vYRcX2iovzoJvkN8K9zDDmQxQWB6H1CDLiOVdw==@protonmail.internalid>
2025-02-17 15:50 ` [PATCH] xfsprogs: Fix mismatched return type of filesize() Pavel Reichl
2025-02-18 8:18 ` Carlos Maiolino
2025-02-18 18:15 ` Darrick J. Wong
2025-02-21 18:57 ` [PATCH v2] " Pavel Reichl
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.