* Should we keep the advice to increase queue max length?
@ 2023-12-12 0:45 Duncan Roe
2023-12-12 1:09 ` Florian Westphal
0 siblings, 1 reply; 2+ messages in thread
From: Duncan Roe @ 2023-12-12 0:45 UTC (permalink / raw)
To: Netfilter Development
Hi Everyone,
Under the title "Performance", libnetfilter_queue main page advises
> increase queue max length with nfq_set_queue_maxlen() to resist to packets burst
but from experimenting it seems to me this does no good at all.
/proc/net/netfilter/nfnetlink_queue has a line for every active queue. The 3rd
field is the number of queued packets. The max length is not in these lines but
kernel source suggests it is 1024 by default. Anyway, I updated nfq6 to be able
to set the max (using mnl functions).
And I found the maximum number of queued packets is: 238. Further packets are
dropped.
If I lower the max below 238, limiting occurs at the new max.
So I propose to drop the advice to increase the queue max length when I revise
the libnetfilter_queue main page as part of the project to stop using
libnfnetlink.
Anyone have any comments?
Cheers ... Duncan.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: Should we keep the advice to increase queue max length?
2023-12-12 0:45 Should we keep the advice to increase queue max length? Duncan Roe
@ 2023-12-12 1:09 ` Florian Westphal
0 siblings, 0 replies; 2+ messages in thread
From: Florian Westphal @ 2023-12-12 1:09 UTC (permalink / raw)
To: Netfilter Development
Duncan Roe <duncan_roe@optusnet.com.au> wrote:
> /proc/net/netfilter/nfnetlink_queue has a line for every active queue. The 3rd
> field is the number of queued packets. The max length is not in these lines but
> kernel source suggests it is 1024 by default. Anyway, I updated nfq6 to be able
> to set the max (using mnl functions).
>
> And I found the maximum number of queued packets is: 238. Further packets are
> dropped.
>
> If I lower the max below 238, limiting occurs at the new max.
>
> So I propose to drop the advice to increase the queue max length when I revise
> the libnetfilter_queue main page as part of the project to stop using
> libnfnetlink.
>
> Anyone have any comments?
This code is very old, it stems from a time where the network stack
did not do any kind of packet aggregation.
I think the hint should be removed as you suggest.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-12-12 1:09 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-12 0:45 Should we keep the advice to increase queue max length? Duncan Roe
2023-12-12 1:09 ` Florian Westphal
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.