Re: bl_list and lockdep

2021-04-13 Thread Thomas Gleixner
Dave, On Tue, Apr 13 2021 at 19:58, Dave Chinner wrote: > On Tue, Apr 13, 2021 at 01:18:35AM +0200, Thomas Gleixner wrote: > So for solving the inode cache scalability issue with RT in mind, > we're left with these choices: > > a) increase memory consumption and cacheline misses for everyone by >

Re: bl_list and lockdep

2021-04-13 Thread Dave Chinner
On Tue, Apr 13, 2021 at 01:18:35AM +0200, Thomas Gleixner wrote: > Dave, > > On Tue, Apr 13 2021 at 08:15, Dave Chinner wrote: > > On Mon, Apr 12, 2021 at 05:20:53PM +0200, Thomas Gleixner wrote: > >> On Wed, Apr 07 2021 at 07:22, Dave Chinner wrote: > >> > And, FWIW, I'm also aware of the

Re: bl_list and lockdep

2021-04-12 Thread Thomas Gleixner
Dave, On Tue, Apr 13 2021 at 08:15, Dave Chinner wrote: > On Mon, Apr 12, 2021 at 05:20:53PM +0200, Thomas Gleixner wrote: >> On Wed, Apr 07 2021 at 07:22, Dave Chinner wrote: >> > And, FWIW, I'm also aware of the problems that RT kernels have with >> > the use of bit spinlocks and being unable

Re: bl_list and lockdep

2021-04-12 Thread Dave Chinner
On Mon, Apr 12, 2021 at 05:20:53PM +0200, Thomas Gleixner wrote: > Dave, > > On Wed, Apr 07 2021 at 07:22, Dave Chinner wrote: > > On Tue, Apr 06, 2021 at 02:28:34PM +0100, Matthew Wilcox wrote: > >> On Tue, Apr 06, 2021 at 10:33:43PM +1000, Dave Chinner wrote: > >> > +++ b/fs/inode.c > >> > @@

Re: bl_list and lockdep

2021-04-12 Thread Thomas Gleixner
Dave, On Wed, Apr 07 2021 at 07:22, Dave Chinner wrote: > On Tue, Apr 06, 2021 at 02:28:34PM +0100, Matthew Wilcox wrote: >> On Tue, Apr 06, 2021 at 10:33:43PM +1000, Dave Chinner wrote: >> > +++ b/fs/inode.c >> > @@ -57,8 +57,7 @@ >> > >> > static unsigned int i_hash_mask __read_mostly; >> >

Re: bl_list and lockdep

2021-04-06 Thread Dave Chinner
On Tue, Apr 06, 2021 at 02:28:34PM +0100, Matthew Wilcox wrote: > On Tue, Apr 06, 2021 at 10:33:43PM +1000, Dave Chinner wrote: > > +++ b/fs/inode.c > > @@ -57,8 +57,7 @@ > > > > static unsigned int i_hash_mask __read_mostly; > > static unsigned int i_hash_shift __read_mostly; > > -static

bl_list and lockdep

2021-04-06 Thread Matthew Wilcox
On Tue, Apr 06, 2021 at 10:33:43PM +1000, Dave Chinner wrote: > +++ b/fs/inode.c > @@ -57,8 +57,7 @@ > > static unsigned int i_hash_mask __read_mostly; > static unsigned int i_hash_shift __read_mostly; > -static struct hlist_head *inode_hashtable __read_mostly; > -static