Bug 428896
Summary: | GFS2: warning: assertion "al->al_alloced" failed | ||||||
---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | Nate Straz <nstraz> | ||||
Component: | kernel | Assignee: | Steve Whitehouse <swhiteho> | ||||
Status: | CLOSED NOTABUG | QA Contact: | GFS Bugs <gfs-bugs> | ||||
Severity: | low | Docs Contact: | |||||
Priority: | low | ||||||
Version: | 5.1 | CC: | cluster-maint, rpeterso | ||||
Target Milestone: | rc | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2008-01-16 15:37:55 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: | |||||||
Attachments: |
|
Description
Nate Straz
2008-01-15 21:57:00 UTC
This basically means that gfs2_write_alloc_required thought that we needed to allocate some blocks for this page fault, but when we came to do the allocation, we discovered that we didn't need to allocate any blocks. I can't see why that should be the case at the moment. No, I tell a lie. I can see exactly why it fails. Bob's latest patch to gfs2_write_alloc_required is wrong. It tests di_blocks (which is wrong because its the number of blocks allocated to the file, not the size of the file). It ought to be testing (di_size + (block size) - 1) >> (block size shift). Also di_blocks covers all the blocks added to the file and not just the data blocks, so its wrong in the normal case too as its not providing any short cut which was the original idea. You can get the result that you bumped into in the case of a file with a hole in it as then di_blocks will be much smaller than the file size. Created attachment 291844 [details]
Potential fix
This should fix it. Its not tested yet, but I'm doing that at the moment.
This was a problem with the fifth performance patch for bug #253990. The patch from comment #3 above has been included in the reworked patch6 for bug @253990, so I'm closing this one as NOTABUG. |