Re: [rfc] /dev/devstat permissions patch

2014-03-20 Thread Maksim Yevmenkin
On Thu, Mar 20, 2014 at 7:05 AM, John Baldwin wrote: > On Tuesday, March 18, 2014 3:29:32 pm Maksim Yevmenkin wrote: >> hello, >> >> would anyone object to the following patch? > > I think this is fine. > > While you are at it, can you test this patch to remove D_NEEDGIANT? > > Index: subr_devstat

Re: [rfc] /dev/devstat permissions patch

2014-03-20 Thread John Baldwin
On Tuesday, March 18, 2014 3:29:32 pm Maksim Yevmenkin wrote: > hello, > > would anyone object to the following patch? I think this is fine. While you are at it, can you test this patch to remove D_NEEDGIANT? Index: subr_devstat.c

Re: [rfc] /dev/devstat permissions patch

2014-03-18 Thread Xin Li
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Hi, Adding phk@ to cc since the 400 is from his changeset (r112001). On 03/18/14 12:29, Maksim Yevmenkin wrote: > hello, > > would anyone object to the following patch? > > == > > Index: subr_devstat.c >

[rfc] /dev/devstat permissions patch

2014-03-18 Thread Maksim Yevmenkin
hello, would anyone object to the following patch? == Index: subr_devstat.c === --- subr_devstat.c (revision 263311) +++ subr_devstat.c (working copy) @@ -503,7 +503,7 @@ mtx_assert(&devstat_mutex, MA_NOTOWNED); if (!once) { m