linux-toradex.git/drivers/md/dm-snap-transient.c, branch v4.16-rc6

dm snapshot: fix hung bios when copy error occurs

2016-01-09T01:03:05+00:00

When there is an error copying a chunk dm-snapshot can incorrectly hold
associated bios indefinitely, resulting in hung IO.

The function copy_callback sets pe->error if there was error copying the
chunk, and then calls complete_exception.  complete_exception calls
pending_complete on error, otherwise it calls commit_exception with
commit_callback (and commit_callback calls complete_exception).

The persistent exception store (dm-snap-persistent.c) assumes that calls
to prepare_exception and commit_exception are paired.
persistent_prepare_exception increases ps->pending_count and
persistent_commit_exception decreases it.

If there is a copy error, persistent_prepare_exception is called but
persistent_commit_exception is not.  This results in the variable
ps->pending_count never returning to zero and that causes some pending
exceptions (and their associated bios) to be held forever.

Fix this by unconditionally calling commit_exception regardless of
whether the copy was successful.  A new "valid" parameter is added to
commit_exception -- when the copy fails this parameter is set to zero so
that the chunk that failed to copy (and all following chunks) is not
recorded in the snapshot store.  Also, remove commit_callback now that
it is merely a wrapper around pending_complete.

Signed-off-by: Mikulas Patocka 
Signed-off-by: Mike Snitzer 
Cc: stable@vger.kernel.org

dm snapshot: add new persistent store option to support overflow

2015-10-09T20:57:03+00:00

Commit 76c44f6d80 introduced the possibly for "Overflow" to be reported
by the snapshot device's status.  Older userspace (e.g. lvm2) does not
handle the "Overflow" status response.

Fix this incompatibility by requiring newer userspace code, that can
cope with "Overflow", request the persistent store with overflow support
by using "PO" (Persistent with Overflow) for the snapshot store type.

Reported-by: Zdenek Kabelac 
Fixes: 76c44f6d80 ("dm snapshot: don't invalidate on-disk image on snapshot write overflow")
Reviewed-by: Mikulas Patocka 
Signed-off-by: Mike Snitzer

md: Add in export.h for files using EXPORT_SYMBOL

2011-10-31T23:31:19+00:00

These files were getting the defines for EXPORT_SYMBOL because
device.h was including module.h.  But we are going to put an
end to that.  So add the proper export.h include now.

Signed-off-by: Paul Gortmaker

dm snapshot: move cow ref from exception store to snap core

2009-12-10T23:52:12+00:00

Store the reference to the snapshot cow device in the core snapshot
code instead of each exception store.  It can be accessed through the
new function dm_snap_cow().  Exception stores should each now maintain a
reference to their parent snapshot struct.

This is cleaner and makes part of the forthcoming snapshot merge code simpler.

Signed-off-by: Mike Snitzer 
Signed-off-by: Alasdair G Kergon 
Reviewed-by: Jonathan Brassow 
Cc: Mikulas Patocka

dm snapshot: add allocated metadata to snapshot status

2009-12-10T23:52:11+00:00

Add number of sectors used by metadata to the end of the snapshot's status
line.

Renamed dm_exception_store_type's 'fraction_full' to 'usage'.  Renamed
arguments to be clearer about what is being returned.  Also added
'metadata_sectors'.

Signed-off-by: Mike Snitzer 
Signed-off-by: Alasdair G Kergon

dm snapshot: rename dm_snap_exception to dm_exception

2009-12-10T23:52:10+00:00

The exception structure is not necessarily just a snapshot
element (especially after we pull it out of dm-snap.c).

Renaming appropriately.

Signed-off-by: Jonathan Brassow 
Reviewed-by: Mike Snitzer 
Signed-off-by: Alasdair G Kergon

dm snapshot: move status to exception store

2009-04-02T18:55:35+00:00

Let the exception store types print out their status through
the new API, rather than having the snapshot code do it.

Adjust the buffer position to allow for the preceding DMEMIT in the
arguments to type->status().

Signed-off-by: Jonathan Brassow 
Signed-off-by: Alasdair G Kergon

dm snapshot: remove dm_snap header use

2009-04-02T18:55:33+00:00

Move useful functions out of dm-snap.h and stop using dm-snap.h.

Signed-off-by: Jonathan Brassow 
Signed-off-by: Alasdair G Kergon

dm exception store: move cow pointer

2009-04-02T18:55:33+00:00

Move COW device from snapshot to exception store.

Signed-off-by: Jonathan Brassow 
Signed-off-by: Alasdair G Kergon

dm exception store: move chunk_fields

2009-04-02T18:55:32+00:00

Move chunk fields from snapshot to exception store.

Signed-off-by: Jonathan Brassow 
Signed-off-by: Alasdair G Kergon