Re: linux-next: Tree for Mar 4 (dm-cache-target)

[Mike Snitzer]

On Mon, Mar 04 2013 at  1:55pm -0500,
Randy Dunlap <rdun...@infradead.org> wrote:

> On 03/03/13 19:43, Stephen Rothwell wrote:
> > Hi all,
> > 
> > Changes since 20130301:
> > 
> > The device-mapper tree gained a conflict against Linus' tree.
> > 
> 
> 
> on i386:
> 
> drivers/built-in.o: In function `is_discarded_oblock':
> dm-cache-target.c:(.text+0x1ea28e): undefined reference to `__udivdi3'
> drivers/built-in.o: In function `remap_to_origin_clear_discard':
> dm-cache-target.c:(.text+0x1ea3ac): undefined reference to `__udivdi3'
> drivers/built-in.o: In function `remap_to_cache_dirty.part.21':
> dm-cache-target.c:(.text+0x1ea48f): undefined reference to `__udivdi3'
> drivers/built-in.o: In function `cache_ctr':
> dm-cache-target.c:(.text+0x1eaa2b): undefined reference to `__udivdi3'
> dm-cache-target.c:(.text+0x1eaa71): undefined reference to `__udivdi3'
> drivers/built-in.o:dm-cache-target.c:(.text+0x1eac51): more undefined 
> references to `__udivdi3' follow
> 
> 
> 
> Full randconfig file is attached.

I was able to reproduce using your randconfig file.

This patch fixed the build for my v3.9-rc1 tree:

diff --git a/drivers/md/dm-cache-target.c b/drivers/md/dm-cache-target.c
index 0f4e84b..96bfec1 100644
--- a/drivers/md/dm-cache-target.c
+++ b/drivers/md/dm-cache-target.c
@@ -415,7 +415,7 @@ static bool block_size_is_power_of_two(struct cache *cache)
 static dm_dblock_t oblock_to_dblock(struct cache *cache, dm_oblock_t oblock)
 {
        sector_t discard_blocks = cache->discard_block_size;
-       dm_block_t b = from_oblock(oblock);
+       sector_t b = from_oblock(oblock);
 
        if (!block_size_is_power_of_two(cache))
                (void) sector_div(discard_blocks, cache->sectors_per_block);
@@ -999,7 +999,7 @@ static void process_discard_bio(struct cache *cache, struct 
bio *bio)
 {
        dm_block_t start_block = dm_sector_div_up(bio->bi_sector,
                                                  cache->discard_block_size);
-       dm_block_t end_block = bio->bi_sector + bio_sectors(bio);
+       sector_t end_block = bio->bi_sector + bio_sectors(bio);
        dm_block_t b;
 
        (void) sector_div(end_block, cache->discard_block_size);
@@ -1801,7 +1801,7 @@ static int cache_create(struct cache_args *ca, struct 
cache **result)
        char **error = &ca->ti->error;
        struct cache *cache;
        struct dm_target *ti = ca->ti;
-       dm_block_t origin_blocks;
+       sector_t origin_blocks;
        struct dm_cache_metadata *cmd;
        bool may_format = ca->features.mode == CM_WRITE;
 
@@ -1845,7 +1845,7 @@ static int cache_create(struct cache_args *ca, struct 
cache **result)
        }
 
        if (ca->block_size & (ca->block_size - 1)) {
-               dm_block_t cache_size = ca->cache_sectors;
+               sector_t cache_size = ca->cache_sectors;
 
                cache->sectors_per_block_shift = -1;
                (void) sector_div(cache_size, ca->block_size);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/