Skip to content

DLM: dlm patches need review

David Teigland cluster-devel 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest0 of 18 patches 0 repliesNew

Patches in v1 No applies recorded yet

None of the 18 patches the cover letter announces are in this archive.

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
Quoted 35 lines · Steven Whitehouse

Hi,

Quoted 17 lines · tsutomu.owa at toshiba.co.jp

Hi,

This series of patches (2nd version after previous review on August) is to fix various bugs. This patch set is against the mainline kernel. We'd like reviewed to make sure those changes are fine.

Patch number 01/02/03 were posted on this mailing list.
However, I've modifed 03 to correct a bug that aquires rwlock multiple times. Other than that, there is no changes.

01:
https://www.redhat.com/archives/cluster-devel/2017-April/msg00021.html
02:
https://www.redhat.com/archives/cluster-devel/2017-August/msg00076.html
03:
https://www.redhat.com/archives/cluster-devel/2017-April/msg00023.html

Quoted 4 lines

thanks in advance.
-- owa

Many thanks for sending the updated patches. Since the merge window is currently open, we are at a good point to look at these, since there is maximum time before the next merge window. Perhaps one way to deal with these patches would be to take the ones that we are most confident in now (of course that could be all of them!) and postpone any that we think might need more review until later. That way it should be easier to review the more complicated patches and we get the benefit of the easier to review patches sooner.

Quoted 1 line

Dave, Bob, does that make sense?

Yes, they look good to me, I expect to push most or all of them into the next branch by the end of the week. If questions remain on any individual patches, please post or let me know.