Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-22 Thread Konstantin Belousov
On Wed, Jan 21, 2015 at 06:04:57PM -0700, Warner Losh wrote: > > > On Jan 21, 2015, at 12:54 AM, Konstantin Belousov > > wrote: > > > > On Tue, Jan 20, 2015 at 07:59:08PM -0800, Nathan Whitehorn wrote: > >> > >> On 01/20/15 11:14, Konstantin Belousov wrote: > >>> On Tue, Jan 20, 2015 at 04:21:

Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-21 Thread Warner Losh
> On Jan 21, 2015, at 12:54 AM, Konstantin Belousov wrote: > > On Tue, Jan 20, 2015 at 07:59:08PM -0800, Nathan Whitehorn wrote: >> >> On 01/20/15 11:14, Konstantin Belousov wrote: >>> On Tue, Jan 20, 2015 at 04:21:59PM +, Nathan Whitehorn wrote: Author: nwhitehorn Date: Tue Jan 2

Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-20 Thread Konstantin Belousov
On Tue, Jan 20, 2015 at 07:59:08PM -0800, Nathan Whitehorn wrote: > > On 01/20/15 11:14, Konstantin Belousov wrote: > > On Tue, Jan 20, 2015 at 04:21:59PM +, Nathan Whitehorn wrote: > >> Author: nwhitehorn > >> Date: Tue Jan 20 16:21:59 2015 > >> New Revision: 277421 > >> URL: https://svnweb.f

Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-20 Thread Nathan Whitehorn
On 01/20/15 11:14, Konstantin Belousov wrote: On Tue, Jan 20, 2015 at 04:21:59PM +, Nathan Whitehorn wrote: Author: nwhitehorn Date: Tue Jan 20 16:21:59 2015 New Revision: 277421 URL: https://svnweb.freebsd.org/changeset/base/277421 Log: There does not seem to be any reason to acquire G

Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-20 Thread John Baldwin
On Tuesday, January 20, 2015 2:14:38 pm Konstantin Belousov wrote: > This is not an acquisition, to be pedantic. Really, it is cdevsw which > has D_NEEDGIANT flag which acquires Giant. After architectures get > rid of GIANT_REQUIRED, flag can be removed. We could #ifdef the flag, but only if it

Re: svn commit: r277421 - head/sys/powerpc/powerpc

2015-01-20 Thread Konstantin Belousov
On Tue, Jan 20, 2015 at 04:21:59PM +, Nathan Whitehorn wrote: > Author: nwhitehorn > Date: Tue Jan 20 16:21:59 2015 > New Revision: 277421 > URL: https://svnweb.freebsd.org/changeset/base/277421 > > Log: > There does not seem to be any reason to acquire GIANT here. Follow amd64 > in removi