ocfs2-devel.oss.oracle.com archive mirror
 help / color / mirror / Atom feed
From: Joel Becker <Joel.Becker@oracle.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] [PATCH 1/1] Ocfs2: Add new OCFS2_IOC_INFO ioctl for ocfs2 v7.
Date: Wed, 5 May 2010 18:05:04 -0700	[thread overview]
Message-ID: <20100506010503.GA9955@mail.oracle.com> (raw)
In-Reply-To: <1272284243-22492-1-git-send-email-tristan.ye@oracle.com>

On Mon, Apr 26, 2010 at 08:17:23PM +0800, Tristan Ye wrote:
> The reason why we need this ioctl is to offer the none-privileged
> end-user a possibility to get filesys info gathering.
> 
> We use OCFS2_IOC_INFO to manipulate the new ioctl, userspace passes a
> structure to kernel containing an array of request pointers and request
> count, such as,

	You're going to hate us, but Sunil and I decided that the
complex error reporting was just too much for what should be a simple
query interface.  We think we should go back to the original, where any
error is returned, and the caller cannot trust any of the results unless
the call returns 0.  This means you don't need the ir_error field.  I
suppose you can keep the ERROR flag, as this can tell the caller which
request caused the error.

Joel

-- 

Life's Little Instruction Book #237

	"Seek out the good in people."

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127

  parent reply	other threads:[~2010-05-06  1:05 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-26 12:17 [Ocfs2-devel] [PATCH 1/1] Ocfs2: Add new OCFS2_IOC_INFO ioctl for ocfs2 v7 Tristan Ye
2010-04-27 20:07 ` Sunil Mushran
2010-05-06  1:05 ` Joel Becker [this message]
2010-05-06  2:09   ` tristan
  -- strict thread matches above, loose matches on Subject: below --
2010-05-19  2:28 Tristan Ye
2010-05-20 23:26 ` Joel Becker
2010-05-20 23:49   ` Joel Becker
2010-05-21  9:07     ` tristan
2010-05-21 10:22       ` Joel Becker
2010-05-21  1:30   ` tristan
2010-05-21  2:41     ` Joel Becker
2010-05-21  3:05       ` tristan
2010-05-11  7:21 [Ocfs2-devel] [PATCH 0/1] Ocfs2: o2info for kernel v7 Tristan Ye
2010-05-11  7:21 ` [Ocfs2-devel] [PATCH 1/1] Ocfs2: Add new OCFS2_IOC_INFO ioctl for ocfs2 v7 Tristan Ye
2010-05-11 20:40   ` Joel Becker
2010-05-18 23:55     ` Joel Becker
2010-05-19  3:03       ` tristan
2010-05-06  8:43 Tristan Ye
2010-05-10 20:01 ` Joel Becker
2010-05-11  2:12   ` tristan
2010-05-11  6:51     ` Joel Becker
2010-04-19 11:00 Tristan Ye
2010-04-19 20:16 ` Sunil Mushran
2010-04-20  2:31   ` tristan
2010-04-20  4:28     ` Sunil Mushran

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100506010503.GA9955@mail.oracle.com \
    --to=joel.becker@oracle.com \
    --cc=ocfs2-devel@oss.oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).