On Mon, 26 Sept 2022 at 16:24, Alex Bennée <alex.ben...@linaro.org> wrote: > > > Peter Maydell <peter.mayd...@linaro.org> writes: > > I still don't see anything in this patchset that updates > > the code which currently assumes requester_id to be a PCI > > index to check that it hasn't been handed a MemTxAttrs > > that uses requester_id as a CPU number. > > OK I'll update so all the existing cases setting requester_id also set > the type to MEMTXATTRS_MSI.
The problem is not the places that set requester_id (you can arrange for the default to be MSI for back-compat if you don't want to explicitly set it), but the places that *read* it. -- PMM