* [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.