On Thu, 2023-03-23 at 08:35 +0000, Tvrtko Ursulin wrote:
On 23/03/2023 00:27, Teres Alexis, Alan Previn wrote:On Fri, 2023-03-17 at 13:37 +0200, Tamminen, Eero T wrote:Hi,On 16.3.2023 10.50, Tvrtko Ursulin wrote:[ 11.674183] i915 0000:00:02.0: PXP init-arb-session-15 failed dueto BIOS/SOC:0x0000101a:ERR_PLATFORM_CONFIG...Alan - is this expected during normal operation on some parts, or it'ssomething truly unexpected/unexplained? If the former then I think itwould be good to downgrade away from drm_WARN so it is less scary.Commit message talks about "HW/platform gaps" - if it is like a missingBIOS support or so then I think WARN_ON is too much.Note that this was on pre-production TGL-H HW with BIOS from April 2021.(I don't know where to get update, nor interested to update it.)- EeroAlan: Hi Tvrtko, thanks for the feedback -i shall change from WARN_ONCE to drm_info_once.Maybe it deserves to be a warning? Or a notice? I was just thinking itdoes not need a call trace and all since it is not a driver issue. Yourcall on the level and whether or not there is any chance for it tohappen in the field to make the discussion relevant or not.
I agree we don't need a call trace. I can't say how often this can happen in the field as it would depend on
when the platform was purchased, whether the owner had been updating the firwmware releases and whether
the bios settings were configured correctly. I'm okay with the drm_info_once - it could be ignored which is fine.
If customers seeing protected rendering/decoding fail when they actually DO need it, I am expecting they will
provide the dmesg that shows above new message at compositor startup (mesa attempting to establish caps)
along with more serious complaints such as their application getting a failure when attempting to create a
protected context via I915 ioctl (which aligns with existing PXP UAPI behavior).
Any chance for an r-b? :)
