From 556a5cb5751ce048d9b64443198bd65593694e7d Mon Sep 17 00:00:00 2001 From: Boris Brezillon Date: Mon, 23 Nov 2020 23:03:59 +0100 Subject: [PATCH] pan/bi: Fix constant slot selection The fau_index field contains the lower 4 bits of the 64bit constant, which allows one to reuse the same clause constant slot from different bundles if the upper 60 bits match. That doesn't work for constants referenced from the same instruction or for constants referenced from two instructions that are part of the same bundle though, since the fau_index is shared in that case. Signed-off-by: Boris Brezillon Reviewed-by: Alyssa Rosenzweig Part-of: --- src/panfrost/bifrost/bi_pack.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/panfrost/bifrost/bi_pack.c b/src/panfrost/bifrost/bi_pack.c index 5758c2bbe8e..9ff6b82d304 100644 --- a/src/panfrost/bifrost/bi_pack.c +++ b/src/panfrost/bifrost/bi_pack.c @@ -70,7 +70,13 @@ bi_lookup_constant(bi_clause *clause, uint32_t cons, bool *hi) /* Try to apply to top or to bottom */ uint64_t top = clause->constants[i]; - if (cons == ((uint32_t) top | (cons & 0xF))) + /* Constant slots can actually be used by a different + * tuples if the 60 upper bits match since the 4 LSB are + * encoded in the tuple itself. Let's not bother about this + * case until we start scheduling more than one tuple per + * clause. + */ + if (cons == (uint32_t) top) return i; if (cons == (top >> 32ul)) {