[BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[Yan, Zheng]

ubuntu-vivid git tree includes commit ebeec2ef4c (ceph: fix message length computation), but does not contain backport for upstream commit 1f041a89b4 (ceph: fix request time stamp encoding). This breaks cephfs kernel driver completely. Please add the later commit to your git tree.

Regards
Yan, Zheng

Re: [BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[Kamal Mostafa]

On Thu, Mar 31, 2016 at 12:06:42PM +0800, Yan, Zheng wrote:
> ubuntu-vivid git tree includes commit ebeec2ef4c (ceph: fix message
> length computation), but does not contain backport for upstream commit
> 1f041a89b4 (ceph: fix request time stamp encoding). This breaks cephfs
> kernel driver completely. Please add the later commit to your git
> tree.
> 
> Regards
> Yan, Zheng

Hi Yan-

Thanks for letting us know about this.  If that patch is required for
all trees that carry a backport of 777d738 (ceph: fix message length
computation), then we'll need to apply 1f041a89b4 to:

  3.19.y-ckt (stable)
  3.16.y-ckt (stable)
  ubuntu-vivid
  ubuntu-utopic

For our reference, can you point me to a bug report or email thread
which discusses the breakage?

 -Kamal

Re: [BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[Yan, Zheng]

> On Mar 31, 2016, at 22:39, Kamal Mostafa <kamal@canonical.com> wrote:
> 
> On Thu, Mar 31, 2016 at 12:06:42PM +0800, Yan, Zheng wrote:
>> ubuntu-vivid git tree includes commit ebeec2ef4c (ceph: fix message
>> length computation), but does not contain backport for upstream commit
>> 1f041a89b4 (ceph: fix request time stamp encoding). This breaks cephfs
>> kernel driver completely. Please add the later commit to your git
>> tree.
>> 
>> Regards
>> Yan, Zheng
> 
> Hi Yan-
> 
> Thanks for letting us know about this.  If that patch is required for
> all trees that carry a backport of 777d738 (ceph: fix message length
> computation), then we'll need to apply 1f041a89b4 to:

yes, that patch is required for all trees include the back port 

> 
>  3.19.y-ckt (stable)
>  3.16.y-ckt (stable)
>  ubuntu-vivid
>  ubuntu-utopic
> 
> For our reference, can you point me to a bug report or email thread
> which discusses the breakage?
> 
http://tracker.ceph.com/issues/15302 <http://tracker.ceph.com/issues/15302>

Regards
Yan, Zheng


> -Kamal

Re: [BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[Kamal Mostafa]

On Fri, Apr 01, 2016 at 10:01:23AM +0800, Yan, Zheng wrote:
> 
> > On Mar 31, 2016, at 22:39, Kamal Mostafa <kamal@canonical.com> wrote:
> > 
> > On Thu, Mar 31, 2016 at 12:06:42PM +0800, Yan, Zheng wrote:
> >> ubuntu-vivid git tree includes commit ebeec2ef4c (ceph: fix message
> >> length computation), but does not contain backport for upstream commit
> >> 1f041a89b4 (ceph: fix request time stamp encoding). This breaks cephfs
> >> kernel driver completely. Please add the later commit to your git
> >> tree.
> >> 
> >> Regards
> >> Yan, Zheng
> > 
> > Hi Yan-
> > 
> > Thanks for letting us know about this.  If that patch is required for
> > all trees that carry a backport of 777d738 (ceph: fix message length
> > computation), then we'll need to apply 1f041a89b4 to:
> 
> yes, that patch is required for all trees include the back port 
> 
> > 
> >  3.19.y-ckt (stable)
> >  3.16.y-ckt (stable)
> >  ubuntu-vivid
> >  ubuntu-utopic
> > 
> > For our reference, can you point me to a bug report or email thread
> > which discusses the breakage?
> > 
> http://tracker.ceph.com/issues/15302 <http://tracker.ceph.com/issues/15302>


Ok, I've filed this Launchpad bug to track the issue in the affected Ubuntu
kernels: http://bugs.launchpad.net/bugs/1564950

We'll get the fix into the pipeline ASAP.

Thanks again Yan!

 -Kamal

Re: [BUG] backport commit ebeec2ef4c in ubuntu-vivid.git break cephfs

[Ilya Dryomov]

On Fri, Apr 1, 2016 at 4:01 AM, Yan, Zheng <zyan@redhat.com> wrote:
>
>> On Mar 31, 2016, at 22:39, Kamal Mostafa <kamal@canonical.com> wrote:
>>
>> On Thu, Mar 31, 2016 at 12:06:42PM +0800, Yan, Zheng wrote:
>>> ubuntu-vivid git tree includes commit ebeec2ef4c (ceph: fix message
>>> length computation), but does not contain backport for upstream commit
>>> 1f041a89b4 (ceph: fix request time stamp encoding). This breaks cephfs
>>> kernel driver completely. Please add the later commit to your git
>>> tree.
>>>
>>> Regards
>>> Yan, Zheng
>>
>> Hi Yan-
>>
>> Thanks for letting us know about this.  If that patch is required for
>> all trees that carry a backport of 777d738 (ceph: fix message length
>> computation), then we'll need to apply 1f041a89b4 to:
>
> yes, that patch is required for all trees include the back port
>
>>
>>  3.19.y-ckt (stable)
>>  3.16.y-ckt (stable)
>>  ubuntu-vivid
>>  ubuntu-utopic
>>
>> For our reference, can you point me to a bug report or email thread
>> which discusses the breakage?
>>
> http://tracker.ceph.com/issues/15302 <http://tracker.ceph.com/issues/15302>

777d738a5e58 "Fixes" commit that went into 3.16, while 1f041a89b4f2
went into 4.0.  This means 3.18 stable series is affected is well.

Sasha, could you cherry-pick 1f041a89b4f2 ("ceph: fix request time
stamp encoding")?

Thanks,

                Ilya