Re: Error in inner loop in validate_cmds_sorted / out of bounds issue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 28, 2015 at 08:45:51AM +0100, Chris Wilson wrote:
> On Mon, Jul 27, 2015 at 09:15:32PM -0700, Hanno Böck wrote:
> > On Mon, 27 Jul 2015 09:59:45 +0100
> > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> > 
> > > The tables aren't sorted, that is worth fixing.
> > 
> > Attached patch should do that and fix the loop. Now it boots without
> > errors.
> > 
> > Does that look okay? If so please apply.
> 
> Indeed, nice catch. Could you please read
> Documentation/SubmittingPatches and apply your Signed-off-by and then we
> can accept this patch under your authorship.
> 
> Preferrably this is two patches, (a) fix the tables, (b) fix the
> validator. That way we can delay enabling the validator if we need to
> fix the tables for others.

Also can you please add signed-off-by lines to your patch when
resubmitting? See Documentation/SubmittingPatches for all the details.

Thanks, Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux