[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [BUG]i2c_hid_acpi broken with 4.17.2 on Framework Laptop 13 AMD
- To: Sébastien Chaumat <euidzero@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
- From: Mario Limonciello <mario.limonciello@xxxxxxx>
- Date: Wed, 6 Mar 2024 11:33:18 -0600
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=6SCUk1blSDm4CBP4y0a2QP95WF0lNlN6ALMTMkwD85c=; b=EwkaDBXQq2bdOW4+MfidivGB0xnOgEtlvfcuPKaJpZOGZuCJU6BPwKjGzgTNW00rh09ezxPyQ8gf+VWFb2lOe/nRZH7wI4YSLQvaZzOf/2EoSDszaGHe9GOdjwn1mYF0tlWpwRR78r5p0nKhcxkAnVk+95nwMfxNFJVmwnRQYo5Zvj/fzfAo3pvD9UwPFOR8UmJcQaZPNafJaeyBghWEUUTXx+7Hlvyt6wzpS1K6J0Ah7V1UHljJ5vWCWHgbUhhSKETKjsMArU0bblSrYtCKuyshrfkmvLrrDPXDe33nmb5tgAaXoycutBVygfoA/3HXIay13xjvki9GjaQTMXpiIA==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Dw9V9IpULwYHSjD7nn46QrROZ2tl9Mm6dz2/PJKPrg2Jg04WcJdFOyv/sp9t9icrlltdYdlO3n2mWc2pP/di3xm6oocSJsMhGOmmVbpNBCg5tra2qgJIOHFg4t3Zr3Y6WiN/cqvFnUUlk0rcjN5u5HAs7cpwL+OY/p4NdTb70MijZX0lICZ4QDLS3FvQxpWOI0PQn1usrMckqfoMd1gxqN+lOmGRZhV145DU2ykMWJVu9v9Ajo3Yd8m6UawGoSeKeGFBm35jvqGDutfqpoDU2kno9ARG80uvD2Ycm95DQJ9e1sfSDWkJfSq5lEmlJyGEaDntUflx3es+LKfErCEouQ==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com;
- Cc: Jan Beulich <jbeulich@xxxxxxxx>, Juergen Gross <jgross@xxxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Wed, 06 Mar 2024 17:33:33 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 3/6/2024 11:28, Sébastien Chaumat wrote:
Reasoning backward (using a kernel without the pinctrl_amd driver
to ensure xen only is at stake) :
checking the diff in IOAPIC between bare metal and xen (IRQ7 is
on pin07 on APIC )
using kernel argument : apic=debug
bare metal :
[ 0.715330] fedora kernel: ... APIC VERSION: 81050010
...
[ 0.715433] fedora kernel: pin07, disabled, edge , high, V(00),
IRR(0), S(0), physical, D(0000), M(0)
xen :
[ 2.249582] fedora kernel: ... APIC VERSION: 00000014
...
[ 2.249730] fedora kernel: pin07, disabled, level, low , V(60),
IRR(0), S(0), physical, D(0000), M(0)
So the APIC table is not the same.
As strange as it looks the (IOAPIC 0) pin07 is correctly described
by the APIC in xen but yet differently than in baremetal.
But the APIC message comes long after the
[ 1.833145] fedora kernel: xen: registering gsi 7 triggering 0
polarity 1
so I wonder if the APIC pin07 info had any influence.
Finally found the fix : adding ioapic_ack=new to xen boot parameters.
Not only the trackpad is now working but also the ACPI Embedded
Controller which is completely disabled.
Sébastien
That's great news! I'm personally totally unfamiliar with
ioapic_ack=new, so I did a quick search and found out it's a Xen
parameter (I came across
https://xenbits.xen.org/docs/4.5-testing/misc/xen-command-line.html).
This mentions that "new" should be the default, so why isn't it the case?
Also; I'd be really interested to hear what happens with s2idle with Xen
now (if it works).
|