search

junneyang / zumastor

Automatically exported from code.google.com/p/zumastor
0 stars 1 forks source link

incorrect/not-very-useful "dmsetup table" output #136

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
$ sudo dmsetup table
safe-debian-snap-2008-05-12_18:00:03: 0 199991296 ddsnap 253:0 253:2 4096
safe-debian-ddreal: 0 199991296 linear 9:0 384
unsafe-fast: 0 208617472 striped 2 128 8:19 41943424 8:4 384
safe-debian-snap-2008-05-12_19:00:03: 0 199991296 ddsnap 253:0 253:2 4096
safe-debian-snap-2008-05-12_16:00:03: 0 199991296 ddsnap 253:0 253:2 4096
unsafe-debian--ddsnap: 0 104857600 linear 8:19 146252160
safe-debian: 0 199991296 ddsnap 253:0 253:2 4096
safe-debian-snap-2008-05-12_17:00:03: 0 199991296 ddsnap 253:0 253:2 4096
safe-debian-snap-2008-05-12_15:00:54: 0 199991296 ddsnap 253:0 253:2 4096

The ddsnap specific part (as in 253:0 253:2 4096) is <origin-dev>
<snap-dev> <chunk-size>.

First, it's confusing because it's the reverse order as in create. And
we're missing the information on the snap id.

Other dm targets given an output that is suitable for input to dmsetup
create, so that dmsetup table one-dev | dmsetup create other-dev, creates a
copy of the original dev.

So ideally, it should ouput <snap-dev> <origin-dev> <control-socket>
<server-socket> <id>.

Please find a patch attached

Original issue reported on code.google.com by s.chaze...@gmail.com on 12 May 2008 at 6:41

Attachments:

GoogleCodeExporter commented 9 years ago 
Sorry, I messed up with the attachment.

Original comment by s.chaze...@gmail.com on 12 May 2008 at 6:42

Attachments:

GoogleCodeExporter commented 9 years ago 
Attaching just a diff, no e-mail included :)

Original comment by williama...@gmail.com on 12 May 2008 at 6:56

Attachments:

GoogleCodeExporter commented 9 years ago 
Updated for changes in r1661. (offset 1)

Original comment by williama...@gmail.com on 12 May 2008 at 7:01

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by daniel.r...@gmail.com on 14 May 2008 at 6:46

GoogleCodeExporter commented 9 years ago 
Patch has passed the uml cbtb tests and committed as revision 1669.
Thanks a lot, Stephane!

Jiaying

Original comment by jiayin...@gmail.com on 21 May 2008 at 10:29

GoogleCodeExporter commented 9 years ago

Original comment by jiahotc...@gmail.com on 28 May 2008 at 6:55