Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-6495

push Fujitsu IB multi-rail patch to Gerrit

Details

    • Improvement
    • Resolution: Duplicate
    • Major
    • None
    • Lustre 2.8.0
    • 9223372036854775807

    Description

      Please push the IB multi-rail patch to the Gerrit fs/lustre-release branch. My understanding is that the patch is currently based on Lustre 2.6.0.

      Ideally the patch would be based on the current master branch. However, if there are a large number of non-trivial conflicts applying the patch to master, it may be necessary to rewind your local branch to 2.7.0 or 2.6.0 (e.g. "git checkout -b 2.6.0") and pushing the patch there as a starting point.

      It should be possible to use "git am <patch file>" to import the Git patch file (which may even apply it to the original commit that it was based on) so that the original author (NAGUCHI Yoshinori <y-naguchi@jp.fujitsu.com>) is preserved. If not, then the patch could be applied with "git apply --author="NAGUCHI Yoshinori <y-naguchi@jp.fujitsu.com>" <patch file>".

      Attachments

        Issue Links

          Activity

            [LU-6495] push Fujitsu IB multi-rail patch to Gerrit

            Reopen to remove fix version, since this code was not landed.

            adilger Andreas Dilger added a comment - Reopen to remove fix version, since this code was not landed.

            Duplicate of LU-6531

            ashehata Amir Shehata (Inactive) added a comment - Duplicate of LU-6531

            I merged the Fujitsu patch in latest master:
            http://review.whamcloud.com/#/c/14625/

            ashehata Amir Shehata (Inactive) added a comment - I merged the Fujitsu patch in latest master: http://review.whamcloud.com/#/c/14625/
            green Oleg Drokin added a comment -

            I just pushed http://review.whamcloud.com/14595 - this is the Fujitsu patch verbatim, and it's against b2_6, but master needs quite some rebasing.
            I just wanted to preserve original submission for posterity too, but I dislike appending large patches to jira.

            The documentation I've got was placed into the commit message.

            green Oleg Drokin added a comment - I just pushed http://review.whamcloud.com/14595 - this is the Fujitsu patch verbatim, and it's against b2_6, but master needs quite some rebasing. I just wanted to preserve original submission for posterity too, but I dislike appending large patches to jira. The documentation I've got was placed into the commit message.

            Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: http://review.whamcloud.com/14595
            Subject: LU-6495 o2iblnd: IB Multi-Rail
            Project: fs/lustre-release
            Branch: b2_6
            Current Patch Set: 1
            Commit: 6f0a16efe51f1f63f1050c16d5fdbfb0870a433e

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: http://review.whamcloud.com/14595 Subject: LU-6495 o2iblnd: IB Multi-Rail Project: fs/lustre-release Branch: b2_6 Current Patch Set: 1 Commit: 6f0a16efe51f1f63f1050c16d5fdbfb0870a433e
            pjones Peter Jones added a comment -

            James

            We received it via email and Amir will be pushing it to gerrit asap

            Peter

            pjones Peter Jones added a comment - James We received it via email and Amir will be pushing it to gerrit asap Peter
            simmonsja James A Simmons added a comment - - edited

            Where is the patch? I will gladly rebase against master

            simmonsja James A Simmons added a comment - - edited Where is the patch? I will gladly rebase against master

            People

              ashehata Amir Shehata (Inactive)
              adilger Andreas Dilger
              Votes:
              0 Vote for this issue
              Watchers:
              16 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: