All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Snitzer <snitzer@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: dm-devel@redhat.com, Zdenek Kabelac <zkabelac@redhat.com>
Subject: Re: [dm-devel] can we fix some dm snapshot crashes please? :)
Date: Fri, 7 May 2021 11:20:25 -0400	[thread overview]
Message-ID: <YJVauS60BjSFkW9/@redhat.com> (raw)
In-Reply-To: <84eaa68f-3ea3-8313-2953-46375dce9f86@msgid.tls.msk.ru>

On Fri, May 07 2021 at 10:10P -0400,
Michael Tokarev <mjt@tls.msk.ru> wrote:

> 07.05.2021 15:31, Zdenek Kabelac wrote:
> > Dne 07. 05. 21 v 12:31 Michael Tokarev napsal(a):
> ...
> > >   sz=$(blockdev --getsize /dev/loop0)
> > >   dmsetup create base --table "0 $sz snapshot-origin /dev/loop0"
> > >   # and now the crash
> > >   mkfs.ext4 /dev/mapper/base
> 
> > Yes reproducible - can you please open BZ report here:
> > 
> > https://bugzilla.redhat.com/enter_bug.cgi?product=LVM%20and%20device-mapper
> 
> Ok, will do, thanks!

Thanks.  But when you do, please tone down the lamenting of how slow
snapshot crashes have been fixed -- or don't.  But that line of
rhetoric shows you're full of it.

> > I assume you are aware you are trying to using snapshot target in wrong way,
> > but it should not be crashing kernel.
> 
> Actually I'm not aware about the wrong way. The documentation is a bit lacking
> in this area. There are a few (maybe 2 or 3) guides describing the steps to do,
> most involves first using an 1:1 linear table, next replacing it on-the-fly with
> snapshot-origin table, but nowhere it is told that I can't create snapshot-origin
> directly, or that I need at least one snapshot device, or that the origin of the
> snapshot-origin table should be a dm device, or many many other variations.
> 
> I just tried to create a snapshot to be able to revert changes if something goes
> wrong, and since I don't use lvm I thought that plain dmsetup should do, but
> discovered that it doesn't quite work and it actually is a dangerous thing to use.. :)

Yes, it is dangerous to stab in the dark like you clearly are doing.

--
dm-devel mailing list
dm-devel@redhat.com
https://listman.redhat.com/mailman/listinfo/dm-devel


  reply	other threads:[~2021-05-07 15:20 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-07 10:31 [dm-devel] can we fix some dm snapshot crashes please? :) Michael Tokarev
2021-05-07 12:31 ` Zdenek Kabelac
2021-05-07 14:10   ` Michael Tokarev
2021-05-07 15:20     ` Mike Snitzer [this message]
2021-05-07 18:42       ` Michael Tokarev

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=YJVauS60BjSFkW9/@redhat.com \
    --to=snitzer@redhat.com \
    --cc=dm-devel@redhat.com \
    --cc=mjt@tls.msk.ru \
    --cc=zkabelac@redhat.com \
    /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.