Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread John Baldwin
On Tuesday, November 18, 2014 10:29:52 PM Chagin Dmitry wrote: > On Tue, Nov 18, 2014 at 07:47:46AM -0800, Adrian Chadd wrote: > > Hi, > > > > Want to post a bootverbose suspend/resume pass with -HEAD without this > > patch in it, then with this patch in it? > > it seems that at resume path METHO

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Andriy Gapon
On 19/11/2014 20:19, Adrian Chadd wrote: > On 19 November 2014 10:17, Andriy Gapon wrote: >> On 19/11/2014 19:46, Adrian Chadd wrote: >>> On 19 November 2014 08:53, Andriy Gapon wrote: On 19/11/2014 18:15, Adrian Chadd wrote: > On 19 November 2014 03:17, Andriy Gapon wrote: >> On 18

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Adrian Chadd
On 19 November 2014 10:17, Andriy Gapon wrote: > On 19/11/2014 19:46, Adrian Chadd wrote: >> On 19 November 2014 08:53, Andriy Gapon wrote: >>> On 19/11/2014 18:15, Adrian Chadd wrote: On 19 November 2014 03:17, Andriy Gapon wrote: > On 18/11/2014 21:29, Chagin Dmitry wrote: >> it s

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Andriy Gapon
On 19/11/2014 19:46, Adrian Chadd wrote: > On 19 November 2014 08:53, Andriy Gapon wrote: >> On 19/11/2014 18:15, Adrian Chadd wrote: >>> On 19 November 2014 03:17, Andriy Gapon wrote: On 18/11/2014 21:29, Chagin Dmitry wrote: > it seems that at resume path METHOD set_powerstate() is not

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Adrian Chadd
On 19 November 2014 08:53, Andriy Gapon wrote: > On 19/11/2014 18:15, Adrian Chadd wrote: >> On 19 November 2014 03:17, Andriy Gapon wrote: >>> On 18/11/2014 21:29, Chagin Dmitry wrote: it seems that at resume path METHOD set_powerstate() is not invoked. fro proper pciX. ie, no "T

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Andriy Gapon
On 19/11/2014 18:15, Adrian Chadd wrote: > On 19 November 2014 03:17, Andriy Gapon wrote: >> On 18/11/2014 21:29, Chagin Dmitry wrote: >>> it seems that at resume path METHOD set_powerstate() is not invoked. fro >>> proper pciX. >>> ie, no "Transition from D3 to D0" prints. i'm trying to understa

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Adrian Chadd
On 19 November 2014 03:17, Andriy Gapon wrote: > On 18/11/2014 21:29, Chagin Dmitry wrote: >> it seems that at resume path METHOD set_powerstate() is not invoked. fro >> proper pciX. >> ie, no "Transition from D3 to D0" prints. i'm trying to understand the reason > > Just a note that I see this a

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-19 Thread Andriy Gapon
On 18/11/2014 21:29, Chagin Dmitry wrote: > it seems that at resume path METHOD set_powerstate() is not invoked. fro > proper pciX. > ie, no "Transition from D3 to D0" prints. i'm trying to understand the reason Just a note that I see this as well. -- Andriy Gapon _

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-18 Thread John Baldwin
On Monday, November 17, 2014 6:17:25 pm Adrian Chadd wrote: > On 17 November 2014 14:45, John Baldwin wrote: > > On Monday, November 17, 2014 08:25:21 PM Dmitry Chagin wrote: > >> Author: dchagin > >> Date: Mon Nov 17 20:25:21 2014 > >> New Revision: 274635 > >> URL: https://svnweb.freebsd.org/cha

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-18 Thread Chagin Dmitry
On Tue, Nov 18, 2014 at 07:47:46AM -0800, Adrian Chadd wrote: > Hi, > > Want to post a bootverbose suspend/resume pass with -HEAD without this > patch in it, then with this patch in it? > it seems that at resume path METHOD set_powerstate() is not invoked. fro proper pciX. ie, no "Transition fr

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-18 Thread Adrian Chadd
Hi, Want to post a bootverbose suspend/resume pass with -HEAD without this patch in it, then with this patch in it? We should likely revert this commit (for correctness) and then dig deeper into what's going on. -adrian On 17 November 2014 22:46, Chagin Dmitry wrote: > On Mon, Nov 17, 2014

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-17 Thread Chagin Dmitry
On Mon, Nov 17, 2014 at 05:45:23PM -0500, John Baldwin wrote: > On Monday, November 17, 2014 08:25:21 PM Dmitry Chagin wrote: > > Author: dchagin > > Date: Mon Nov 17 20:25:21 2014 > > New Revision: 274635 > > URL: https://svnweb.freebsd.org/changeset/base/274635 > > > > Log: > > Use the correct

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-17 Thread Adrian Chadd
On 17 November 2014 14:45, John Baldwin wrote: > On Monday, November 17, 2014 08:25:21 PM Dmitry Chagin wrote: >> Author: dchagin >> Date: Mon Nov 17 20:25:21 2014 >> New Revision: 274635 >> URL: https://svnweb.freebsd.org/changeset/base/274635 >> >> Log: >> Use the correct device as the power_f

Re: svn commit: r274635 - head/sys/dev/pci

2014-11-17 Thread John Baldwin
On Monday, November 17, 2014 08:25:21 PM Dmitry Chagin wrote: > Author: dchagin > Date: Mon Nov 17 20:25:21 2014 > New Revision: 274635 > URL: https://svnweb.freebsd.org/changeset/base/274635 > > Log: > Use the correct device as the power_for_sleep() method > always pass request up to parent b

svn commit: r274635 - head/sys/dev/pci

2014-11-17 Thread Dmitry Chagin
Author: dchagin Date: Mon Nov 17 20:25:21 2014 New Revision: 274635 URL: https://svnweb.freebsd.org/changeset/base/274635 Log: Use the correct device as the power_for_sleep() method always pass request up to parent bridge. Reviewed by: jhb MFC after:1 week xMFC: r274386,r