On Tue, 2007-06-26 at 11:48 +0200, Valerie Clement wrote: > Shouldn't it be : > > struct commit_header *tmp = (struct commit_header*)(bh->b_data + i); > Ohhh, yes you are right. This is the correct thing to do. This patch which is been used has some endian-ness bugs. I had sent an updated patch for 2.6.22-rc5 on 19 June. I think we should be using the updated patch else it may break on big-endian machines. Even the updated patch has this bug i.e "struct commit_header *tmp =...." Thanks, Girish. - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html