Commit 5a5e0aac authored by Ming Huang's avatar Ming Huang
Browse files

fix(gicv3): add dsb in both disable and enable function of gicv3_cpuif



A RAS error may be triggered while offline core in OS. Error:
Uncorrected software error in the Distributor, with IERR=9,SERR=f.
Core put to sleep before its Group enables were cleared.

gicv3_cpuif_disable() will be called in offline core flow.
According to GIC architecture version 3 and version 4:
Architectural execution of a DSB instruction guarantees that
the last value written to ICC_IGRPEN0_EL1, ICC_IGRPEN1_EL1,
ICC_IGRPEN1_EL3 or GICC_CTLR.{EnableGrp0, EnableGrp1}is observed
by the associated Redistributor.
An ISB or other context synchronization operation must precede
the DSB to ensure visibility of System register writes.
Signed-off-by: default avatarMing Huang <huangming@linux.alibaba.com>
Change-Id: Iff1475657f401374c761b5e8f2f5b3a4b2040e9d
parent c7911137
...@@ -332,6 +332,8 @@ void gicv3_cpuif_enable(unsigned int proc_num) ...@@ -332,6 +332,8 @@ void gicv3_cpuif_enable(unsigned int proc_num)
write_icc_igrpen1_el3(read_icc_igrpen1_el3() | write_icc_igrpen1_el3(read_icc_igrpen1_el3() |
IGRPEN1_EL3_ENABLE_G1S_BIT); IGRPEN1_EL3_ENABLE_G1S_BIT);
isb(); isb();
/* Add DSB to ensure visibility of System register writes */
dsb();
} }
/******************************************************************************* /*******************************************************************************
...@@ -363,6 +365,8 @@ void gicv3_cpuif_disable(unsigned int proc_num) ...@@ -363,6 +365,8 @@ void gicv3_cpuif_disable(unsigned int proc_num)
/* Synchronise accesses to group enable registers */ /* Synchronise accesses to group enable registers */
isb(); isb();
/* Add DSB to ensure visibility of System register writes */
dsb();
/* Mark the connected core as asleep */ /* Mark the connected core as asleep */
gicr_base = gicv3_driver_data->rdistif_base_addrs[proc_num]; gicr_base = gicv3_driver_data->rdistif_base_addrs[proc_num];
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment