Re: [Nfs-ganesha-devel] rpcping

2018-03-14 Thread Matt Benjamin
Hi Bill, I was not (not intentionally, and, I think, not at all) imputing sentiment to Daniel nor myself. I was paraphrasing statements Daniel made not only informally to me, but publically in this week's nfs-ganesha call, in which you were present. I'm not denigrating your work, but I did

Re: [Nfs-ganesha-devel] rpcping

2018-03-14 Thread William Allen Simpson
On 3/14/18 7:27 AM, Matt Benjamin wrote: Daniel doesn't think you've measured much accurately yet, but at least the effort (if not the discussion) aims to. I'm sure Daniel can speak for himself. At your time of writing, Daniel had not yet arrived in the office after my post this am. So I'm

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: ci test: do not merge

2018-03-14 Thread GerritHub
>From : ka...@redhat.com has uploaded this change for review. ( https://review.gerrithub.io/403880 Change subject: ci test: do not merge .. ci test: do not merge Change-Id:

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: FSAL: remove xattr_access_rights

2018-03-14 Thread GerritHub
>From Jeff Layton : Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/403849 Change subject: FSAL: remove xattr_access_rights .. FSAL: remove xattr_access_rights This ops

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: manpage: add missing options for the RecoveryBackend section

2018-03-14 Thread GerritHub
>From Jeff Layton : Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/403844 Change subject: manpage: add missing options for the RecoveryBackend section .. manpage: add

[Nfs-ganesha-devel] Change in ffilz/nfs-ganesha[next]: config: flesh out sample config file for FSAL_CEPH

2018-03-14 Thread GerritHub
>From Jeff Layton : Jeff Layton has uploaded this change for review. ( https://review.gerrithub.io/403825 Change subject: config: flesh out sample config file for FSAL_CEPH .. config: flesh out sample

Re: [Nfs-ganesha-devel] rpcping

2018-03-14 Thread Matt Benjamin
Daniel doesn't think you've measured much accurately yet, but at least the effort (if not the discussion) aims to. On Wed, Mar 14, 2018 at 2:54 AM, William Allen Simpson wrote: Matt -- Matt Benjamin Red Hat, Inc. 315 West Huron Street, Suite 140A Ann Arbor,

Re: [Nfs-ganesha-devel] rpcping

2018-03-14 Thread William Allen Simpson
On 3/13/18 1:58 PM, Daniel Gryniewicz wrote: rpcping was not thread safe.  I have fixes for it incoming. With DanG's significant help, we now have better timing results. There was an implicit assumption in the ancient code that it was calling single threaded tirpc, while ntirpc is

Re: [Nfs-ganesha-devel] rpcping

2018-03-14 Thread William Allen Simpson
On 3/13/18 8:27 AM, Matt Benjamin wrote: On Tue, Mar 13, 2018 at 2:38 AM, William Allen Simpson wrote: but if we assume xids retire in xid order also, They do. Should be no variance. Eliminating the dupreq caching -- also using the rbtree -- significantly