Re: svn commit: r362012 - head/sys/dev/acpica

[John Baldwin]

On 6/10/20 7:39 AM, Ruslan Bukin wrote:
> Author: br
> Date: Wed Jun 10 14:39:54 2020
> New Revision: 362012
> URL: https://svnweb.freebsd.org/changeset/base/362012
> 
> Log:
>   All the ARM Coresight interconnect devices set ResourceProducer on memory
>   resources, ignore it.
>   
>   The devices found in the ARM Neoverse N1 System Development Platform
>   (N1SDP).
>   
>   Sponsored by:   DARPA, AFRL

1) We should perhaps think about adding a quirk table or at least just adding an
   array of ACPI handles which need this quirk as a separate variable perhaps?

2) Given that ARM is a frequent offender here, perhaps we should provide them
   some feedback to see if they can fix this in future ROMs?

-- 
John Baldwin
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

svn commit: r362012 - head/sys/dev/acpica

[Ruslan Bukin]

Author: br
Date: Wed Jun 10 14:39:54 2020
New Revision: 362012
URL: https://svnweb.freebsd.org/changeset/base/362012

Log:
  All the ARM Coresight interconnect devices set ResourceProducer on memory
  resources, ignore it.
  
  The devices found in the ARM Neoverse N1 System Development Platform
  (N1SDP).
  
  Sponsored by: DARPA, AFRL

Modified:
  head/sys/dev/acpica/acpi_resource.c

Modified: head/sys/dev/acpica/acpi_resource.c
==
--- head/sys/dev/acpica/acpi_resource.c Wed Jun 10 14:28:36 2020
(r362011)
+++ head/sys/dev/acpica/acpi_resource.c Wed Jun 10 14:39:54 2020
(r362012)
@@ -484,8 +484,16 @@ acpi_parse_resources(device_t dev, ACPI_HANDLE handle,
 if (acpi_MatchHid(handle, "ARMH0011") != ACPI_MATCHHID_NOMATCH)
arc.ignore_producer_flag = true;
 
-/* ARM Coresight on N1SDP set ResourceProducer on memory resources. */
-if (acpi_MatchHid(handle, "ARMHC500") != ACPI_MATCHHID_NOMATCH)
+/*
+ * ARM Coresight on N1SDP set ResourceProducer on memory resources.
+ * Coresight devices: ETM, STM, TPIU, ETF/ETR, REP, FUN.
+ */
+if (acpi_MatchHid(handle, "ARMHC500") != ACPI_MATCHHID_NOMATCH ||
+acpi_MatchHid(handle, "ARMHC502") != ACPI_MATCHHID_NOMATCH ||
+acpi_MatchHid(handle, "ARMHC979") != ACPI_MATCHHID_NOMATCH ||
+acpi_MatchHid(handle, "ARMHC97C") != ACPI_MATCHHID_NOMATCH ||
+acpi_MatchHid(handle, "ARMHC98D") != ACPI_MATCHHID_NOMATCH ||
+acpi_MatchHid(handle, "ARMHC9FF") != ACPI_MATCHHID_NOMATCH)
arc.ignore_producer_flag = true;
 
 status = AcpiWalkResources(handle, "_CRS", acpi_parse_resource, );
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"