All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joel Becker <Joel.Becker@oracle.com>
To: ocfs2-devel@oss.oracle.com
Subject: [Ocfs2-devel] [PATCH 1/2] ocfs2-tools: Add extended attribute	support in mkfs.ocfs2
Date: Fri, 25 Jul 2008 02:33:15 -0700	[thread overview]
Message-ID: <20080725093315.GF6317@ca-server1.us.oracle.com> (raw)
In-Reply-To: <1216969599-9280-1-git-send-email-tiger.yang@oracle.com>

On Fri, Jul 25, 2008 at 03:06:38PM +0800, Tiger Yang wrote:
> diff --git a/include/ocfs2-kernel/ocfs2_fs.h b/include/ocfs2-kernel/ocfs2_fs.h
> index 0cfef2a..ade9ec1 100644
> --- a/include/ocfs2-kernel/ocfs2_fs.h
> +++ b/include/ocfs2-kernel/ocfs2_fs.h

	You've got an ancient version of the xattr ocfs2_fs.h here.

> +struct ocfs2_xattr_entry {
> +	__le32	xe_name_hash;
> +	__le16	xe_name_offset;
> +	__u8	xe_name_len;
> +	__u8	xe_type : 7;
> +	__u8	xe_local : 1;
> +	__le64	xe_value_size;
> +};

	We removed the bitfields.

> +struct ocfs2_xattr_block {
> +/*00*/	__u8	xb_signature[8];
> +	__le16	xb_suballoc_slot;
> +	__le16	xb_suballoc_bit;
> +	__le32	xb_fs_generation;
> +/*10*/	__le32	xb_csum;
> +	__le16	xb_flags;
> +	__le16	xb_reserved0;
> +	__le64	xb_blkno;
> +/*20*/	__le64	xb_reserved1[2];
> +/*30*/	union {
> +		struct ocfs2_xattr_header	xb_header;
> +		struct ocfs2_xattr_tree_root	xb_root;
> +	} xb_attrs;
> +};

	This got shuffled around for a 64bit checksum field.  Etc.  Can
you sync up with Tao?

Joel

-- 

"The nice thing about egotists is that they don't talk about other
 people."
         - Lucille S. Harper

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

  parent reply	other threads:[~2008-07-25  9:33 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-25  7:06 [Ocfs2-devel] [PATCH 1/2] ocfs2-tools: Add extended attribute support in mkfs.ocfs2 Tiger Yang
2008-07-25  7:06 ` [Ocfs2-devel] [PATCH 2/2] ocfs2-tools: Add extended attribute support in fsck.ocfs2 Tiger Yang
2008-07-25  7:52   ` Tao Ma
2008-07-25  8:56     ` Tiger Yang
2008-07-25  9:08       ` Tao Ma
2008-07-25  9:33 ` Joel Becker [this message]
2008-07-29  5:49   ` [Ocfs2-devel] [PATCH 1/2] ocfs2-tools: Add extended attribute support in mkfs.ocfs2 Tiger Yang
2008-07-29  7:09     ` Tao Ma
2008-07-29 22:23     ` Joel Becker

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=20080725093315.GF6317@ca-server1.us.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.