On Fri, Sep 27, 2019 at 06:37:00AM -0700, Eric Dumazet wrote:
>
>
> On 9/27/19 1:55 AM, Stefano Garzarella wrote:
>
> > Good catch!
> >
> > Maybe we can solve in this way:
> >
> > list_for_each_entry(pkt, &vvs->rx_queue, list) {
> > size_t off = pkt->off;
> >
> > if (total == len)
> > break;
> >
> > while (total < len && off < pkt->len) {
> > /* using 'off' instead of 'pkt->off' */
> > ...
> >
> > total += bytes;
> > off += bytes;
> > }
> > }
> >
> > What do you think?
> >
>
> Maybe, but I need to see a complete patch, evil is in the details :)
>
Thanks both for your comments, I will take them into account and submit
a second version.
Matias
