* [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
@ 2017-09-12 8:54 tsutomu.owa
2017-09-13 9:51 ` Steven Whitehouse
2017-09-14 12:47 ` Bob Peterson
0 siblings, 2 replies; 6+ messages in thread
From: tsutomu.owa @ 2017-09-12 8:54 UTC (permalink / raw)
To: cluster-devel.redhat.com
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
thanks in advance.
-- owa
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
2017-09-12 8:54 [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review tsutomu.owa
@ 2017-09-13 9:51 ` Steven Whitehouse
2017-09-13 16:18 ` David Teigland
2017-09-14 12:48 ` Bob Peterson
2017-09-14 12:47 ` Bob Peterson
1 sibling, 2 replies; 6+ messages in thread
From: Steven Whitehouse @ 2017-09-13 9:51 UTC (permalink / raw)
To: cluster-devel.redhat.com
Hi,
On 12/09/17 09:54, tsutomu.owa at toshiba.co.jp wrote:
> 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
>
> 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.
Dave, Bob, does that make sense?
Steve.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
2017-09-13 9:51 ` Steven Whitehouse
@ 2017-09-13 16:18 ` David Teigland
2017-09-18 20:20 ` David Teigland
2017-09-14 12:48 ` Bob Peterson
1 sibling, 1 reply; 6+ messages in thread
From: David Teigland @ 2017-09-13 16:18 UTC (permalink / raw)
To: cluster-devel.redhat.com
On Wed, Sep 13, 2017 at 10:51:06AM +0100, Steven Whitehouse wrote:
> Hi,
>
>
> On 12/09/17 09:54, tsutomu.owa at toshiba.co.jp wrote:
> > 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
> >
> > 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.
>
> 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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
2017-09-13 9:51 ` Steven Whitehouse
2017-09-13 16:18 ` David Teigland
@ 2017-09-14 12:48 ` Bob Peterson
1 sibling, 0 replies; 6+ messages in thread
From: Bob Peterson @ 2017-09-14 12:48 UTC (permalink / raw)
To: cluster-devel.redhat.com
----- Original Message -----
| Hi,
|
|
| On 12/09/17 09:54, tsutomu.owa at toshiba.co.jp wrote:
| > 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
| >
| > 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.
|
| Dave, Bob, does that make sense?
|
| Steve.
Hi,
Yes, sounds okay to me.
Bob Peterson
^ permalink raw reply [flat|nested] 6+ messages in thread
* [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review
2017-09-12 8:54 [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review tsutomu.owa
2017-09-13 9:51 ` Steven Whitehouse
@ 2017-09-14 12:47 ` Bob Peterson
1 sibling, 0 replies; 6+ messages in thread
From: Bob Peterson @ 2017-09-14 12:47 UTC (permalink / raw)
To: cluster-devel.redhat.com
----- Original Message -----
| 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
|
| thanks in advance.
| -- owa
|
|
|
Hi,
This patch set looks okay to me.
Reviewed-by: Bob Peterson <rpeterso@redhat.com>
Regards,
Bob Peterson
Red Hat File System
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-09-18 20:20 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-12 8:54 [Cluster-devel] [PATCH 00/18] [try #2] DLM: dlm patches need review tsutomu.owa
2017-09-13 9:51 ` Steven Whitehouse
2017-09-13 16:18 ` David Teigland
2017-09-18 20:20 ` David Teigland
2017-09-14 12:48 ` Bob Peterson
2017-09-14 12:47 ` Bob Peterson
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.