> On Tue, Mar 01, 2016 at 10:33:12AM +0000, Lim, Siew Hoon wrote: > > > > > > On Tue, Mar 01, 2016 at 09:49:21AM +0800, Lim Siew Hoon wrote: > > > > The assert check for c->bo should be done before DBG, else it will > > > > hit memory access violation when c->bo is null. > > > > > > > > Signed-off-by: Lim Siew Hoon <siew.hoon.lim@xxxxxxxxx> > > > > Reviewed-by: Matt Roper <matthew.d.roper@xxxxxxxxx> > > > > > > Pardon? With a r-b? > > > > What is a r-b? > Reviewed-by. ohh....sorry about this. It should another 1 lines signed-off-by after Reviewed-by. > > > I'm helping to fix the KW issue hit in our repo side. > > There is no bug in the code, just kw. It is a ternary operator guarding the > dereference. > > > I suggest the assert checking should be move up before DBG(...) print > statement to avoid memory access violation happen if c->bo is NULL for print > message. > > Of cause the chance will be very low to get c->bo is NULL. > > If you look at the code if prints handle=0 if c->bo to indicate the NULL. Please > read the code first. Oh...ok. And you are right. Thank you for feedback. > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx