Re: Bug 215744 - input from the accelerometer disappeared, regression on amd_sfh on kernel 5.17

From: Basavaraj Natikar
Date: Fri Apr 01 2022 - 15:47:57 EST


Hi,

Committed patch is disabling the interrupt mode and does not cause any
functionality or working issues.

I also cross verified on 3 system and working fine on 5.17 and not able
to reproduce or recreate.

Output on 5.17:

--------------------------------------------------------------

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd# uname -a

Linux amd-HP-ENVY-x360-Convertible-15-eu0xxx 5.17.0+ #5 SMP PREEMPT Fri
Apr 1 23:28:34 IST 2022 x86_64 x86_64 x86_64 GNU/Linux

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd# monitor-sensor

    Waiting for iio-sensor-proxy to appear

+++ iio-sensor-proxy appeared

=== Has accelerometer (orientation: normal)

=== No ambient light sensor

    Accelerometer orientation changed: right-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: right-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: left-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: right-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: left-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: right-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: bottom-up

    Accelerometer orientation changed: normal

    Accelerometer orientation changed: right-up

    Accelerometer orientation changed: normal

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd# dmidecode -s
bios-version

B.16T19

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd# dmidecode -s
system-product-name

HP ENVY x360 Convertible 15-eu0xxx

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd# dmidecode -s
system-manufacturer

HP

root@amd-HP-ENVY-x360-Convertible-15-eu0xxx:/home/amd#

------------------------------------------------

Looks like this is not regression. May be some hardware/firmware bug.

Just curious reverting this patch how it is working just suspecting
firmware undefined behavior.

If possible, please check on other platform/system also if same behavior
occurs.

Could you please provide me platform/system details so that I can check
this behavior?

Thanks,
Basavaraj

On 4/1/2022 1:36 PM, Thorsten Leemhuis wrote:
> Hi, this is your Linux kernel regression tracker.
>
> I noticed a regression report in bugzilla.kernel.org that afaics nobody
> acted upon since it was reported about a week ago, that's why I decided
> to forward it to the lists and all people that seemed to be relevant
> here. It looks to me like this is something for Basavaraj, as it seems
> to be caused by b300667b33b2 ("HID: amd_sfh: Disable the interrupt for
> all command"). But I'm not totally sure, I only looked briefly into the
> details. Or was this discussed somewhere else already? Or even fixed?
>
> To quote from https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.kernel.org%2Fshow_bug.cgi%3Fid%3D215744&data=04%7C01%7CBasavaraj.Natikar%40amd.com%7C9155f6987d45479f721208da13b682f9%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637843972013904577%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=HbQ5LymDwsc94RZhNuzN83%2F0BUplqRUG7bB%2BXymViP0%3D&reserved=0 :
>
>> Marco 2022-03-25 15:22:19 UTC
>>
>> After updating to 5.17, the input from the accelerometer disappeared, completely. No devices available from IIO tree. First bad commit causing it is https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Fcommit%2Fdrivers%2Fhid%2Famd-sfh-hid%2Famd_sfh_pcie.c%3Fid%3Db300667b33b2b5a2c8e5f8f22826befb3d7f4f2b&data=04%7C01%7CBasavaraj.Natikar%40amd.com%7C9155f6987d45479f721208da13b682f9%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637843972013904577%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&sdata=odHA0EIxLb0xpSJJ9ShzgGL0%2BOAJ6tbG68lQe0QOqLY%3D&reserved=0. Reverting this and the the other two on top fixed this. Tried to not revert only the above mentioned commit, but it's still not working.
>>
>> Marco.
> Anyway, to get this tracked:
>
> #regzbot introduced: b300667b33b2b5a2c8e5f8f22826befb3d7f4
> #regzbot from: Marco <rodomar705@xxxxxxxxxxxxxx>
> #regzbot title: input: hid: input from the accelerometer disappeared due
> to changes to amd_sfh
> #regzbot link: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fbugzilla.kernel.org%2Fshow_bug.cgi%3Fid%3D215744&amp;data=04%7C01%7CBasavaraj.Natikar%40amd.com%7C9155f6987d45479f721208da13b682f9%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637843972013904577%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000&amp;sdata=HbQ5LymDwsc94RZhNuzN83%2F0BUplqRUG7bB%2BXymViP0%3D&amp;reserved=0
>
> Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
>
> P.S.: As the Linux kernel's regression tracker I'm getting a lot of
> reports on my table. I can only look briefly into most of them and lack
> knowledge about most of the areas they concern. I thus unfortunately
> will sometimes get things wrong or miss something important. I hope
> that's not the case here; if you think it is, don't hesitate to tell me
> in a public reply, it's in everyone's interest to set the public record
> straight.
>