Re: [U-Boot] [PATCH v2] fs/ext4: Support device block sizes != 512 bytes.

2013-04-30 Thread Tom Rini
On Tue, Mar 26, 2013 at 08:05:59AM -0400, egbert.e...@gmail.com wrote:

 From: Egbert Eich e...@suse.com
 
 The 512 byte block size was hard coded in the ext4 file systems. Large
 harddisks today support bigger block sizes typically 4096 bytes.
 
 Signed-off-by: Egbert Eich e...@suse.com
 ---
  fs/ext4/dev.c  |   60 +++
  fs/ext4/ext4_common.c  |   42 ++---
  fs/ext4/ext4_common.h  |2 +-
  fs/ext4/ext4_journal.c |6 +---
  fs/ext4/ext4_write.c   |   32 +
  fs/ext4/ext4fs.c   |   14 ++
  include/ext4fs.h   |1 +
  include/ext_common.h   |   12 ++---
  8 files changed, 94 insertions(+), 75 deletions(-)

This breaks the sandbox and trats boards (same problem on both), please
fix, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] fs/ext4: Support device block sizes != 512 bytes.

2013-03-26 Thread egbert . eich
From: Egbert Eich e...@suse.com

The 512 byte block size was hard coded in the ext4 file systems. Large
harddisks today support bigger block sizes typically 4096 bytes.

Signed-off-by: Egbert Eich e...@suse.com
---
 fs/ext4/dev.c  |   60 +++
 fs/ext4/ext4_common.c  |   42 ++---
 fs/ext4/ext4_common.h  |2 +-
 fs/ext4/ext4_journal.c |6 +---
 fs/ext4/ext4_write.c   |   32 +
 fs/ext4/ext4fs.c   |   14 ++
 include/ext4fs.h   |1 +
 include/ext_common.h   |   12 ++---
 8 files changed, 94 insertions(+), 75 deletions(-)

diff --git a/fs/ext4/dev.c b/fs/ext4/dev.c
index 464a67d..7b6fd16 100644
--- a/fs/ext4/dev.c
+++ b/fs/ext4/dev.c
@@ -40,6 +40,7 @@
 #include config.h
 #include ext4fs.h
 #include ext_common.h
+#include ext4_common.h
 
 unsigned long part_offset;
 
@@ -48,37 +49,41 @@ static disk_partition_t *part_info;
 
 void ext4fs_set_blk_dev(block_dev_desc_t *rbdd, disk_partition_t *info)
 {
+   assert(rbdd-blksz == (1  rbdd-log2blksz));
ext4fs_block_dev_desc = rbdd;
part_info = info;
part_offset = info-start;
-   get_fs()-total_sect = (info-size * info-blksz) / SECTOR_SIZE;
+   get_fs()-total_sect = (info-size * info-blksz) 
+   get_fs()-dev_desc-log2blksz;
get_fs()-dev_desc = rbdd;
 }
 
 int ext4fs_devread(int sector, int byte_offset, int byte_len, char *buf)
 {
-   ALLOC_CACHE_ALIGN_BUFFER(char, sec_buf, SECTOR_SIZE);
unsigned block_len;
+   int log2blksz = ext4fs_block_dev_desc-log2blksz;
+   ALLOC_CACHE_ALIGN_BUFFER(char, sec_buf, (ext4fs_block_dev_desc ?
+ext4fs_block_dev_desc-blksz :
+0));
+   if (ext4fs_block_dev_desc == NULL) {
+   printf(** Invalid Block Device Descriptor (NULL)\n);
+   return 0;
+   }
 
/* Check partition boundaries */
if ((sector  0)
-   || ((sector + ((byte_offset + byte_len - 1)  SECTOR_BITS)) =
-   part_info-size)) {
+   || ((sector + ((byte_offset + byte_len - 1)  log2blksz))
+   = part_info-size)) {
printf(%s read outside partition %d\n, __func__, sector);
return 0;
}
 
/* Get the read to the beginning of a partition */
-   sector += byte_offset  SECTOR_BITS;
-   byte_offset = SECTOR_SIZE - 1;
+   sector += byte_offset  log2blksz;
+   byte_offset = ext4fs_block_dev_desc-blksz - 1;
 
debug( %d, %d, %d\n, sector, byte_offset, byte_len);
 
-   if (ext4fs_block_dev_desc == NULL) {
-   printf(** Invalid Block Device Descriptor (NULL)\n);
-   return 0;
-   }
-
if (byte_offset != 0) {
/* read first part which isn't aligned with start of sector */
if (ext4fs_block_dev_desc-
@@ -89,9 +94,12 @@ int ext4fs_devread(int sector, int byte_offset, int 
byte_len, char *buf)
return 0;
}
memcpy(buf, sec_buf + byte_offset,
-   min(SECTOR_SIZE - byte_offset, byte_len));
-   buf += min(SECTOR_SIZE - byte_offset, byte_len);
-   byte_len -= min(SECTOR_SIZE - byte_offset, byte_len);
+   min(ext4fs_block_dev_desc-blksz
+   - byte_offset, byte_len));
+   buf += min(ext4fs_block_dev_desc-blksz
+  - byte_offset, byte_len);
+   byte_len -= min(ext4fs_block_dev_desc-blksz
+   - byte_offset, byte_len);
sector++;
}
 
@@ -99,12 +107,12 @@ int ext4fs_devread(int sector, int byte_offset, int 
byte_len, char *buf)
return 1;
 
/* read sector aligned part */
-   block_len = byte_len  ~(SECTOR_SIZE - 1);
+   block_len = byte_len  ~(ext4fs_block_dev_desc-blksz - 1);
 
if (block_len == 0) {
-   ALLOC_CACHE_ALIGN_BUFFER(u8, p, SECTOR_SIZE);
+   ALLOC_CACHE_ALIGN_BUFFER(u8, p, ext4fs_block_dev_desc-blksz);
 
-   block_len = SECTOR_SIZE;
+   block_len = ext4fs_block_dev_desc-blksz;
ext4fs_block_dev_desc-block_read(ext4fs_block_dev_desc-dev,
  part_info-start + sector,
  1, (unsigned long *)p);
@@ -114,16 +122,16 @@ int ext4fs_devread(int sector, int byte_offset, int 
byte_len, char *buf)
 
if (ext4fs_block_dev_desc-block_read(ext4fs_block_dev_desc-dev,
   part_info-start + sector,
-  block_len / SECTOR_SIZE,
+  block_len  log2blksz,
   (unsigned long *) buf) !=
-