All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Rabbitson <rabbit+list@rabbit.us>
To: Andre Noll <maan@systemlinux.org>
Cc: LinuxRaid <linux-raid@vger.kernel.org>
Subject: Re: [PATCH 0/2] md: More block -> sector conversions.
Date: Mon, 26 Jan 2009 18:26:08 +0100	[thread overview]
Message-ID: <497DF230.1080908@rabbit.us> (raw)
In-Reply-To: <20090126160814.GD17185@skl-net.de>

Andre Noll wrote:
> On 16:31, Peter Rabbitson wrote:
> 
>> Is it possible that the following issue is related to your earlier
>> patches: http://marc.info/?l=linux-raid&m=123202024116493&w=2
> 
> I don't think this is possible because you are using Linux 2.6.24.7
> which does not contain any of the block -> sector conversion changes.
> 
> Moreover, those changes should be invisible to userspace. In
> particular, mdadm does not need to know whether the kernel uses blocks
> or sectors for its internal representations. In fact, no changes to
> mdadm were made wrt. the block -> sector conversions.
> 
> Can you reproduce the incorrect output for the used device size
> with the recent master branch of the mdadm git tree?
> 

Hi Andre,

Yes I can:

root@Thesaurus:/home/rabbit/src/git/mdadm# ./mdadm --version
mdadm - v2.6.8 - 28th November 2008


root@Thesaurus:/home/rabbit/src/git/mdadm# ./mdadm -E /dev/sda1
/dev/sda1:
          Magic : a92b4efc
        Version : 1.1
    Feature Map : 0x0
     Array UUID : 6b11b1ba:78985745:b320fc1a:1db68bcf
           Name : Thesaurus:Crypta  (local to host Thesaurus)
  Creation Time : Sat Mar  8 16:33:41 2008
     Raid Level : raid5
   Raid Devices : 4

 Avail Dev Size : 578130828 (275.67 GiB 296.00 GB)
     Array Size : 1730162688 (825.01 GiB 885.84 GB)
  Used Dev Size : 576720896 (275.00 GiB 295.28 GB)
    Data Offset : 264 sectors
   Super Offset : 0 sectors
          State : active
    Device UUID : 85e0e62b:56129e5f:b0d82e78:2ace7d37

    Update Time : Mon Jan 26 18:24:19 2009
       Checksum : 61b214ff - correct
         Events : 772406

         Layout : left-symmetric
     Chunk Size : 2048K

    Array Slot : 5 (failed, failed, 2, 3, 0, 1)
   Array State : uUuu 2 failed


root@Thesaurus:/home/rabbit/src/git/mdadm# ./mdadm -D /dev/md5
/dev/md5:
        Version : 1.01
  Creation Time : Sat Mar  8 16:33:41 2008
     Raid Level : raid5
     Array Size : 865081344 (825.01 GiB 885.84 GB)
  Used Dev Size : 576720896 (550.00 GiB 590.56 GB)
   Raid Devices : 4
  Total Devices : 4
Preferred Minor : 5
    Persistence : Superblock is persistent

    Update Time : Mon Jan 26 18:24:34 2009
          State : clean
 Active Devices : 4
Working Devices : 4
 Failed Devices : 0
  Spare Devices : 0

         Layout : left-symmetric
     Chunk Size : 2048K

           Name : Thesaurus:Crypta  (local to host Thesaurus)
           UUID : 6b11b1ba:78985745:b320fc1a:1db68bcf
         Events : 772406

    Number   Major   Minor   RaidDevice State
       4       8       67        0      active sync   /dev/sde3
       5       8        1        1      active sync   /dev/sda1
       2       8       19        2      active sync   /dev/sdb3
       3       8       35        3      active sync   /dev/sdc3

  reply	other threads:[~2009-01-26 17:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-26 15:14 [PATCH 0/2] md: More block -> sector conversions Andre Noll
2009-01-26 15:14 ` [PATCH 1/2] md: Make mddev->size sector-based Andre Noll
2009-01-26 15:14 ` [PATCH 2/2] md: Represent raid device size in sectors Andre Noll
2009-01-26 15:31 ` [PATCH 0/2] md: More block -> sector conversions Peter Rabbitson
2009-01-26 16:08   ` Andre Noll
2009-01-26 17:26     ` Peter Rabbitson [this message]
2009-01-26 18:33       ` Andre Noll
2009-01-27  7:52         ` [PATCH] mdadm: Fix the used device size in mdadm -D output Andre Noll
2009-02-01 23:04           ` Neil Brown
2009-01-27  6:39 ` [PATCH 0/2] md: More block -> sector conversions Neil Brown

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=497DF230.1080908@rabbit.us \
    --to=rabbit+list@rabbit.us \
    --cc=linux-raid@vger.kernel.org \
    --cc=maan@systemlinux.org \
    /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.