Re: [PATCH 2/4] soc: qcom: stats: don't crash if DDR offset contains invalid data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 9.12.2023 22:55, Dmitry Baryshkov wrote:
> The stats ram on sm8150 platform contains invalid data at the
> DDR_DYNAMIC_OFFSET. Most likely this is because the platform didn't
> support DDR sleep stats.
Interesting. Can you read back DDR_DYNAMIC_OFFSET on 8350/8280 and
see if 8150 has correct data in there?

> However this platform uses generic
> "qcom,rpmh-stats" compatible, which implies presense of the DDR data.
> Add safety net to prevent old DTB files from crashing the
> qcom,rpmh-stats driver.
Yeah I'dve never thought there would be garbage in there..

I'd advocate for simply not doing anything wrt sleep stats if DDR
stats are unavailable though. The QMP handle can stay, as there
may (I don't know) be more data available that we want to export
through this driver.

Konrad




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux