On Thu, 2006-11-30 at 14:07 +0100, Patrick McHardy wrote:
> Qdiscs don't use RTABs to measure rates but to calculate
> transmission times. Transmission time is always related
> to the length, the difference between our patches is that
> you modify the RTABs in advance to include the overhead
> in the calculation, my patch changes the length used to
> look up the transmission time. Which works with or
> without RTABs.
Thanks, I understand now.
I had decided that that STAB was indeed the way forward,
and hence I should devote my time to making it work
with ADSL/ATM. The only issue is now I don't have
any spare time right now.
Yesterday I was chatting about this at LCA 2007, and
it dawned on me that there is a problem with the dual
RTAB/STAB approach.
Currently the lookup in the kernel is
time_to_transmit_a_packet = RTAB[packet_length_seen_by_kernel]
As I understand it, you are proposing to change that to
time_to_transmit_a_packet
= RTAB[STAB[packet_length_seen_by_kernel]]
= RTAB[packet_length_seen_on_the_wire]
Given RTAB is the same in both cases the results of the
calculation will be different (and ergo wrong in one case
or the other). RTAB can't change and remain compatible
with old kernels. Ergo this approach breaks backward
compatibility.
I can't see any way forward but to break the link
between RTAB and STAB. Personally I think this is a
good thing, as I think STAB should replace RTAB
completely. One way to do this is to give the kernel
STAB + transmission rate, so the calculation above
becomes:
time_to_transmit_a_packet
= STAB[packet_length_seen_by_kernel] * transmission_rate
RTAB disappears in new kernels, and the compatibility
problem goes with it.
Can you think of other ways forward? I need some way
forward so I can get this thing working.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
