Re: [PATCH v3 4/5] net: qrtr: ns: Limit the total number of nodes

[Manivannan Sadhasivam]

On Wed, Jul 08, 2026 at 06:49:39PM +0100, Youssef Samir wrote:
> 
> 
> On 4/9/2026 6:34 PM, Manivannan Sadhasivam wrote:
> > Currently, the nameserver doesn't limit the number of nodes it handles.
> > This can be an attack vector if a malicious client starts registering
> > random nodes, leading to memory exhaustion.
> > 
> > Hence, limit the maximum number of nodes to 64. Note that, limit of 64 is
> > chosen based on the current platform requirements. If requirement changes
> > in the future, this limit can be increased.
> 
> Hi Mani,
> 
> There are AI200 setups that can reach 384 nodes (192 * (AI200PF + AI200VF)).
> I'm not sure about limiting the number of nodes, but if there's a use-case
> that led to enforcing that limit, could we increase it to something like 512?
> 

Sure. As mentioned in the comment, we can increase the numbers based on the
requirements.

- Mani

-- 
மணிவண்ணன் சதாசிவம்