Date: Wed, 21 Jan 2015 09:54:10 +0200 From: Konstantin Belousov <kostikbel@gmail.com> To: Nathan Whitehorn <nwhitehorn@freebsd.org> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org Subject: Re: svn commit: r277421 - head/sys/powerpc/powerpc Message-ID: <20150121075410.GM42409@kib.kiev.ua> In-Reply-To: <54BF240C.2090107@freebsd.org> References: <201501201621.t0KGLx26096724@svn.freebsd.org> <20150120191438.GL42409@kib.kiev.ua> <54BF240C.2090107@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
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 +0000, 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 removing it. > >> > >> MFC after: 1 month > >> > >> Modified: > >> head/sys/powerpc/powerpc/mem.c > >> > >> Modified: head/sys/powerpc/powerpc/mem.c > >> ============================================================================== > >> --- head/sys/powerpc/powerpc/mem.c Tue Jan 20 15:45:09 2015 (r277420) > >> +++ head/sys/powerpc/powerpc/mem.c Tue Jan 20 16:21:59 2015 (r277421) > >> @@ -100,8 +100,6 @@ memrw(struct cdev *dev, struct uio *uio, > >> cnt = 0; > >> error = 0; > >> > >> - GIANT_REQUIRED; > >> - > > 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. > > > Just so I understand, you are not objecting to this commit, right? Absolutely not, this is the right thing to do. > Just > pointing out that (a) my commit message was wrong and that (b) once all > architectures make this change (presumably more involved) we can get rid > of the D_NEEDGIANT in /sys/dev/mem/memdev.c? Exactly.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150121075410.GM42409>