From 5386b7e011b4986a52de08d0b944eb317e5134fd Mon Sep 17 00:00:00 2001 From: Alyssa Rosenzweig Date: Thu, 12 Dec 2019 11:30:20 -0500 Subject: [PATCH] panfrost: Remove asserts in panfrost_pack_work_groups_compute It's a hot routine and these are exceedingly unlikely to break. Signed-off-by: Alyssa Rosenzweig Tested-by: Marge Bot Part-of: --- src/panfrost/encoder/pan_invocation.c | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/src/panfrost/encoder/pan_invocation.c b/src/panfrost/encoder/pan_invocation.c index ecde3da4648..0ec7c634a24 100644 --- a/src/panfrost/encoder/pan_invocation.c +++ b/src/panfrost/encoder/pan_invocation.c @@ -50,17 +50,7 @@ panfrost_pack_work_groups_compute( unsigned size_z, bool quirk_graphics) { - /* First of all, all 6 values are off-by-one (strictly positive). - * Account for that, first by ensuring all values are strictly positive - * and then by offsetting */ - - assert(num_x > 0); - assert(num_y > 0); - assert(num_z > 0); - - assert(size_x > 0); - assert(size_y > 0); - assert(size_z > 0); + /* First of all, all 6 values are off-by-one (strictly positive). */ num_x = MALI_POSITIVE(num_x); num_y = MALI_POSITIVE(num_y); -- 2.30.2