mbox series

[Xenial,SRU,0/6] LP #1737033

Message ID 20171208184313.32316-1-cascardo@canonical.com
Headers show
Series LP #1737033 | expand

Message

Thadeu Lima de Souza Cascardo Dec. 8, 2017, 6:43 p.m. UTC 
Revert the changes for lp #1728739, as they introduce a regression when using
ceph.

Tested by davecore and jvosburgh:
15:45 < davecore> jvosburgh: I can mount the cephfs filesystem with your 4.4.0-103-generic #126 kernel

Thadeu Lima de Souza Cascardo (6):
  Revert "libceph: MOSDOpReply v7 encoding"
  Revert "libceph: advertise support for TUNABLES5"
  Revert "crush: decode and initialize chooseleaf_stable"
  Revert "crush: add chooseleaf_stable tunable"
  Revert "crush: ensure take bucket value is valid"
  Revert "crush: ensure bucket id is valid before indexing buckets
    array"

 include/linux/ceph/ceph_features.h | 16 +---------------
 include/linux/crush/crush.h        |  8 +-------
 net/ceph/crush/mapper.c            | 33 +++++++--------------------------
 net/ceph/osd_client.c              | 10 ----------
 net/ceph/osdmap.c                  | 19 +++++--------------
 5 files changed, 14 insertions(+), 72 deletions(-)

Comments

Kleber Sacilotto de Souza Dec. 11, 2017, 9:37 a.m. UTC | #1 
On 12/08/17 19:43, Thadeu Lima de Souza Cascardo wrote:
> Revert the changes for lp #1728739, as they introduce a regression when using
> ceph.
> 
> Tested by davecore and jvosburgh:
> 15:45 < davecore> jvosburgh: I can mount the cephfs filesystem with your 4.4.0-103-generic #126 kernel
> 
> Thadeu Lima de Souza Cascardo (6):
>   Revert "libceph: MOSDOpReply v7 encoding"
>   Revert "libceph: advertise support for TUNABLES5"
>   Revert "crush: decode and initialize chooseleaf_stable"
>   Revert "crush: add chooseleaf_stable tunable"
>   Revert "crush: ensure take bucket value is valid"
>   Revert "crush: ensure bucket id is valid before indexing buckets
>     array"
> 
>  include/linux/ceph/ceph_features.h | 16 +---------------
>  include/linux/crush/crush.h        |  8 +-------
>  net/ceph/crush/mapper.c            | 33 +++++++--------------------------
>  net/ceph/osd_client.c              | 10 ----------
>  net/ceph/osdmap.c                  | 19 +++++--------------
>  5 files changed, 14 insertions(+), 72 deletions(-)
> 

Revert tested to fix the issue:

Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Stefan Bader Dec. 11, 2017, 11:03 a.m. UTC | #2 
On 08.12.2017 18:43, Thadeu Lima de Souza Cascardo wrote:
> Revert the changes for lp #1728739, as they introduce a regression when using
> ceph.
> 
> Tested by davecore and jvosburgh:
> 15:45 < davecore> jvosburgh: I can mount the cephfs filesystem with your 4.4.0-103-generic #126 kernel
> 
> Thadeu Lima de Souza Cascardo (6):
>   Revert "libceph: MOSDOpReply v7 encoding"
>   Revert "libceph: advertise support for TUNABLES5"
>   Revert "crush: decode and initialize chooseleaf_stable"
>   Revert "crush: add chooseleaf_stable tunable"
>   Revert "crush: ensure take bucket value is valid"
>   Revert "crush: ensure bucket id is valid before indexing buckets
>     array"
> 
>  include/linux/ceph/ceph_features.h | 16 +---------------
>  include/linux/crush/crush.h        |  8 +-------
>  net/ceph/crush/mapper.c            | 33 +++++++--------------------------
>  net/ceph/osd_client.c              | 10 ----------
>  net/ceph/osdmap.c                  | 19 +++++--------------
>  5 files changed, 14 insertions(+), 72 deletions(-)
> 
Acked-by: Stefan Bader <stefan.bader@canonical.com>
Kleber Sacilotto de Souza Dec. 11, 2017, 11:24 a.m. UTC | #3 
Applied to xenial/master-next branch. Thanks.