Messages in this thread | | | Subject | Re: [tip:locking/core] rwsem: Add comments to explain the meaning of the rwsem's count field | From | Davidlohr Bueso <> | Date | Mon, 05 May 2014 19:30:37 -0700 |
| |
On Mon, 2014-05-05 at 15:51 -0700, Tim Chen wrote: > On Mon, 2014-05-05 at 20:27 +0200, Ingo Molnar wrote: > > > > Ingo, > > > > > > The delta patch is included below. Thinking a bit more, > > > the state diagram approach is not necessarily less verbose > > > because the state is a tuple (count, wait queue state). > > > After enumerating the states, we may wind up with very similar > > > to what I have. > > > > Could we at least try with one diagram and see how it goes? > > > > I've tried (see below). But I don't like how it came out :(
And quite nice, thanks for doing this. Personally, however, I much prefer the already applied patch to this approach.
Thanks, Davidlohr
| |