Re: lockmgr() api removal

2016-06-12 Thread Christian Weisgerber
On 2016-06-08, Christian Weisgerber wrote: >> That's still testing server side for the contents of the ports tree, >> isn't it? Not as heavily stressed as putting it on the server would >> be, but it still gives it a bit of a workout. > > I have now put the patch on the central server in the pack

Re: lockmgr() api removal

2016-06-08 Thread Christian Weisgerber
On 2016-06-08, Stuart Henderson wrote: > That's still testing server side for the contents of the ports tree, > isn't it? Not as heavily stressed as putting it on the server would > be, but it still gives it a bit of a workout. I have now put the patch on the central server in the package buildi

Re: lockmgr() api removal

2016-06-08 Thread Stuart Henderson
On 2016/06/08 14:48, Christian Weisgerber wrote: > On 2016-06-07, Theo de Raadt wrote: > > > Did I miss a report about nfs server and client? I know I have not tested > > this diff. > > I put it on the amd64 package building machines and ran two bulk > builds with it. That qualifies as a succe

Re: lockmgr() api removal

2016-06-08 Thread Christian Weisgerber
On 2016-06-07, Theo de Raadt wrote: > Did I miss a report about nfs server and client? I know I have not tested > this diff. I put it on the amd64 package building machines and ran two bulk builds with it. That qualifies as a successful client test. I haven't gotten around yet to testing the

Re: lockmgr() api removal

2016-06-07 Thread Theo de Raadt
> Martin Natano wrote: > > It is time for the lockmgr() api to die. The api is only used by > > filesystems, where it is a trivial change to use rrw locks instead. All > > it needs is LK_* defines for the RW_* flags. (See the sys/lock.h hunk in > > the diff below.) > > > > The ffs regress tests di

Re: lockmgr() api removal

2016-06-07 Thread Ted Unangst
Martin Natano wrote: > It is time for the lockmgr() api to die. The api is only used by > filesystems, where it is a trivial change to use rrw locks instead. All > it needs is LK_* defines for the RW_* flags. (See the sys/lock.h hunk in > the diff below.) > > The ffs regress tests display the same

Re: lockmgr() api removal

2016-05-31 Thread Theo de Raadt
> Martin Natano wrote: > > It is time for the lockmgr() api to die. The api is only used by > > filesystems, where it is a trivial change to use rrw locks instead. All > > it needs is LK_* defines for the RW_* flags. (See the sys/lock.h hunk in > > the diff below.) > > > > The ffs regress tests di

Re: lockmgr() api removal

2016-05-31 Thread Ted Unangst
Martin Natano wrote: > It is time for the lockmgr() api to die. The api is only used by > filesystems, where it is a trivial change to use rrw locks instead. All > it needs is LK_* defines for the RW_* flags. (See the sys/lock.h hunk in > the diff below.) > > The ffs regress tests display the same

lockmgr() api removal

2016-05-29 Thread Martin Natano
It is time for the lockmgr() api to die. The api is only used by filesystems, where it is a trivial change to use rrw locks instead. All it needs is LK_* defines for the RW_* flags. (See the sys/lock.h hunk in the diff below.) The ffs regress tests display the same number of fail/ok results before