From 2bd6e4dcd43bb0d836f12232050e73ce1510bb0f Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Thu, 3 Jan 2013 16:38:33 +0000 Subject: [PATCH] sna: Add a pair of asserts to validate fls()/cache_bucket() Signed-off-by: Chris Wilson --- src/sna/kgem.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/sna/kgem.c b/src/sna/kgem.c index df8c4464..398244dc 100644 --- a/src/sna/kgem.c +++ b/src/sna/kgem.c @@ -603,11 +603,15 @@ static void __kgem_request_free(struct kgem_request *rq) static struct list *inactive(struct kgem *kgem, int num_pages) { + assert(num_pages < MAX_CACHE_SIZE / PAGE_SIZE); + assert(cache_bucket(num_pages) < NUM_CACHE_BUCKETS); return &kgem->inactive[cache_bucket(num_pages)]; } static struct list *active(struct kgem *kgem, int num_pages, int tiling) { + assert(num_pages < MAX_CACHE_SIZE / PAGE_SIZE); + assert(cache_bucket(num_pages) < NUM_CACHE_BUCKETS); return &kgem->active[cache_bucket(num_pages)][tiling]; }