Re: [PATCH 4/5] infiniband-diags/libibnetdisc: Introduce a context object.

Sasha Khapyorsky
2009-10-23T17:45:43Z

On 17:01 Mon 31 Aug , Ira Weiny wrote:
>
> The discussion on the list has digressed from this patch. I still think this
> patch is valid and adds a level of flexibility which is needed regardless of
> what is decided about libibmad. Do you agree?

Not really. I still think that needed level of flexibility is available
for us without adding new mandatory APIs.

> Also, the last patch in the series ([PATCH 5/5] infiniband-diags/libibnetdisc:
> remove members of the fabric struct which are used in the scan only) cleans up
> some stuff from the external interface. If you really don't want to introduce
> a context object, then I can regenerate that final patch without the context.

Yes, since this patch is unrelated logically to a context stuff. It would
be good to rebase it back and repost as independent change.