Bug 150914
Summary: | Add ocfs2 support to RHEL4 mount | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 4 | Reporter: | Manish Singh <manish.singh> | ||||||||
Component: | util-linux | Assignee: | Karel Zak <kzak> | ||||||||
Status: | CLOSED ERRATA | QA Contact: | Ben Levenson <benl> | ||||||||
Severity: | medium | Docs Contact: | |||||||||
Priority: | medium | ||||||||||
Version: | 4.0 | CC: | tao, van.okamura | ||||||||
Target Milestone: | --- | Keywords: | FutureFeature | ||||||||
Target Release: | --- | ||||||||||
Hardware: | All | ||||||||||
OS: | Linux | ||||||||||
Whiteboard: | |||||||||||
Fixed In Version: | RHBA-2005-669 | Doc Type: | Enhancement | ||||||||
Doc Text: | Story Points: | --- | |||||||||
Clone Of: | Environment: | ||||||||||
Last Closed: | 2005-10-05 16:51:10 UTC | Type: | --- | ||||||||
Regression: | --- | Mount Type: | --- | ||||||||
Documentation: | --- | CRM: | |||||||||
Verified Versions: | Category: | --- | |||||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||||
Embargoed: | |||||||||||
Bug Depends On: | |||||||||||
Bug Blocks: | 156322 | ||||||||||
Attachments: |
|
Description
Manish Singh
2005-03-11 22:58:54 UTC
Created attachment 111909 [details]
Patch against RHEL4's mount
Issue tracker 68424 filed for this. I've made another patch which calls a special mount program for guessed fs types too. I've also submitted this upstream. Created attachment 112080 [details]
Patch to use special mount program for guessed fs types
I don't see your second patch (id=112080, "Patch to use special mount program for guessed fs types") in upstream code. There is your first patch only. Please, can you explain which patch is required add to mount? Thanks. Both are needed for util-linux 2.12a. The second patch I only submitted recently, so I haven't received a response from the maintainer yet. I would like to see it in upstream and our devel (FC4) branch first. Don't forget it's new feature and RHEL3|4 is already stable and we prefer bugfixes there. Created attachment 114070 [details]
This is a new version of the guessed fs types patch
I just resubmitted the new patch to upstream. The maintainer seems AWOL. Has any of this gone into FC4? You're right, util-linux upstream seems pretty inactive... It seems that we will have to add it to the code without upstream. Now, in FC4 is your first patch (comment #1) only. BTW, in your second patch should be: mnt5_res = guess_fstype_and_mount(....); if (special) { block_signals (SIG_UNBLOCK); <----- return status; } The rest of your patch looks good. My plan: after FC4 GA release I will add the second patch to separate util-linux FC4 "testing" update. If I won't found any problem we can add it directly to FC4 and RHEL3/4. It's probably good way how we can test it. PM ACK for U2. Released FC3 (testing) update with OCFS2 support: util-linux-2.12a-24.4. The package contains both patches. The code in FC3 is almost same like in RHEL4 so it's better for tests that FC4. In FC4 the mount uses libblkid (from e2fsprogs package) instead mount_guess_fstype.c -- it means the patch from comment #1 is useless in FC4 and also very probably in the future RHEL releases. Manish, can you check e2fsprogs-1.37/lib/blkid/probe.c where is ocfs2 detection code for FC4? Thanks. The patch added to util-linux-2.12p-9.7 FC4 testing update. Note: I'm going to fix old OCFS support in mount too. It doesn't work with UUID now. there's: struct ocfs_volume_label { u_char disk_lock[48]; u_char label[64]; u_char label_len[2]; }; should be: struct ocfs_volume_label { unsigned char disk_lock[48]; unsigned char label[64]; unsigned char label_len[2]; unsigned char vol_id[16]; unsigned char vol_id_len[2]; }; Testers wanted: RHEL4-U2: version: util-linux-2.12a-16.EL4.11 test pkgs: http://people.redhat.com/kzak/util-linux/ocfs2/RHEL4-U2/ RHEL3-U6: version: util-linux-2.11y-31.10 test pkgs: http://people.redhat.com/kzak/util-linux/ocfs2/RHEL3-U6/ Thanks for arbitrary feedback :-) I can confirm that the mount command with the patch correctly detects OCFS2 filesystem and found device by filesystem label: # mkfs.ocfs2 -q -L ORATEST /dev/loop0 # mount -L ORATEST mnt ocfs2_hb_ctl: Unable to access cluster service while starting heartbeat (note that I don't have running cluster service) Old version: # mount -L ORATEST mnt mount: no such partition found The new packages work for me properly. An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on the solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2005-669.html |