From: Valerie Clement <valerie.clement@bull.net>
To: Theodore Tso <tytso@mit.edu>
Cc: ext4 development <linux-ext4@vger.kernel.org>
Subject: Re: [PATCH 0/8][e2fsprogs] 64-bit block number support - first part
Date: Wed, 17 Oct 2007 16:13:50 +0200 [thread overview]
Message-ID: <4716189E.7060301@bull.net> (raw)
In-Reply-To: <20071014151211.GA6328@thunk.org>
Theodore Tso wrote:
> It's good to clean this up, but this presumes that blk_t is going to
> change to a 64-bit type. I'd much rather *add* a blk64_t, and keep
> blk_t at 32 bits. This patch series in a number of places makes the
> presumption that blk_t will change size --- in particular when it
> creates new interfaces that returns 64-bits through a blk_t. Please
> don't do that.
Does it mean that we'll have to duplicate all of the structures
containing fields of type blk_t (like process_block_struct,
e2fsck_struct, ext2_inode_cache, ...) and also duplicate most of
the code for the functions which use variables of type blk_t?
Valérie
next prev parent reply other threads:[~2007-10-17 14:14 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-30 15:28 [PATCH 0/8][e2fsprogs] 64-bit block number support - first part Valerie Clement
2007-08-30 15:31 ` [PATCH 1/8][e2fsprogs] use existing function ext2fs_group_first_block() Valerie Clement
2007-10-14 22:53 ` Theodore Tso
2007-08-30 15:33 ` [PATCH 2/8][e2fsprogs] convert block number definition to blk_t Valerie Clement
2007-10-15 1:02 ` Theodore Tso
2007-08-30 15:37 ` [PATCH 3/8][e2fsprogs] add new -D <desc-size> option to mkfs Valerie Clement
2007-08-30 14:12 ` Andreas Dilger
2007-08-30 19:12 ` Mingming Cao
2007-08-30 20:51 ` Andreas Dilger
2007-09-01 0:25 ` Mingming Cao
2007-09-04 6:29 ` Andreas Dilger
2007-10-15 1:51 ` Theodore Tso
2007-08-30 15:38 ` [PATCH 4/8][e2fsprogs] add new macro to get descriptor number Valerie Clement
2007-10-15 2:09 ` Theodore Tso
2007-08-30 15:39 ` [PATCH 5/8][e2fsprogs] add new inline functions to get blocks counters Valerie Clement
2007-08-30 15:40 ` [PATCH 6/8][e2fsprogs] change access to the filesystem blocks counter Valerie Clement
2007-08-30 15:42 ` [PATCH 7/8][e2fsprogs] change access to the reserved " Valerie Clement
2007-08-30 15:43 ` [PATCH 8/8][e2fsprogs] change access to the free " Valerie Clement
2007-08-30 16:20 ` [PATCH 1/8][e2fsprogs] use existing function ext2fs_group_first_block() Valerie Clement
2007-10-14 15:12 ` [PATCH 0/8][e2fsprogs] 64-bit block number support - first part Theodore Tso
2007-10-17 14:13 ` Valerie Clement [this message]
2007-10-17 14:53 ` Theodore Tso
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=4716189E.7060301@bull.net \
--to=valerie.clement@bull.net \
--cc=linux-ext4@vger.kernel.org \
--cc=tytso@mit.edu \
/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.