On Mon, 12 Jun 2006, Sridhar Samudrala wrote:
- sendpage = sock-ops-sendpage ? : sock_no_sendpage;
+ sendpage = kernel_sendpage ? : sock_no_sendpage;
This is not equivalent.
Actually, we could make this a simple assignment as we check for
sock-ops-sendpage in
On Tue, 2006-06-13 at 10:07 -0400, James Morris wrote:
On Mon, 12 Jun 2006, Sridhar Samudrala wrote:
- sendpage = sock-ops-sendpage ? : sock_no_sendpage;
+ sendpage = kernel_sendpage ? : sock_no_sendpage;
This is not equivalent.
Actually, we could
On Tue, 13 Jun 2006, Sridhar Samudrala wrote:
My patch doesn't touch this section of the code and this is called
after the assignment we are talking about. So we should be using the
right sendpage in the actual call.
Ok.
Acked-by: James Morris [EMAIL PROTECTED]
(for both patches).
--
This patch updates sunrpc to use in-kernel sockets API.
Thanks
Sridhar
diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c
index a27905a..ee80b3c 100644
--- a/net/sunrpc/svcsock.c
+++ b/net/sunrpc/svcsock.c
@@ -388,7 +388,7 @@ svc_sendto(struct svc_rqst *rqstp, struc
/* send head */
On Mon, 12 Jun 2006, Sridhar Samudrala wrote:
- sendpage = sock-ops-sendpage ? : sock_no_sendpage;
+ sendpage = kernel_sendpage ? : sock_no_sendpage;
This is not equivalent.
--
James Morris
[EMAIL PROTECTED]
-
To unsubscribe from this list: send the line unsubscribe netdev in
the
James Morris wrote:
On Mon, 12 Jun 2006, Sridhar Samudrala wrote:
- sendpage = sock-ops-sendpage ? : sock_no_sendpage;
+ sendpage = kernel_sendpage ? : sock_no_sendpage;
This is not equivalent.
Actually, we could make this a simple assignment as we check for