On Wed, 2025-02-12 at 08:57 -0800, Dave Hansen wrote: > On 2/12/25 05:43, Zhang, Rui wrote: > > I agree. > > adjust_tjmax() contains a list of quirks based on PCI- > > ID/x86_vendor_id/x86_model/x86_stepping. The common problem is that > > all > > the quirks are for Fam6 processors but the family id is not > > checked. So > > the fix is sufficient. In fact, I think it is better to move the > > check > > to the very beginning of adjust_tjmax(). > > Or, heck, just remove the model list. dev_warn_once() if the rdmsr > fails. Who cares about one more line in dmesg? > > Why not do the attached patch? The patch looks good to me. -rui
