[ofa-general] Re: [PATCHv2 1/2] ipoib: handle pkey change events
Michael S. Tsirkin
mst at dev.mellanox.co.il
Tue May 8 08:19:27 PDT 2007
> Quoting Yosef Etigin <yosefe at voltaire.com>:
> Subject: Re: [PATCHv2 1/2] ipoib: handle pkey change events
>
> Michael S. Tsirkin wrote:
> >>Quoting Yosef Etigin <yosefe at voltaire.com>:
> >>Subject: [PATCHv2 1/2] ipoib: handle pkey change events
> >>
> >>This issue was found during partitioning & SM fail over testing.
> >>
> >> * Added flush flag to ipoib_ib_dev_stop(), ipoib_ib_dev_down() alike
> >> * Rename the polling thread work to 'pkey_poll_task' to avoid ambiguity
> >> * Upon PKEY_CHANGE event, schedule a work that restarts the QP
> >> * Restart child interfaces before parent
> >
> >
> > What's the reason for this change?
> > Is this a separate bugfix?
> > You might want to put this info in the log.
> >
>
> The reason is that if the child are restarted after the parent, and the parent is
> not up, then the flush function returns immediately due to the INITIALLIZED bit test.
> Now I think that we might use a goto statement instead.
So ... what the problem? I still don't see it.
--
MST
More information about the general
mailing list