Re: [PATCH] cxl: Test decoder flags as bitmasks

[Alison Schofield <alison.schofield@intel.com>]

On Fri, Feb 06, 2026 at 12:33:34PM -0700, Dave Jiang wrote:
> 
> 
> On 2/6/26 11:14 AM, Alison Schofield wrote:
> > The CXL decoder flags are defined as bitmasks, not bit indices.
> > Using test_bit() to check them interprets the mask value as a bit
> > index, which is the wrong test.
> > 
> > For CXL_DECODER_F_NORMALIZED_ADDRESSING the test reads beyond the
> > flags word, making the flag sometimes appear set and blocking creation
> > of CXL region debugfs attributes that support poison operations.
> > 
> > For CXL_DECODER_F_LOCK the test reads beyond the defined bits, causing
> > the test to always return false and allowing resets that should have
> > been blocked.
> > 
> > Replace test_bit() with a bitmask check.
> > 
> > Found with cxl-test.
> > 
> > Fixes: 208f432406b7 ("cxl: Disable HPA/SPA translation handlers for Normalized Addressing")
> > Fixes: 2230c4bdc412 ("cxl: Add handling of locked CXL decoder")
> 
> Given it needs to back port and fix 2 different commits, maybe this needs to split?

OK - will do, ignore this patch.

It'll be a 2 patch series because of the dependency, but I believe that
is still preferred by backport folks.