Re: [PATCH] can-utils: AOSP build improvements

[Oliver Hartkopp <socketcan@hartkopp.net>]

On 18.02.2013 10:21, Marc Kleine-Budde wrote:

> On 02/18/2013 10:10 AM, Yegor Yefremov wrote:


>>
>> Btw what about isotp.h? Is it going to be mainlined?
> 
> That's Oliver's cup of tea :)


Hm - yes :-)

Of course it is going to be mainlined.

The code itself is working properly for a long time now. The only missing
point is that i do not have a really good idea how to cope with the fact that
up to 500 CAN frames can be pushed into the CAN netdev tx queue at a time.

The advisory to increase the tx_queue_len to 1000 is not a solution.

Maybe qdiscs is a solution or we need to be able to peek the tx queue state
whether we can push the next CAN frame to the device, or if we need to wait.

Once there is a good solution for that, it should be easy to make isotp fit
for mainline inclusion.

Regards,
Oliver