Page MenuHomePhabricator

mpam_supported wrong register
Closed, ResolvedPublic



while working with the MPAM extension, I noticed that in the lib/extensions/mpam/mpam.c file there is a mistake when accessing the feature register inside the mpam_supported() function. In particular the read_id_aa64dfr0_el1 register is accessed instead of the read_id_aa64pfr0_el1 register, where the bits regarding the availability of MPAM are.
I hope my assumption is correct and this is the right place to report it.

Regards and thank you in advance.

Event Timeline

Scimmiaparlante updated the task description. (Show Details)
AlexeiFedorov added a subscriber: AlexeiFedorov.EditedMay 10 2020, 12:18 PM

Hello Matteo,
Yes, you are right ID_AA64PFR0_EL1 must be used. Thanks for spotting this!

Please review the patch:


Scimmiaparlante closed this task as Resolved.May 28 2020, 7:42 AM
Scimmiaparlante removed Scimmiaparlante as the assignee of this task.