diff options
author | Zhang Rui <[email protected]> | 2024-07-03 13:54:45 +0800 |
---|---|---|
committer | Rafael J. Wysocki <[email protected]> | 2024-07-09 18:29:07 +0200 |
commit | b755367602d70deade956cbe0b8a3f5a12f569dc (patch) | |
tree | 1e2c29f780bff8ac0d8f7055b12e2493a658966e /drivers/fpga/fpga-bridge.c | |
parent | 55397323f3eaba16b93182c35be85aa35cad1432 (diff) |
thermal: intel: hfi: Give HFI instances package scope
The Intel Software Developer's Manual defines the scope of HFI (registers
and memory buffer) as a package. Use package scope(*) in the software
representation of an HFI instance.
Using die scope in HFI instances has the effect of creating multiple
conflicting instances for the same package: each instance allocates its
own memory buffer and configures the same package-level registers.
Specifically, only one of the allocated memory buffers can be set in the
MSR_IA32_HW_FEEDBACK_PTR register. CPUs get incorrect HFI data from the
table.
The problem does not affect current HFI-capable platforms because they
all have single-die processors.
(*) We used die scope for HFI instances because there had been
processors with packages enumerated as dies. None of those systems
supported HFI, though. If such a system emerged, it would need to
be quirked.
Co-developed-by: Chen Yu <[email protected]>
Signed-off-by: Chen Yu <[email protected]>
Signed-off-by: Zhang Rui <[email protected]>
Reviewed-by: Ricardo Neri <[email protected]>
Link: https://patch.msgid.link/[email protected]
[ rjw: Changelog edits ]
Signed-off-by: Rafael J. Wysocki <[email protected]>
Diffstat (limited to 'drivers/fpga/fpga-bridge.c')
0 files changed, 0 insertions, 0 deletions