cvs commit: src/sys/dev/acpica acpi_pci.c src/sys/dev/pci pci.c

[John Baldwin]

On Tuesday 27 April 2004 09:31 am, Ken Smith wrote:
> On Tue, Apr 27, 2004 at 09:12:47AM -0400, John Baldwin wrote:
> > Unfortunately, the PR number is wrong (that's some ports PR) and there
> > are no PR's in the database with an originator that contains 'Remesov' or
> > the text 'acpi_pci' in the text fields at least none that
> > query-pr-summary.cgi can find.  Ugh, actually, query-pr is just hosed as
> > it thinks we have no open PRs.  *sigh*
>
> freefall 13 % query-pr --originator="Remesov"
>
> >Number:         65964
> >Category:       kern
> >Synopsis:       Critical typos in setting power state routines
> >Confidential:   no
> >Severity:       critical
> >Priority:       high
> >Responsible:    imp
> >State:          closed
> >Class:          sw-bug
> >Quarter:
> >Keywords:
> >Date-Required:
> >Submitter-Id:   current-users
> >Arrival-Date:   Sun Apr 25 13:10:21 PDT 2004
> >Closed-Date:    Sun Apr 25 19:12:59 PDT 2004
> >Last-Modified:  Sun Apr 25 19:12:59 PDT 2004
> >Originator:     Dmitry Remesov
> >Release:        5.2-CURRENT
>
> Running 'query-pr --state=open' returns a *lot* of PR's.  Did I just
> try this at a better time than you did?  :-)

Web interface searching doesn't and query open reports doesn't seem to be 
working.  I did manage to typo the PR number and not notice it when I double 
checked though, and the actual patch is correct in that there was a typo.  
Now to make sure that it isn't backwards. :)  (The typo was my bad).  Hmm, I 
just need my morning caffeine I guess.  I must have misread the commit 
message, but the actual patch is correct (PCI then ACPI for power down, ACPI 
then PCI for power up).

-- 
John Baldwin <jhb at FreeBSD.org>  <><  http://www.FreeBSD.org/~jhb/
"Power Users Use the Power to Serve"  =  http://www.FreeBSD.org