Hello, On Wed, Jun 05, 2019 at 10:50:18AM -0300, Martin Pieuchot wrote: > On 04/06/19(Tue) 23:57, Alexandr Nedvedicky wrote: > > Hello, > > > > I see, I owe some clarification to share wider context of this change. > > > > On Tue, Jun 04, 2019 at 10:32:57AM -0300, Martin Pieuchot wrote: > > > On 04/06/19(Tue) 01:37, Alexandr Nedvedicky wrote: > > > > Hello, > > > > > > > > diff below is just cosmetic change, which has no impact on current > > > > functionality, because there is just single network task to deliver > > > > packets to > > > > IP stack. I just want to push this small change to tree to minimize > > > > delta > > > > between current and my experimental branch. > > > > > > But why is it fine? > > > > currently all packets reaching IP stack are processed by if_netisr(), > > which > > runs as a task. The task pops up a packet from its queue (a.k.a. taskq) > > and > > passes the packet to corresponding xintr() handler for further > > processing. > > Only for packets that needs to be delivered/processed locally. In the > case of forwarding if_input_process() is the task that process packets. > >
thanks for pointing that out. I keep forgetting this important detail. regards sashan