On Thu, Apr 19, 2018 at 02:23:25PM -0400, Doug Ledford wrote: > On Thu, 2018-04-05 at 18:55 +0300, Leon Romanovsky wrote: > > On Wed, Apr 04, 2018 at 09:00:01PM -0600, Jason Gunthorpe wrote: > > > This is done by auditing all callers of ucma_get_ctx and switching the > > > ones that unconditionally touch ->device to ucma_get_ctx_dev. This covers > > > a little less than half of the call sites. > > > > > > The 11 remaining call sites to ucma_get_ctx() were manually audited. > > > > > > It looks like none of these cases cause bugs due to the FSM system > > > inside the CMA, but documenting the requirement and the result of this > > > audit is still productive. > > > > > > Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx> > > > drivers/infiniband/core/ucma.c | 36 ++++++++++++++++++++++++------------ > > > 1 file changed, 24 insertions(+), 12 deletions(-) > > > > > > v3: > > > - Reorder the function after put_ctx so it compiles warning free > > > I really shouldn't rush so much :( > > > > > > > Thanks, > > Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxxxx> > > Thanks, applied. This seemed important enough to warrant early -rc, so > I put it there instead of for-next. You may want to change the last paragraph there as it apparently does fix syzkaller bugs :( If I'd known that it would have gone weeks ago. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
