[ofa-general] Re: Re: [PATCH] for-2.6.23 ib/umad: add partition support

[Michael S. Tsirkin]

> Quoting Michael S. Tsirkin <mst at dev.mellanox.co.il>:
> Subject: Re: Re: [PATCH] for-2.6.23 ib/umad: add partition support
> 
> > Quoting Sean Hefty <sean.hefty at intel.com>:
> > Subject: RE: Re: [PATCH] for-2.6.23 ib/umad: add partition support
> > 
> > >This assumes an open file desriptor per-pkey, so the proposed API
> > >extension umad_set_pkey would have to be changed to be per-port rather
> > >than per-mad. But I think this is a better API, too: most apps
> > >likely work within a single partition.
> > 
> > I don't think this is true for apps that use the userspace MAD interface (e.g.
> > opensm).
> 
> SM (rather, SA) can just open file descriptor per pkey - it created them itself,
> and there's a small number of partitions.
> 
> > Beyond that, this approach doesn't work for receiving MADs on different PKeys.
> 
> Yes, it does: we just filter out the MADs where pkey does not match.
> 
> I think that most other apps (besides SA) should really treat
> each partition as a separate network. So getting MADs for a specific
> pkey, rather than all pkeys, makes total sense to me.

Hal, could you pls comment on whether this approach will work for opensm?

-- 
MST