Re: [PATCH net-next V2 12/13] selftests: drv-net: psp: Fix responder parsing
Jakub Kicinski <[email protected]>
| Newsgroups | org.kernel.vger.netdev,org.kernel.vger.linux-kernel,org.kernel.vger.linux-kselftest,org.kernel.vger.linux-rdma |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 10 Aug 2026 11:42:36 +0000 Cosmin Ratiu wrote:
> > > +#define cmd(_name,
> > > _extra_sz) \
> > > ({
> > > \
> > > ssize_t sz =
> > > sizeof(_name); \
> > > - bool match = n >= sz
> > > && !memcmp(buf, _name, sz); \
> > > + bool match = off >= sz + (_extra_sz)
> > > && \
> > > + !memcmp(buf, _name,
> > > sz); \
> >
> > Please don't replace off by n.
>
> I actually replaced n by off. Did you mean "don't replace n by off"?
Of course!
> Because the two are identical after the recv error checking. I don't
> see the point of maintaining two variables when one suffices. Either n
> or off can describe the number of bytes in buf. off is static and
> persists across chunks, while n is temporary.
The variables have semantic meaning, which makes it easier to follow
the code.
> So to clarify, do you want to keep both?
Yes.