[Ocfs2-devel] [PATCH 0/4] ocfs2: bugfix for hard readonly mount

[Tristan Ye]

Sunil Mushran wrote:
> On 05/26/2011 09:16 AM, Tristan Ye wrote:
>> On 05/27/2011 12:03 AM, Sunil Mushran wrote:
>>> Tiger, Thanks. Was this tested with all fs-features enabled? I am
>>> worried about quotas. If not, please test that.
>>>
>>> Tristan, Please can you cross check these patches. Enable all features
>>> in a volume, make it readonly and do mount, ls -lR, umount, read, stat,
>>> stat -f. This calls for a test script. We should be able to use
>>> loopback.
>>> losetup -rf.
>>
>> Sure,
>>
>> I'm going to write a dedicated script for sanity check, including these
>> generic fs syscalls.
> 
> Thanks Tristan.
> 
> Tiger, I would suggest you roll up the patch into one. I see no reason
> for the breakup considering it fixes just one bug and is small enough.


After running tests on block/loopback device, it shows that:

1. 'ls -lR /kernel/tree' dumps slightly different output:

6c6
< total 436
---
> total 428
37c37
< total 52
---
> total 44
394c394
< total 252
---
> total 228
1408c1408
< total 8
---
> total 4
1636c1636
< total 8

while the data is consistent anyway.


2. 'stat -f' also dumps a slightly different map:

4,5c4,5
< Blocks: Total: 176120528  Free: 170080504  Available: 170080504
< Inodes: Total: 22015066   Free: 21260063
---
> Blocks: Total: 176120528  Free: 170088304  Available: 170088304
> Inodes: Total: 22015066   Free: 21261038


3. Enabling quota on readonly fs caused kernel oops

4. Otherwise, everything is fine(xattr,reflink,..etc)


Testing script attached.


Tristan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: verify_readonly.sh
Type: application/x-sh
Size: 15228 bytes
Desc: not available
Url : http://oss.oracle.com/pipermail/ocfs2-devel/attachments/20110602/b891e97d/attachment-0001.sh