[LU-6495] push Fujitsu IB multi-rail patch to Gerrit Created: 24/Apr/15 Updated: 25/Oct/15 Resolved: 25/Oct/15 |
|
| Status: | Closed |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.8.0 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major |
| Reporter: | Andreas Dilger | Assignee: | Amir Shehata (Inactive) |
| Resolution: | Duplicate | Votes: | 0 |
| Labels: | patch | ||
| Issue Links: |
|
||||||||
| Rank (Obsolete): | 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>". |
| Comments |
| Comment by James A Simmons [ 24/Apr/15 ] |
|
Where is the patch? I will gladly rebase against master |
| Comment by Peter Jones [ 24/Apr/15 ] |
|
James We received it via email and Amir will be pushing it to gerrit asap Peter |
| Comment by Gerrit Updater [ 25/Apr/15 ] |
|
Oleg Drokin (oleg.drokin@intel.com) uploaded a new patch: http://review.whamcloud.com/14595 |
| Comment by Oleg Drokin [ 25/Apr/15 ] |
|
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. The documentation I've got was placed into the commit message. |
| Comment by Amir Shehata (Inactive) [ 28/Apr/15 ] |
|
I merged the Fujitsu patch in latest master: |
| Comment by Amir Shehata (Inactive) [ 19/Jun/15 ] |
|
Duplicate of |
| Comment by Andreas Dilger [ 25/Oct/15 ] |
|
Reopen to remove fix version, since this code was not landed. |