[Bug 285007] Warning message about decoded bus range in acpi_pcib_acpi_attach doesn't seem to match code/comments

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=285007

Mark Johnston <markj@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|New                         |Open
                 CC|                            |imp@FreeBSD.org,
                   |                            |jhb@FreeBSD.org,
                   |                            |markj@FreeBSD.org

--- Comment #2 from Mark Johnston <markj@FreeBSD.org> ---
The check in question:

438             /*                                                              
439              * If there is a decoded bus range, assume the bus number is    
440              * the first value in the range.  Warn if _BBN doesn't match.   
441              */                                                             
442             if (get_decoded_bus_range(sc, &start, &end)) {                  
443                     if (sc->ap_bus != start) {                              
444                             device_printf(dev,                              
445                                 "WARNING: BIOS configured bus number (%d)
is "                                                                            
446                                 "not within decoded bus number range "      
447                                 "(%ju - %ju).\n",                           
448                                 sc->ap_bus, (uintmax_t)start,
(uintmax_t)end);                                                                
449                             device_printf(dev,                              
450                                 "Using range start (%ju) as bus number.\n", 
451                                 (uintmax_t)start);                          
452                             sc->ap_bus = start;                             
453                     }                                                       
454             }

So really we're checking if the bus number is the first value in that range. 
It's not clear to me which of the warning or the check is correct.

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