Thread (1 message) 1 message, 1 author, 2023-05-17

Re: [PATCH 05/12] cifs: Pass a pointer to virt_to_page()

flat view

From: Tom Talpey <tom@talpey.com>
Date: 2023-05-17 14:01:08
Also in: linux-arch, linux-cifs, linux-fsdevel, linux-m68k, lkml

On 5/11/2023 7:59 AM, Linus Walleij wrote:
Like the other calls in this function virt_to_page() expects
a pointer, not an integer.

However since many architectures implement virt_to_pfn() as
a macro, this function becomes polymorphic and accepts both a
(unsigned long) and a (void *).

Fix this up with an explicit cast.

Signed-off-by: Linus Walleij <redacted>
For fs/cifs:

Acked-by: Tom Talpey <tom@talpey.com>
quoted hunk
---
  fs/cifs/smbdirect.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/cifs/smbdirect.c b/fs/cifs/smbdirect.c
index 0362ebd4fa0f..964f07375a8d 100644
--- a/fs/cifs/smbdirect.c
+++ b/fs/cifs/smbdirect.c
@@ -2500,7 +2500,7 @@ static ssize_t smb_extract_kvec_to_rdma(struct iov_iter *iter,
  			if (is_vmalloc_or_module_addr((void *)kaddr))
  				page = vmalloc_to_page((void *)kaddr);
  			else
-				page = virt_to_page(kaddr);
+				page = virt_to_page((void *)kaddr);
  
  			if (!smb_set_sge(rdma, page, off, seg))
  				return -EIO;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help