Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 01 Mar 2019 18:14:06 +0000
From:      bugzilla-noreply@freebsd.org
To:        acpi@FreeBSD.org
Subject:   [Bug 235876] ACPI related commit r330113 broke Microsoft Hyper-V passthrough device on Generation 2 FreeBSD VM
Message-ID:  <bug-235876-16045-sB7mohgHNb@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-235876-16045@https.bugs.freebsd.org/bugzilla/>
References:  <bug-235876-16045@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D235876

--- Comment #7 from John Baldwin <jhb@FreeBSD.org> ---
So this ASL is definitely using the flag explicitly.  None of them are in
ExtendedAddressSpaces, and some of them appear to be correct (e.g. the Host=
-PCI
bridge is using ResourceProducer in the way the non-arm-specific ACPI Host-=
PCI
bridge driver expects: to list ranges it decodes that are then used by child
devices).  It seems that a couple of the uses of ResourceProducer in this A=
SL
are just wrong though.  First, numbers of times the flag is used vs not use=
d:

> grep ResourceConsumer tx2.asl  | wc -l
      53
> grep ResourceProducer tx2.asl  | wc -l
      11

Of those 11 cases, SB.ECAM is probably a consumer, not a producer, though we
already treat acpi_sysres devices special.

SB.PCI0 resources should be producers.  If the committed patch actually wor=
ked
fully it would have broken booting completely since PCI0 would have claimed=
 all
the resources for all its children blocking all of them from being able to
alloc bars.  We probably skip resources for PCI bridges which is why this m=
ight
still be working.

The URT0 device's memory should be a consumer.  Note that the interrupt
resource is a consumer, and other similar devices use consumer.  This is ju=
st
firmware writer incompetence.

URT1, URT2, and URT3 are same as URT0.

PCI1 is same as PCI0.

So really, the real issue on the Cavium box is that the UART _CRS buffers h=
ave
a typo.  I think the commit should be changed so that we just ignore
'ResourceProducer' on devices with the ARMH0011 _HID but otherwise leave it=
 in
place.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-235876-16045-sB7mohgHNb>