Unverified Commit 07c574eb authored by David Howells's avatar David Howells Committed by Christian Brauner
Browse files

netfs: Fix netfs_unbuffered_read() to return ssize_t rather than int 



Fix netfs_unbuffered_read() to return an ssize_t rather than an int as
netfs_wait_for_read() returns ssize_t and this gets implicitly truncated.

Signed-off-by: default avatarDavid Howells <dhowells@redhat.com>
Link: https://lore.kernel.org/r/20250314164201.1993231-5-dhowells@redhat.com


Acked-by: default avatar"Paulo Alcantara (Red Hat)" <pc@manguebit.com>
cc: Jeff Layton <jlayton@kernel.org>
cc: Viacheslav Dubeyko <slava@dubeyko.com>
cc: Alex Markuze <amarkuze@redhat.com>
cc: Ilya Dryomov <idryomov@gmail.com>
cc: ceph-devel@vger.kernel.org
cc: linux-fsdevel@vger.kernel.org
Signed-off-by: default avatarChristian Brauner <brauner@kernel.org>
parent 15e9aaf9

fs/netfs/direct_read.c

+3 −3
Original line number Diff line number Diff line
@@ -125,9 +125,9 @@ static int netfs_dispatch_unbuffered_reads(struct netfs_io_request *rreq) 
 * Perform a read to an application buffer, bypassing the pagecache and the

 * local disk cache.

 */

static int netfs_unbuffered_read(struct netfs_io_request *rreq, bool sync)

static ssize_t netfs_unbuffered_read(struct netfs_io_request *rreq, bool sync)

{

	int ret;

	ssize_t ret;



	_enter("R=%x %llx-%llx",

	       rreq->debug_id, rreq->start, rreq->start + rreq->len - 1);
@@ -155,7 +155,7 @@ static int netfs_unbuffered_read(struct netfs_io_request *rreq, bool sync) 
	else

		ret = -EIOCBQUEUED;

out:

	_leave(" = %d", ret);

	_leave(" = %zd", ret);

	return ret;

}