On Wed, Apr 8, 2015 at 12:47 PM, Anna Schumaker <Anna.Schumaker@xxxxxxxxxx> wrote: > I had some trouble using mountstats to analyze READ_PLUS operations, so I > wrote these patches to fix my issues. > > I found that /proc/self/mountstats was reporting "GETDEVICELIST" as "51", Shouldn't we try to fix that as being a regression in the client? I see no reason why we shouldn't be able to add in a p_name for GETDEVICELIST. > most likely because it was removed from the client but the nfs4_procedures > array still has a placeholder for it. Mountstats crashes when it tries to > analyze "51", so the second patch fixes this issue by looking for numeric > operation names. I'm willing to put in stubs for GETDEVICELIST back into > the client if that would be a better fix. > > Thanks, > Anna > > > Anna Schumaker (2): > mountstats: Resync NFSv4 Operations List > mountstats: Only count operations that have names > > tools/mountstats/mountstats.py | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > > -- > 2.3.5 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-nfs" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
