pacific: isa-l: incorporate fix for aarch64 text relocation

[rosinL]

Fixes: https://tracker.ceph.com/issues/48681
Fixes: https://tracker.ceph.com/issues/50384

Signed-off-by: luo rixin luorixin@huawei.com

Contribution Guidelines

• To sign and title your commits, please refer to Submitting Patches to Ceph.

• If you are submitting a fix for a stable branch (e.g. "pacific"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.

Checklist

• Tracker (select at least one)
  • References tracker ticket
  • Very recent bug; references commit where it was introduced
  • New feature (ticket optional)
  • Doc update (no ticket needed)
  • Code cleanup (no ticket needed)
• Component impact
  • Affects Dashboard, opened tracker ticket
  • Affects Orchestrator, opened tracker ticket
  • No impact that needs to be tracked
• Documentation (select at least one)
  • Updates relevant documentation
  • No doc update is appropriate
• Tests (select at least one)
  • Includes unit test(s)
  • Includes integration test(s)
  • Includes bug reproducer
  • No tests

Show available Jenkins commands

• jenkins retest this please
• jenkins test classic perf
• jenkins test crimson perf
• jenkins test signed
• jenkins test make check
• jenkins test make check arm64
• jenkins test submodules
• jenkins test dashboard
• jenkins test dashboard cephadm
• jenkins test api
• jenkins test docs
• jenkins render docs
• jenkins test ceph-volume all
• jenkins test ceph-volume tox
• jenkins test windows

[rosinL]

@rzarzynski this a backport pr to pacific, to fix ceph-mon init failed on aarch64, please help to review.

[rosinL]

@rzarzynski this a backport pr to pacific, to fix ceph-mon init failed on aarch64, please help to review.

[rosinL]

@rzarzynski this a backport pr to pacific, to fix ceph-mon init failed on aarch64, please help to review.

[rzarzynski]

@rosinL: between 806b55ee578efd8158962b90121a4568eb1ecb66 and bee5180a1517f8b5e70b02fcd66790c623536c5d there is a lot of unrelated commits, which – as this is for pacific – makes a bit nervous. I'm not saying no but I want to know alternatives. What do you think about bringing the fix on top of 806b55ee578efd8158962b90121a4568eb1ecb66? We already have ceph/isa-l.git fork.

[rosinL]

@rzarzynski: ceph/isa-l.git is just forked from intel/isa-l.git,

bringing the fix on top of 806b55ee578efd8158962b90121a4568eb1ecb66

It is ok for this case. But, if the upstream intel/isa-l.git have fixed some bugs or fulfilled some features or optimizations we need, we need to cherry pick all of this and get a huge amount of work.
So, I think it is a better choice that keep ceph/isa-l just following the upstream

[rosinL]

@rzarzynski Hi, it has been suspended for a long time. If we just cherry pick bee5180a1517f8b5e70b02fcd66790c623536c5d to 806b55ee578efd8158962b90121a4568eb1ecb66, would you help to create a new branch (for pacific only) base from https://github.com/ceph/isa-l/tree/806b55ee578efd8158962b90121a4568eb1ecb66, and I will do the left work. Thank you.

[k0ste]

Closed #53463 in favor to this PR

[ljflores]

@rosinL @rzarzynski @k0ste If this is a must for 16.2.15, please add it to https://github.com/ceph/ceph/milestone/17 since Pacific is going EOL.

[rzarzynski]

I'm happy to resurrect this backport but under assumption of taking only pin-pointed changes -- extra features, optimizations etc. seems too much for an old, stable release.

[rosinL]

For pin-pointed changes, we need another isa-l branch to patch the fix. Would you create the isa-l branch, or is there other options? @rzarzynski

[rzarzynski]

If it's missed, creating it will be the first thing I'm going to do in the morning.

[rosinL]

OK, I will update the pr after you create the branch

[rzarzynski]

@rosinL: at the moment created the pacific branch in my fork –  https://github.com/rzarzynski/isa-l/tree/pacific. Let's use this as the starting point.
In the meantime I will look for somebody with w+ rights to the ceph/isa-l.git repo.

[rzarzynski]

@rosinL: https://github.com/ceph/isa-l/tree/pacific is ready :-).

[rosinL]

@rzarzynski please merge the PR ceph/isa-l#3, after the merge, I will upadte this PR to ceph/isa-l:pacific branch

[rzarzynski]

9c242d38321847acf04e9ccfc394ebf4506b239e is merged.

[rosinL]

@rzarzynski make check is passed, please review.

[rzarzynski]

Looks good!

commit 4b36e413c9ac28b4757b297779470693b699aeae (origin/pacific)
Merge: 806b55e 9c242d3
Author: Radoslaw Zarzynski <radoslaw.zarzynski@gmail.com>
Date:   Thu Jan 11 10:31:23 2024 +0100

    Merge pull request #3 from rosinL/pacific
    
    erasure_code: Fix text relocation on aarch64
    
    Reviewed-by: Radoslaw Zarzynski <rzarzyns@redhat.com>

commit 9c242d38321847acf04e9ccfc394ebf4506b239e
Author: luo rixin <luorixin@huawei.com>
Date:   Fri Jan 8 16:16:02 2021 +0800

    erasure_code: Fix text relocation on aarch64
    
    Here is the bug report on ceph. https://tracker.ceph.com/issues/48681
    
    Change-Id: Ie1c60a71f28c1a169c8899a621be9bb455f5e244
    Signed-off-by: luo rixin <luorixin@huawei.com>
    (cherry picked from commit bee5180a1517f8b5e70b02fcd66790c623536c5d)

commit 806b55ee578efd8158962b90121a4568eb1ecb66 (HEAD -> pacific, rzarzynski/pacific, rzarzynski/master)
Author: Greg Tucker <greg.b.tucker@intel.com>
Date:   Wed Feb 26 17:40:37 2020 -0700

    build: Bump revision to 2.29
    
    Change-Id: I78cfa77864f3fd77c3b63199bc18fd1782fe3dc2
    Signed-off-by: Greg Tucker <greg.b.tucker@intel.com>

[pereman2]

Rados approved: https://tracker.ceph.com/projects/rados/wiki/MAIN#httpstrellocomcYriUU51p1929-wip-yuri2-testing-2024-01-18-1314-pacific-old-wip-yuri2-testing-2024-01-12-1128-pacific