Re: [PATCH v4 4/4] IB/sa: Route SA pathrecord query through netlink

Jason Gunthorpe
2015-06-10T19:10:26Z

On Wed, Jun 10, 2015 at 01:47:36PM -0400, Hal Rosenstock wrote:
> On 6/9/2015 10:57 AM, kaike.wan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org wrote:
> > From: Kaike Wan <kaike.wan-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> >
> > This patch routes a SA pathrecord query to netlink first
>
> Should only unicast PRs be done in this manner or should API support
> enabling for unicast and/or multicast ?
>
> AFAIK kernel doesn't query multicast PRs now (queries MCMRs) but this
> seems like it would help make it future proof and not have to take
> timeout on local query unless app supports it.

It is a good question. We can clearly extend toward that, using a MGID
as the DGID and adding additional nested netlink fields.

However, does it make sense? I don't think user space should be
joining on behalf of the kernel... Do we ever query without a join?

Jasno