Bug 517162
| Summary: | cthon test5 failing on nfsv4 with rhel6 client vs. rhel4 server | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Linux 4 | Reporter: | Mike Gahagan <mgahagan> | ||||
| Component: | kernel | Assignee: | Jeff Layton <jlayton> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Mike Gahagan <mgahagan> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | urgent | ||||||
| Version: | 4.8 | CC: | dhoward, jburke, jlayton, pbunyan, steved, syeghiay | ||||
| Target Milestone: | rc | Keywords: | ZStream | ||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | |||||||
| : | 573995 (view as bug list) | Environment: | |||||
| Last Closed: | 2011-02-16 15:43:13 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: | 597314 | ||||||
| Attachments: |
|
||||||
|
Description
Mike Gahagan
2009-08-12 19:21:02 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux major release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Major release. This request is not yet committed for inclusion. *** Bug 567333 has been marked as a duplicate of this bug. *** On the open(), the RHEL6 client does a SETATTR call to take care of the O_TRUNC open flag. When it does this however, it sends a zero stateid, even though it has the file open and has gotten a valid stateid back. I suspect that RHEL4 handling this case incorrectly, but I'll need to verify it with the spec. Created attachment 400268 [details]
patch -- 2 patches to fix zero-stateid handling in nfsd setattr
These two (really old) patches seem to fix the problem for me. They change the code to handle the zero-stateid differently. I need to look over them a bit more carefully but they seem sane.
It's also not clear to me whether rhel6 is doing the right thing. The file has been opened, so it seems like it should be using the stateid we got from that. It's tough to tell from the spec whether RHEL6 is doing the right thing. I think that it's technically *not*, but most servers apparently don't really care. Fixing this in RHEL6 will be more of a VFS patch though. We'd probably have to fix may_open to take an "nd" arg, and then have it check to see whether there's an already instantiated filp in the open intent stuff, and then have it pass that to do_truncate. It's probably the right thing to do, but really isn't high priority. For giggles, I went ahead and sent a patch upstream to fix this behavior in the client too. It turns out to be a pretty small patch there, but will probably be a bit larger for RHEL6. http://marc.info/?l=linux-kernel&m=126868917104621&w=2 ...if this ends up flying upstream, I'll clone this BZ for RHEL6 too and we'll fix it there as well. RHEL6 bug 573995. *** Bug 486768 has been marked as a duplicate of this bug. *** This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. Committed in 89.26.EL . RPMS are available at http://people.redhat.com/vgoyal/rhel4/ 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 therefore 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/RHSA-2011-0263.html |