Hello,

</snip>
> > 
> >     would it make sense to have let's say
> > 
> >         mq_move2mlist(struct mbuf_queue *, struct mbuf_list *)
> 
> This already exists, it's called mq_delist()
> 

    thanks, I've missed that when skimming through sources.

> > 
> >     This would allow as to move whole globally visible la->la_mq into
> >     into mbuf list, which will be a local variable. This way we won't
> >     need to jump on la->la_mq's mutex with every loop iteration.
> > 
> >     I it makes sense,  we can do it as a follow up change.
> 
> We'd need some other way to do the 'mbuf is back in queue' detection,
> but I agree this seems like a sensible thing to do.
> 

    I agree, I think this should be discussed in new email thread,
    because I don't want to hijack bluhm's diff for that.

as I've said earlier I'm OK if bluhm's will commit his diff.

thanks and
regards
sashan

Reply via email to