From mboxrd@z Thu Jan 1 00:00:00 1970 From: tristan Date: Mon, 04 Jan 2010 17:21:30 +0800 Subject: [Ocfs2-devel] [PATCH 1/2] Ocfs2: Add stuffs associated with ocfs2_info_request ioctls to ioctl.h In-Reply-To: <20100104085806.GA13852@mail.oracle.com> References: <1262247524-26824-1-git-send-email-tristan.ye@oracle.com> <20091231201059.GB3301@mail.oracle.com> <4B41541F.8030704@oracle.com> <20100104085806.GA13852@mail.oracle.com> Message-ID: <4B41B31A.8030307@oracle.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ocfs2-devel@oss.oracle.com Joel Becker wrote: > On Mon, Jan 04, 2010 at 10:36:15AM +0800, tristan wrote: > >> Oh, I may misunderstand your words, Joel. Did you mean we only split the on-disk layout and ioctls into two parts(ocfs2_fs.h and ocfs2_ioctl.h) in ocfs2-tools for userspace, >> while kernel part still maintains one ocfs2_fs.h to cover both definitions for on-disk and ioctl strutures? >> > > No, I don't mean that. I mean we have ocfs2_fs.h and > ocfs2_ioctl.h in both the kernel and the tools. > > >> If not, why we need maintain two ioctl headers in kernel part(ioctl.h and ocfs2_ioctl.h)? >> > > ioctl.h contains things that are only important inside the > kernel build; specifically, the declarations of the ioctl functions > called by file_operations. No userspace program needs the declaration > of kernel code functions. ocfs2_ioctl.h should contain the things a > userspace program needs to invoke the ioctls. The IOC definitions and > any structures they use. > Basically, anything ocfs2_*.h is a userspace header that can be > installed for userspace programs to use. They get copied to ocfs2-tools > include/ocfs2-kernel. All the other headers are kernel-only. > Oh, I got the naming convention of "ocfs2_*.h":-) Thanks for the clarification! > Joel > > >