[LU-6648] Wireshark 1.8.10 does not correctly parse LDLM_ENQUEUE intent : getxattr Created: 26/May/15 Updated: 03/Jun/17 Resolved: 03/Jun/17 |
|
| Status: | Resolved |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | Lustre 2.10.0 |
| Type: | Bug | Priority: | Minor |
| Reporter: | Andrew Uselton (Inactive) | Assignee: | Nathaniel Clark |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | wireshark | ||
| Attachments: |
|
||||||||||||
| Issue Links: |
|
||||||||||||
| Severity: | 3 | ||||||||||||
| Rank (Obsolete): | 9223372036854775807 | ||||||||||||
| Description |
|
Running the experiment 'multiop <file> Oac' to generate a getxattr VFS method call results in a tcpdump trace that the Lustre Wireshark extension fails to parse correctly. wireshark-1.8.10 was used built against a fairly recent checkout of Lustre HEAD (as of May 26, 2015). The tcpdump file it failed to parse is attached. Look for the "LDLM_ENQUEUE request [Protected Read][ intent : getxattr ]" and its reply. The tcpdump itself is taken from the client that ran the 'mulitop' command. A followup experiment that does the complimentary 'setxattr' ('multiop <file> OAc') also shows a LustreBUG in the summary output for the client that does the operation (2015-05-26_14-55-43_c20.tcpdump). When I look at the detailed output for both of these ('tshark -v ...') neither of them shows the LustreBUG output from the summary view. Not sure if that means it's just a reporting problem or a real failure to parse. For the 'getxattr' experiment the header says there are four buffers with non-zero length, but only three get displayed: For the 'setxattr' case thingsare a litle different: The "mds xattr eadata" is not interpreted. I'd be interested to know if there's anything in there I should know about. |
| Comments |
| Comment by Gerrit Updater [ 27/May/15 ] |
|
Nathaniel Clark (nathaniel.l.clark@intel.com) uploaded a new patch: http://review.whamcloud.com/14961 |
| Comment by Gerrit Updater [ 03/Jun/17 ] |
|
Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/14961/ |
| Comment by Peter Jones [ 03/Jun/17 ] |
|
Landed for 2.10 |