Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > Quoting Mika Kuoppala (2018-01-15 12:04:40) >> Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: >> >> > While we talk to the punit over its sideband, we need to prevent the cpu >> > from sleeping in order to prevent a potential machine hang. >> > >> > Note that by itself, it appears that pm_qos_update_request (via >> > intel_idle) doesn't provide a sufficient barrier to ensure that all core >> > are indeed awake (out of Cstate) and that the package is awake. To do so, >> > we need to supplement the pm_qos with a manual ping on_each_cpu. >> > >> > v2: Restrict the heavy-weight wakeup to just the ISOF_PORT_PUNIT, there >> > is insufficient evidence to implicate a wider problem atm. Similarly, >> > restrict the w/a to Valleyview, as Cherryview doesn't have an angry cadre >> > of users. >> > >> >> One datapoint about the v1 of this patch with the cpu ping in it. >> The j1900 byt did end up with system hang during weekend with >> drm-tip + v1. > > Question: did you set CONFIG_I2C_DESIGNWARE_BAYTRAIL=y? I do have: CONFIG_I2C_DESIGNWARE_BAYTRAIL=y -Mika _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
