[LU-2606] lustre 2.4 don't able to start on 2.1 disks Created: 11/Jan/13 Updated: 27/Jul/15 Resolved: 07/Feb/13 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.4.0 |
| Fix Version/s: | Lustre 2.4.0 |
| Type: | Bug | Priority: | Blocker |
| Reporter: | Alexey Lyashkov | Assignee: | Di Wang |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | HB | ||
| Issue Links: |
|
||||||||
| Severity: | 3 | ||||||||
| Rank (Obsolete): | 6079 | ||||||||
| Description |
|
[ 363.060205] LDISKFS-fs (loop0): mounted filesystem with ordered data mode. quota=off. Opts: |
| Comments |
| Comment by Andreas Dilger [ 11/Jan/13 ] |
|
I think that this problem will be fixed as soon as the next patches in the DNE series land, since the group file is no longer used? If that isn't the case, Di can you make the code able to accept a 4-byte file and just treat it as a __u32 instead of a __u64? |
| Comment by Alex Zhuravlev [ 11/Jan/13 ] |
|
a bit unexpected because we do have a test for the case in conf-sanity.sh |
| Comment by Di Wang [ 11/Jan/13 ] |
|
Yes, we would not need group file anymore after that patch is landed. I just checked our disk2_1-ldiskfs.tar.bz2 (under tests), it seems [root@testnode tests]# mount -t ldiskfs -o loop ./ost /mnt/mds1 Apparently, we need to make a better disk image here. |
| Comment by Andreas Dilger [ 14/Jan/13 ] |
|
Di, could you please make a patch for this, and also fix the test image at the same time. |
| Comment by Di Wang [ 23/Jan/13 ] |
|
this patch http://review.whamcloud.com/#change,4325 (already merged) should fix this problem. I will update the test image later. |
| Comment by Alexey Lyashkov [ 24/Jan/13 ] |
|
I will retest today. |
| Comment by Andreas Dilger [ 30/Jan/13 ] |
|
Shadow, any update from your testing of the patch? |
| Comment by Jodi Levi (Inactive) [ 06/Feb/13 ] |
|
Are there any updates on the test of this patch? |
| Comment by Andreas Dilger [ 07/Feb/13 ] |
|
Closing bug per Di's comment that the fix has been merged. |
| Comment by Gerrit Updater [ 27/Jul/15 ] |
|
Alexander Boyko (alexander.boyko@seagate.com) uploaded a new patch: http://review.whamcloud.com/15731 |