All of lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP
@ 2018-05-14 15:34 Vasily Khoruzhick
  2018-05-16 22:26 ` Joe Hershberger
  2018-06-13 19:01 ` [U-Boot] " Joe Hershberger
  0 siblings, 2 replies; 5+ messages in thread
From: Vasily Khoruzhick @ 2018-05-14 15:34 UTC (permalink / raw)
  To: u-boot

That can happen if duplicate UDP packet arrived, and that's not uncommon.
Anyway, we ignore packets with rpc_id lower than last we sent for other
requests, so it makes sense to do that for read request as well.

Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
---
 net/nfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/net/nfs.c b/net/nfs.c
index 83ed0a7c37..9a16765ba1 100644
--- a/net/nfs.c
+++ b/net/nfs.c
@@ -822,6 +822,8 @@ static void nfs_handler(uchar *pkt, unsigned dest, struct in_addr sip,
 
 	case STATE_READ_REQ:
 		rlen = nfs_read_reply(pkt, len);
+		if (rlen == -NFS_RPC_DROP)
+			break;
 		net_set_timeout_handler(nfs_timeout, nfs_timeout_handler);
 		if (rlen > 0) {
 			nfs_offset += rlen;
-- 
2.17.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP
  2018-05-14 15:34 [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP Vasily Khoruzhick
@ 2018-05-16 22:26 ` Joe Hershberger
  2018-06-05  2:53   ` Vasily Khoruzhick
  2018-06-13 19:01 ` [U-Boot] " Joe Hershberger
  1 sibling, 1 reply; 5+ messages in thread
From: Joe Hershberger @ 2018-05-16 22:26 UTC (permalink / raw)
  To: u-boot

On Mon, May 14, 2018 at 10:34 AM, Vasily Khoruzhick <anarsoul@gmail.com> wrote:
> That can happen if duplicate UDP packet arrived, and that's not uncommon.
> Anyway, we ignore packets with rpc_id lower than last we sent for other
> requests, so it makes sense to do that for read request as well.
>
> Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>

Acked-by: Joe Hershberger <joe.hershberger@ni.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP
  2018-05-16 22:26 ` Joe Hershberger
@ 2018-06-05  2:53   ` Vasily Khoruzhick
  2018-06-06 22:25     ` Joe Hershberger
  0 siblings, 1 reply; 5+ messages in thread
From: Vasily Khoruzhick @ 2018-06-05  2:53 UTC (permalink / raw)
  To: u-boot

On Wed, May 16, 2018 at 3:26 PM Joe Hershberger <joe.hershberger@ni.com> wrote:
>
> On Mon, May 14, 2018 at 10:34 AM, Vasily Khoruzhick <anarsoul@gmail.com> wrote:
> > That can happen if duplicate UDP packet arrived, and that's not uncommon.
> > Anyway, we ignore packets with rpc_id lower than last we sent for other
> > requests, so it makes sense to do that for read request as well.
> >
> > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
>
> Acked-by: Joe Hershberger <joe.hershberger@ni.com>

Hey,

Can anyone pick this patch up?

Regards,
Vasily

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP
  2018-06-05  2:53   ` Vasily Khoruzhick
@ 2018-06-06 22:25     ` Joe Hershberger
  0 siblings, 0 replies; 5+ messages in thread
From: Joe Hershberger @ 2018-06-06 22:25 UTC (permalink / raw)
  To: u-boot

On Mon, Jun 4, 2018 at 9:53 PM, Vasily Khoruzhick <anarsoul@gmail.com> wrote:
> On Wed, May 16, 2018 at 3:26 PM Joe Hershberger <joe.hershberger@ni.com> wrote:
>>
>> On Mon, May 14, 2018 at 10:34 AM, Vasily Khoruzhick <anarsoul@gmail.com> wrote:
>> > That can happen if duplicate UDP packet arrived, and that's not uncommon.
>> > Anyway, we ignore packets with rpc_id lower than last we sent for other
>> > requests, so it makes sense to do that for read request as well.
>> >
>> > Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
>>
>> Acked-by: Joe Hershberger <joe.hershberger@ni.com>
>
> Hey,
>
> Can anyone pick this patch up?

Will do.

-Joe

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP
  2018-05-14 15:34 [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP Vasily Khoruzhick
  2018-05-16 22:26 ` Joe Hershberger
@ 2018-06-13 19:01 ` Joe Hershberger
  1 sibling, 0 replies; 5+ messages in thread
From: Joe Hershberger @ 2018-06-13 19:01 UTC (permalink / raw)
  To: u-boot

Hi Vasily,

https://patchwork.ozlabs.org/patch/913047/ was applied to http://git.denx.de/?p=u-boot/u-boot-net.git

Thanks!
-Joe

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2018-06-13 19:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-14 15:34 [U-Boot] [PATCH] net: nfs: don't fail when nfs_read_reply returns -NFS_RPC_DROP Vasily Khoruzhick
2018-05-16 22:26 ` Joe Hershberger
2018-06-05  2:53   ` Vasily Khoruzhick
2018-06-06 22:25     ` Joe Hershberger
2018-06-13 19:01 ` [U-Boot] " Joe Hershberger

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.