Re: port-amd64/54147 (device trouble (failed to map RSDT, mapping interrupts fail))

[Mark Davies <mark%ecs.vuw.ac.nz@localhost>]

The following reply was made to PR port-amd64/54147; it has been noted by GNATS.

From: Mark Davies <mark%ecs.vuw.ac.nz@localhost>
To: Maxime Villard <max%m00nbsd.net@localhost>
Cc: gnats-bugs%NetBSD.org@localhost
Subject: Re: port-amd64/54147 (device trouble (failed to map RSDT, mapping
 interrupts fail))
Date: Sun, 19 May 2019 14:29:36 +1200

 On 18/05/19 7:57 pm, Maxime Villard wrote:
 > Le 05/05/2019 à 10:27, Maxime Villard a écrit :
 >> How about this. No need to create entries if we know we won't
 >> internalize them.
 >> See x86_parse_clusters(), we're only interested in BIM_Memory,
 >> BIM_ACPI and
 >> BIM_NVS.
 >>
 >> Index: efi.c
 >> ===================================================================
 >> RCS file: /cvsroot/src/sys/arch/x86/x86/efi.c,v
 >> retrieving revision 1.19
 >> diff -u -r1.19 efi.c
 >> --- efi.c    3 Dec 2018 19:46:43 -0000    1.19
 >> +++ efi.c    5 May 2019 08:21:34 -0000
 >> @@ -517,6 +517,16 @@
 >> Â Â Â Â Â Â Â Â Â  size = md->md_pages * EFI_PAGE_SIZE;
 >> Â Â Â Â Â Â Â Â Â  type = efi_getbiosmemtype(md->md_type, md->md_attr);
 >>
 >> +Â Â Â Â Â Â Â  /* If the segment is not memory, skip it. */
 >> +Â Â Â Â Â Â Â  switch (type) {
 >> +Â Â Â Â Â Â Â  case BIM_Memory:
 >> +Â Â Â Â Â Â Â  case BIM_ACPI:
 >> +Â Â Â Â Â Â Â  case BIM_NVS:
 >> +Â Â Â Â Â Â Â Â Â Â Â  break;
 >> +Â Â Â Â Â Â Â  default:
 >> +Â Â Â Â Â Â Â Â Â Â Â  continue;
 >> +Â Â Â Â Â Â Â  }
 >> +
 >> Â Â #ifdef DEBUG_MEMLOAD
 >> Â Â Â Â Â Â Â Â Â  printf("MEMMAP: p0x%016" PRIx64 "-0x%016" PRIx64
 >> Â Â Â Â Â Â Â Â Â Â Â Â Â  ", v0x%016" PRIx64 "-0x%016" PRIx64
 > 
 > So what's up?
 
 I tried this patch and got no change.
 
 Also as a test bumped VM_PHYSSEG_MAX up to 512 and it made no difference
 
 I did also define DEBUG_MEMLOAD so would have expected to see some
 additional output but didn't so I might be missing something.
 
 cheers
 mark