Re: [openib-general] [PATCH] mthca: initialize send and receive queue locks separately

From: Michael S. Tsirkin
Date: Tue Jul 04 2006 - 23:22:57 EST


Quoting r. Roland Dreier <rdreier@xxxxxxxxx>:
> I think that is actually a very minor bug you've found. If someone
> were posting a work request at the same time as they transitioned a QP
> to reset (which is a legitimate if not sensible thing to do), then the
> spinlock could get reinitialized while it was held. Which would be
> bad.

I think you can't post work requests to a QP in reset state, since IB spec
forbids this. If you do, it seems bad things will happen anyway, for example
head/tail pointers may get out of sync as mthca_wq_init clears these.

--
MST
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/