[Gluster-devel] Regarding default_forget/releasedir/release() fops

[Soumya Koduri]

Hi Jeff,

Recently while doing some tests (which involved lots of inode_forget()), 
I have noticed that my log file got flooded with below messages -

[2016-02-22 08:57:44.025565] W [defaults.c:2889:default_forget] (--> 
/usr/local/lib/libglusterfs.so.0(_gf_log_callingfn+0x231)[0x7fd00f63c15d] (--> 
/usr/local/lib/libglusterfs.so.0(default_forget+0x44)[0x7fd00f6cda2b] 
(--> /usr/local/lib/libglusterfs.so.0(+0x39706)[0x7fd00f64b706] (--> 
/usr/local/lib/libglusterfs.so.0(+0x397d2)[0x7fd00f64b7d2] (--> 
/usr/local/lib/libglusterfs.so.0(+0x3be08)[0x7fd00f64de08] ))))) 
0-gfapi: xlator does not implement forget_cbk

 From the code, looks like we throw a warning in default-tmpl.c if any 
xlator hasn't implemented forget(), releasedir() and release().

Though I agree it warns us about possible leaks which may happen if 
these fops are not supported, it is annoying to have these messages 
flooded in the log file which grew >1GB within few minutes.

Could you please confirm if it was intentional to throw this warning so 
that all xlators shall have these fops implemented or if we can change 
the log level to DEBUG?

Thanks,
Soumya