[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <OF60B75A13.F85EAE42-ONC125771F.00518947-C125771F.0051E8DC@transmode.se>
Date: Mon, 10 May 2010 16:54:40 +0200
From: Joakim Tjernlund <joakim.tjernlund@...nsmode.se>
To: Patrick McHardy <kaber@...sh.net>
Cc: Eric Dumazet <eric.dumazet@...il.com>, netdev@...r.kernel.org
Subject: Re: VLAN I/F's and TX queue.
Patrick McHardy <kaber@...sh.net> wrote on 2010/05/10 16:41:40:
>
> Eric Dumazet wrote:
> > Le lundi 10 mai 2010 à 16:26 +0200, Patrick McHardy a écrit :
> >
> >> Is the intention just to avoid accounting the packet as dropped?
> >> That seems fine to me since in case of NET_XMIT_CN its actually
> >> not the currently transmitted packet that was dropped.
> >>
> >> But part of the intention of the above mentioned patch was actually
> >> to inform higher layers of congestion so they can take action if
> >> desired, which would be defeated by this patch.
> >>
> >
> > I see, so maybe we want following patch instead ?
> >
> > (letting NET_XMIT_CN be given to caller, but accounting current packet
> > as transmitted ?)
>
> Perfect, thanks. I'd suggest to change macvlan in a similar fashion
> for consistency though.
>
> In any case please feel free to add my
>
> Acked-by: Patrick McHardy <kaber@...sh.net>
hmm, as I don't follow this could you tell me where the dropped pkgs
are accounted in my case:
eth0 Link encap:Ethernet HWaddr 00:AA:BB:CC:DD:EE
UP BROADCAST RUNNING MULTICAST MTU:1500 Metric:1
RX packets:8886910 errors:0 dropped:0 overruns:0 frame:0
TX packets:8880219 errors:0 dropped:0 overruns:0 carrier:0
collisions:0 txqueuelen:100
RX bytes:1626842951 (1.5 GiB) TX bytes:1555540810 (1.4 GiB)
eth0.1 Link encap:Ethernet HWaddr 00:AA:BB:CC:DD:EE
UP BROADCAST RUNNING MULTICAST MTU:1500 Metric:1
RX packets:2163164 errors:0 dropped:0 overruns:0 frame:0
TX packets:2161943 errors:0 dropped:98 overruns:0 carrier:0
collisions:0 txqueuelen:0
RX bytes:2467090557 (2.2 GiB) TX bytes:2480246455 (2.3 GiB)
eth0.1.1 Link encap:Ethernet HWaddr 00:AA:BB:CC:DD:EE
UP BROADCAST RUNNING MULTICAST MTU:1500 Metric:1
RX packets:2163164 errors:0 dropped:0 overruns:0 frame:0
TX packets:2161943 errors:0 dropped:98 overruns:0 carrier:0
collisions:0 txqueuelen:0
RX bytes:2458437901 (2.2 GiB) TX bytes:2471598683 (2.3 GiB)
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists