Re: [v2] fs/ceph/super: document struct ceph_dentry_info and its flags

Alex Markuze <[email protected]>
Newsgroups org.kernel.vger.ceph-devel
Message-ID <[email protected]>
Hi Max,

The /** before struct ceph_dentry_info and before the CEPH_DENTRY_*
macro definitions isn't valid kernel-doc — it generates a dozen new
scripts/kernel-doc warnings where there were none before. Either use
a proper "struct ceph_dentry_info - description" header with @field:
entries, or drop to plain /* */ for the struct-level and macro comments.

The dir-lease time field description still says "last touched" — that's
still wrong from v1. It's the insert/requeue timestamp used for aging
against dir_lease_ttl, not a last-touch time.

Could you respin?

--
Alex Markuze
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.