Date: Wed, 21 Jan 2015 18:04:57 -0700 From: Warner Losh <imp@bsdimp.com> To: Konstantin Belousov <kostikbel@gmail.com> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, Nathan Whitehorn <nwhitehorn@freebsd.org> Subject: Re: svn commit: r277421 - head/sys/powerpc/powerpc Message-ID: <81FEE3A9-C522-48A3-9292-893685C640AF@bsdimp.com> In-Reply-To: <20150121075410.GM42409@kib.kiev.ua> References: <201501201621.t0KGLx26096724@svn.freebsd.org> <20150120191438.GL42409@kib.kiev.ua> <54BF240C.2090107@freebsd.org> <20150121075410.GM42409@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
> On Jan 21, 2015, at 12:54 AM, Konstantin Belousov = <kostikbel@gmail.com> wrote: >=20 > On Tue, Jan 20, 2015 at 07:59:08PM -0800, Nathan Whitehorn wrote: >>=20 >> 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 >>>>=20 >>>> Log: >>>> There does not seem to be any reason to acquire GIANT here. = Follow amd64 >>>> in removing it. >>>>=20 >>>> MFC after: 1 month >>>>=20 >>>> Modified: >>>> head/sys/powerpc/powerpc/mem.c >>>>=20 >>>> Modified: head/sys/powerpc/powerpc/mem.c >>>> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >>>> --- 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 =3D 0; >>>> error =3D 0; >>>>=20 >>>> - 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. >>>=20 >> Just so I understand, you are not objecting to this commit, right? > Absolutely not, this is the right thing to do. >=20 >> Just=20 >> pointing out that (a) my commit message was wrong and that (b) once = all=20 >> architectures make this change (presumably more involved) we can get = rid=20 >> of the D_NEEDGIANT in /sys/dev/mem/memdev.c? > Exactly. There doesn=E2=80=99t seem to be a reason for i386 either. Was just = looking at the code today on the plane for unrelated reasons. Warner
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?81FEE3A9-C522-48A3-9292-893685C640AF>