Junxiao Bi
2017-Mar-29 01:07 UTC
[Ocfs2-devel] [PATCH] ocfs2: o2hb: revert hb threshold to keep compatible
On 03/29/2017 06:31 AM, Andrew Morton wrote:> On Tue, 28 Mar 2017 09:40:45 +0800 Junxiao Bi <junxiao.bi at oracle.com> wrote: > >> Configfs is the interface for ocfs2-tools to set configure to >> kernel. Change heartbeat dead threshold name in configfs will >> cause compatible issue, so revert it. >> >> Fixes: 45b997737a80 ("ocfs2/cluster: use per-attribute show and store methods") > > I don't get it. 45b997737a80 was merged nearly two years ago, so isn't > it a bit late to fix compatibility issues? >This compatibility will not cause ocfs2 down, just some configure (hb dead threshold) lose effect. If someone want to use the new kernel, they should apply this fix. Thanks, Junxiao.
Andrew Morton
2017-Mar-29 03:31 UTC
[Ocfs2-devel] [PATCH] ocfs2: o2hb: revert hb threshold to keep compatible
On Wed, 29 Mar 2017 09:07:08 +0800 Junxiao Bi <junxiao.bi at oracle.com> wrote:> On 03/29/2017 06:31 AM, Andrew Morton wrote: > > On Tue, 28 Mar 2017 09:40:45 +0800 Junxiao Bi <junxiao.bi at oracle.com> wrote: > > > >> Configfs is the interface for ocfs2-tools to set configure to > >> kernel. Change heartbeat dead threshold name in configfs will > >> cause compatible issue, so revert it. > >> > >> Fixes: 45b997737a80 ("ocfs2/cluster: use per-attribute show and store methods") > > > > I don't get it. 45b997737a80 was merged nearly two years ago, so isn't > > it a bit late to fix compatibility issues? > > > This compatibility will not cause ocfs2 down, just some configure (hb > dead threshold) lose effect. If someone want to use the new kernel, they > should apply this fix.Well could someone please send a better changelog? One which carefully describes the present behaviour, what is wrong with it and how the patch fixes it? One reason for doing this is to permit effecitive patch review. Another reason is to permit others to decide whether the patch should be backported into -stable kernels. Yet another reason is so that maintainers of other kernels can determine whether this patch will fix behaviour which their users are reporting. Thanks.
Joseph Qi
2017-Mar-29 04:01 UTC
[Ocfs2-devel] [PATCH] ocfs2: o2hb: revert hb threshold to keep compatible
On 17/3/29 09:07, Junxiao Bi wrote:> On 03/29/2017 06:31 AM, Andrew Morton wrote: >> On Tue, 28 Mar 2017 09:40:45 +0800 Junxiao Bi <junxiao.bi at oracle.com> wrote: >> >>> Configfs is the interface for ocfs2-tools to set configure to >>> kernel. Change heartbeat dead threshold name in configfs will >>> cause compatible issue, so revert it. >>> >>> Fixes: 45b997737a80 ("ocfs2/cluster: use per-attribute show and store methods") >> I don't get it. 45b997737a80 was merged nearly two years ago, so isn't >> it a bit late to fix compatibility issues? >> > This compatibility will not cause ocfs2 down, just some configure (hb > dead threshold) lose effect. If someone want to use the new kernel, they > should apply this fix.The threshold configuration file has default value in kernel, so it will only affect changing this value in user space. Thanks, Joseph> > Thanks, > Junxiao. > > _______________________________________________ > Ocfs2-devel mailing list > Ocfs2-devel at oss.oracle.com > https://oss.oracle.com/mailman/listinfo/ocfs2-devel