Re: [PATCH v2] NFSv4: replace seqcount_t with a seqlock_t

2016-10-31 Thread Sebastian Andrzej Siewior
On 2016-10-28 22:24:52 [+], Trond Myklebust wrote: > > diff --git a/fs/nfs/nfs4state.c b/fs/nfs/nfs4state.c > > index 5f4281ec5f72..a442d9867942 100644 > > --- a/fs/nfs/nfs4state.c > > +++ b/fs/nfs/nfs4state.c > > @@ -1497,8 +1498,18 @@ static int nfs4_reclaim_open_state(struct > > nfs4_state_

Re: [PATCH v2] NFSv4: replace seqcount_t with a seqlock_t

2016-10-28 Thread Trond Myklebust
> On Oct 28, 2016, at 17:05, Sebastian Andrzej Siewior > wrote: > > The raw_write_seqcount_begin() in nfs4_reclaim_open_state() bugs me > because it maps to preempt_disable() in -RT which I can't have at this > point. So I took a look at the code. > It the lockdep part was removed in commit abb

[PATCH v2] NFSv4: replace seqcount_t with a seqlock_t

2016-10-28 Thread Sebastian Andrzej Siewior
The raw_write_seqcount_begin() in nfs4_reclaim_open_state() bugs me because it maps to preempt_disable() in -RT which I can't have at this point. So I took a look at the code. It the lockdep part was removed in commit abbec2da13f0 ("NFS: Use raw_write_seqcount_begin/end int nfs4_reclaim_open_state"