2007-07-01 07:38:52

by Mingming Cao

[permalink] [raw]
Subject: [EXT4 set 9][PATCH 4/5]Morecleanups:ext4_extent_compilation_fixes

From: Dmitry Monakhov <[email protected]>
Subject: ext4: extent compilation fixes

Fix compilation with EXT_DEBUG, also fix leXX_to_cpu convertions.

Signed-off-by: Dmitry Monakhov <[email protected]>
Acked-by: Alex Tomas <[email protected]>
Signed-off-by: Dave Kleikamp <[email protected]>
---
fs/ext4/extents.c | 18 ++++++++++--------
1 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
index 6f72dcb..12fe3d7 100644
--- a/fs/ext4/extents.c
+++ b/fs/ext4/extents.c
@@ -382,13 +382,14 @@ ext4_ext_binsearch_idx(struct inode *inode, struct ext4_ext_path *path, int bloc
r = m - 1;
else
l = m + 1;
- ext_debug("%p(%u):%p(%u):%p(%u) ", l, l->ei_block,
- m, m->ei_block, r, r->ei_block);
+ ext_debug("%p(%u):%p(%u):%p(%u) ", l, le32_to_cpu(l->ei_block),
+ m, le32_to_cpu(m->ei_block),
+ r, le32_to_cpu(r->ei_block));
}

path->p_idx = l - 1;
ext_debug(" -> %d->%lld ", le32_to_cpu(path->p_idx->ei_block),
- idx_block(path->p_idx));
+ idx_pblock(path->p_idx));

#ifdef CHECK_BINSEARCH
{
@@ -447,8 +448,9 @@ ext4_ext_binsearch(struct inode *inode, struct ext4_ext_path *path, int block)
r = m - 1;
else
l = m + 1;
- ext_debug("%p(%u):%p(%u):%p(%u) ", l, l->ee_block,
- m, m->ee_block, r, r->ee_block);
+ ext_debug("%p(%u):%p(%u):%p(%u) ", l, le32_to_cpu(l->ee_block),
+ m, le32_to_cpu(m->ee_block),
+ r, le32_to_cpu(r->ee_block));
}

path->p_ext = l - 1;
@@ -580,7 +582,7 @@ static int ext4_ext_insert_index(handle_t *handle, struct inode *inode,
if (curp->p_idx != EXT_LAST_INDEX(curp->p_hdr)) {
len = (len - 1) * sizeof(struct ext4_extent_idx);
len = len < 0 ? 0 : len;
- ext_debug("insert new index %d after: %d. "
+ ext_debug("insert new index %d after: %llu. "
"move %d from 0x%p to 0x%p\n",
logical, ptr, len,
(curp->p_idx + 1), (curp->p_idx + 2));
@@ -591,7 +593,7 @@ static int ext4_ext_insert_index(handle_t *handle, struct inode *inode,
/* insert before */
len = len * sizeof(struct ext4_extent_idx);
len = len < 0 ? 0 : len;
- ext_debug("insert new index %d before: %d. "
+ ext_debug("insert new index %d before: %llu. "
"move %d from 0x%p to 0x%p\n",
logical, ptr, len,
curp->p_idx, (curp->p_idx + 1));
@@ -791,7 +793,7 @@ static int ext4_ext_split(handle_t *handle, struct inode *inode,
BUG_ON(EXT_MAX_INDEX(path[i].p_hdr) !=
EXT_LAST_INDEX(path[i].p_hdr));
while (path[i].p_idx <= EXT_MAX_INDEX(path[i].p_hdr)) {
- ext_debug("%d: move %d:%d in new index %llu\n", i,
+ ext_debug("%d: move %d:%llu in new index %llu\n", i,
le32_to_cpu(path[i].p_idx->ei_block),
idx_pblock(path[i].p_idx),
newblock);


2007-07-11 06:19:16

by Andrew Morton

[permalink] [raw]
Subject: Re: [EXT4 set 9][PATCH 4/5]Morecleanups:ext4_extent_compilation_fixes

On Sun, 01 Jul 2007 03:38:51 -0400 Mingming Cao <[email protected]> wrote:

> Subject: [EXT4 set 9][PATCH 4/5]Morecleanups:ext4_extent_compilation_fixes
> Date: Sun, 01 Jul 2007 03:38:51 -0400
> Sender: [email protected]
> Organization: IBM Linux Technology Center
> X-Mailer: Evolution 2.8.0 (2.8.0-33.el5)
>
> From: Dmitry Monakhov <[email protected]>
> Subject: ext4: extent compilation fixes

I hope this patch series will hit git with nice titles like "ext4: extent
compilation fixes" and not funny ones like
"Morecleanups:ext4_extent_compilation_fixes".

> Fix compilation with EXT_DEBUG, also fix leXX_to_cpu convertions.

"conversions".

2007-07-16 07:20:02

by Mingming Cao

[permalink] [raw]
Subject: Re: [EXT4 set 9][PATCH 4/5]Morecleanups:ext4_extent_compilation_fixes

On Tue, 2007-07-10 at 23:18 -0700, Andrew Morton wrote:
> On Sun, 01 Jul 2007 03:38:51 -0400 Mingming Cao <[email protected]> wrote:
>
> > Subject: [EXT4 set 9][PATCH 4/5]Morecleanups:ext4_extent_compilation_fixes
> > Date: Sun, 01 Jul 2007 03:38:51 -0400
> > Sender: [email protected]
> > Organization: IBM Linux Technology Center
> > X-Mailer: Evolution 2.8.0 (2.8.0-33.el5)
> >
> > From: Dmitry Monakhov <[email protected]>
> > Subject: ext4: extent compilation fixes
>
> I hope this patch series will hit git with nice titles like "ext4: extent
> compilation fixes" and not funny ones like
> "Morecleanups:ext4_extent_compilation_fixes".
>
Sure, the changelog keep the nice tile "Subject: ext4: extent
compilation fixes"

> > Fix compilation with EXT_DEBUG, also fix leXX_to_cpu convertions.
>
> "conversions".

Done.

Thanks,
Mingming