Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

From: Geliang Tang
Date: Sun Oct 04 2015 - 22:31:14 EST


On Sun, Oct 04, 2015 at 06:50:55PM -0700, Davidlohr Bueso wrote:
> On Sun, 04 Oct 2015, Geliang Tang wrote:
>
> >BUG_ON() already contain an unlikely compiler flag. Drop it.
> >
> >Signed-off-by: Geliang Tang <geliangtang@xxxxxxx>
>
> Acked-by: Davidlohr Bueso <dave@xxxxxxxxxxxx>
>
> ... but I believe you do have some left:
>
> drivers/scsi/scsi_lib.c: BUG_ON(unlikely(count > ivecs));
> drivers/scsi/scsi_lib.c: BUG_ON(unlikely(count > queue_max_integrity_segments(rq->q)));
> kernel/sched/core.c: BUG_ON(unlikely(task_stack_end_corrupted(prev)));

Thanks for your review, the left have been sended out already in two other patches.

--
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/