On 12/6/2022 03:06, Tvrtko Ursulin wrote:
On 05/12/2022 18:44, Michal Wajdeczko wrote:
On 05.12.2022 14:16, Tvrtko Ursulin wrote:
On 02/12/2022 20:14, John Harrison wrote:
[snip]
Random meaningless (to me) message that is apparently a display thing:
drm_dbg_kms(&dev_priv->drm, "disabling %s\n", pll->info->name);
i915 0000:00:02.0: [drm:intel_disable_shared_dpll [i915]] disabling
PORT PLL B
Plan is to not touch outside gt/.
For some unexplicable reason that means it is almost impossible to see
the actual problems in most CI dmesg logs because they are swamped with
irrelevant display messages that cannot be filtered out. For example, I
recently manually grep'd out all the display spam from a bug report log.
The dmesg file went from 12MB to 700KB. That is a significant problem
that makes bug triage way harder than it needs to be.
Maybe as a way forward work could be split? If John wants to deal with
gt_xxx macros, avoid touching GuC (putting his original motivation
aside) and you want to convert the gt/uc folder? Assuming John you are
okay with "GuC:" and "CT:" prefixes.
Meaning just repost patch #1 only and expand to more intel_gt_* files?
Sure, if someone will actually reply to that patch with some kind of r-b
first so I know I'm not still wasting my time on a huge re-write that
will to be redone multiple times when someone objects to the use of a
colon or the lack of spaces, braces or whatever.
John.
