Re: svn commit: r323393 - in head/sys: sys vm

2017-09-22 Thread Mateusz Guzik
On Thu, Sep 14, 2017 at 12:32 AM, Gleb Smirnoff wrote: > On Mon, Sep 11, 2017 at 09:30:10AM +0200, Mateusz Guzik wrote: > M> First, there is a bunch of counter(9) fields. I don't know the original > M> reasoning. I would expect these counters to be statically defined in a > M> per-cpu struct. > >

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-13 Thread Gleb Smirnoff
On Mon, Sep 11, 2017 at 09:30:10AM +0200, Mateusz Guzik wrote: M> First, there is a bunch of counter(9) fields. I don't know the original M> reasoning. I would expect these counters to be statically defined in a M> per-cpu struct. The reasoning was to remove 'struct vmmeter' from the 'struct pcpu'

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-11 Thread Bruce Evans
On Mon, 11 Sep 2017, Mateusz Guzik wrote: On Mon, Sep 11, 2017 at 10:24 AM, Ryan Libby wrote: On Mon, Sep 11, 2017 at 12:30 AM, Mateusz Guzik wrote: [...] That said, looking now at the struct I think its use should be retired from the kernel. It can remain in headers for userspace use. Mo

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-11 Thread Mateusz Guzik
On Mon, Sep 11, 2017 at 10:24 AM, Ryan Libby wrote: > On Mon, Sep 11, 2017 at 12:30 AM, Mateusz Guzik wrote: > [...] > > That said, looking now at the struct I think its use should be retired > > from the kernel. It can remain in headers for userspace use. > > > > First, there is a bunch of coun

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-11 Thread Ryan Libby
On Mon, Sep 11, 2017 at 12:30 AM, Mateusz Guzik wrote: [...] > That said, looking now at the struct I think its use should be retired > from the kernel. It can remain in headers for userspace use. > > First, there is a bunch of counter(9) fields. I don't know the original > reasoning. I would expe

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-11 Thread Mateusz Guzik
On Mon, Sep 11, 2017 at 11:56:21AM +1000, Bruce Evans wrote: > On Sun, 10 Sep 2017, Mateusz Guzik wrote: > > > Log: > > Move vmmeter atomic counters into dedicated cache lines > > > > Prior to the change they were subject to extreme false sharing. > > In particular this change shaves about 3 sec

Re: svn commit: r323393 - in head/sys: sys vm

2017-09-10 Thread Bruce Evans
On Sun, 10 Sep 2017, Mateusz Guzik wrote: Log: Move vmmeter atomic counters into dedicated cache lines Prior to the change they were subject to extreme false sharing. In particular this change shaves about 3 seconds real time of -j 80 buildkernel. Changes that small are hard to measure.

svn commit: r323393 - in head/sys: sys vm

2017-09-10 Thread Mateusz Guzik
Author: mjg Date: Sun Sep 10 19:00:38 2017 New Revision: 323393 URL: https://svnweb.freebsd.org/changeset/base/323393 Log: Move vmmeter atomic counters into dedicated cache lines Prior to the change they were subject to extreme false sharing. In particular this change shaves about 3 secon