linux-toradex.git/drivers/md/dm-table.c, branch v3.15

dm table: add dm_table_run_md_queue_async

2014-03-27T20:56:24+00:00

Introduce dm_table_run_md_queue_async() to run the request_queue of the
mapped_device associated with a request-based DM table.

Also add dm_md_get_queue() wrapper to extract the request_queue from a
mapped_device.

Signed-off-by: Mike Snitzer 
Signed-off-by: Hannes Reinecke 
Reviewed-by: Jun'ichi Nomura

dm: make dm_table_alloc_md_mempools static

2014-03-27T20:56:23+00:00

Make the function dm_table_alloc_md_mempools static because it is not
called from another file.

Signed-off-by: Mikulas Patocka 
Signed-off-by: Mike Snitzer

dm table: remove unused buggy code that extends the targets array

2014-01-07T15:11:44+00:00

A device mapper table is allocated in the following way:
* The function dm_table_create is called, it gets the number of targets
  as an argument -- it allocates a targets array accordingly.
* For each target, we call dm_table_add_target.

If we add more targets than were specified in dm_table_create, the
function dm_table_add_target reallocates the targets array.  However,
this reallocation code is wrong - it moves the targets array to a new
location, while some target constructors hold pointers to the array in
the old location.

The following DM target drivers save the pointer to the target
structure, so they corrupt memory if the target array is moved:
multipath, raid, mirror, snapshot, stripe, switch, thin, verity.

Under normal circumstances, the reallocation function is not called
(because dm_table_create is called with the correct number of targets),
so the buggy reallocation code is not used.

Prior to the fix "dm table: fail dm_table_create on dm_round_up
overflow", the reallocation code could only be used in case the user
specifies too large a value in param->target_count, such as 0xffffffff.

Signed-off-by: Mikulas Patocka 
Signed-off-by: Mike Snitzer

dm table: fail dm_table_create on dm_round_up overflow

2013-12-10T21:34:27+00:00

The dm_round_up function may overflow to zero.  In this case,
dm_table_create() must fail rather than go on to allocate an empty array
with alloc_targets().

This fixes a possible memory corruption that could be caused by passing
too large a number in "param->target_count".

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

dm table: print error on preresume failure

2013-11-09T23:20:21+00:00

If preresume fails it is worth logging an error given that a device is
left suspended due to the failure.

This change was motivated by local preresume error logging that was
added to the cache target ("preresume failed").  Elevating this
target-agnostic context for the where the target-specific error occurred
relative to the DM core's callouts makes sense.

Signed-off-by: Mike Snitzer 
Signed-off-by: Joe Thornber

dm: allocate buffer for messages with small number of arguments using GFP_NOIO

2013-10-31T17:55:45+00:00

dm-mpath and dm-thin must process messages even if some device is
suspended, so we allocate argv buffer with GFP_NOIO. These messages have
a small fixed number of arguments.

On the other hand, dm-switch needs to process bulk data using messages
so excessive use of GFP_NOIO could cause trouble.

The patch also lowers the default number of arguments from 64 to 8, so
that there is smaller load on GFP_NOIO allocations.

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

dm ioctl: increase granularity of type_lock when loading table

2013-09-06T00:46:06+00:00

Hold the mapped device's type_lock before calling populate_table() since
it is where the table's type is determined based on the specified
targets.  There is no need to allow concurrent table loads to race to
establish the table's targets or type.

This eliminates the need to grab the lock in dm_table_set_type().

Also verify that the type_lock is held in both dm_set_md_type() and
dm_get_md_type().

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

dm: allow error target to replace bio-based and request-based targets

2013-09-06T00:46:05+00:00

It may be useful to switch a request-based table to the "error" target.
Enhance the DM core to allow a hybrid target_type which is capable of
handling either bios (via .map) or requests (via .map_rq).

Add a request-based map function (.map_rq) to the "error" target_type;
making it DM's first hybrid target.  Train dm_table_set_type() to prefer
the mapped device's established type (request-based or bio-based).  If
the mapped device doesn't have an established type default to making the
table with the hybrid target(s) bio-based.

Tested 'dmsetup wipe_table' to work on both bio-based and request-based
devices.

Signed-off-by: Mike Snitzer 
Signed-off-by: Joe Jin 
Signed-off-by: Jun'ichi Nomura 
Acked-by: Mikulas Patocka 
Signed-off-by: Alasdair G Kergon

dm: optimize use SRCU and RCU

2013-07-10T22:41:18+00:00

This patch removes "io_lock" and "map_lock" in struct mapped_device and
"holders" in struct dm_table and replaces these mechanisms with
sleepable-rcu.

Previously, the code would call "dm_get_live_table" and "dm_table_put" to
get and release table. Now, the code is changed to call "dm_get_live_table"
and "dm_put_live_table". dm_get_live_table locks sleepable-rcu and
dm_put_live_table unlocks it.

dm_get_live_table_fast/dm_put_live_table_fast can be used instead of
dm_get_live_table/dm_put_live_table. These *_fast functions use
non-sleepable RCU, so the caller must not block between them.

If the code changes active or inactive dm table, it must call
dm_sync_table before destroying the old table.

Signed-off-by: Mikulas Patocka 
Signed-off-by: Jun'ichi Nomura 
Signed-off-by: Alasdair G Kergon

dm table: fix write same support

2013-05-10T13:37:16+00:00

If device_not_write_same_capable() returns true then the iterate_devices
loop in dm_table_supports_write_same() should return false.

Reported-by: Bharata B Rao 
Signed-off-by: Mike Snitzer 
Cc: stable@vger.kernel.org # v3.8+
Signed-off-by: Alasdair G Kergon