Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1288735
| Path | csiph.com!eternal-september.org!feeder.eternal-september.org!aioe.org!bofh.it!news.nic.it!robomod |
|---|---|
| From | Mikulas Patocka <mpatocka@redhat.com> |
| Newsgroups | linux.kernel |
| Subject | [PATCH 9/15] dm: implement copy |
| Date | Thu, 10 Dec 2015 18:40:04 +0100 |
| Message-ID | <qEdmc-OR-67@gated-at.bofh.it> (permalink) |
| References | <qEdma-OR-9@gated-at.bofh.it> |
| X-Original-To | "James E.J. Bottomley" <JBottomley@odin.com>, "Martin K. Petersen" <martin.petersen@oracle.com>, Jens Axboe <axboe@kernel.dk>, Mike Snitzer <msnitzer@redhat.com>, Jonathan Brassow <jbrassow@redhat.com> |
| X-X-Sender | mpatocka@file01.intranet.prod.int.rdu2.redhat.com |
| User-Agent | Alpine 2.02 (LRH 1266 2009-07-14) |
| MIME-Version | 1.0 |
| Content-Type | TEXT/PLAIN; charset=US-ASCII |
| X-Scanned-By | MIMEDefang 2.68 on 10.5.11.23 |
| Sender | robomod@news.nic.it |
| List-ID | <linux-kernel.vger.kernel.org> |
| X-Mailing-List | linux-kernel@vger.kernel.org |
| Approved | robomod@news.nic.it |
| Lines | 123 |
| Organization | linux.* mail to news gateway |
| X-Original-Cc | dm-devel@redhat.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-block@vger.kernel.org |
| X-Original-Date | Thu, 10 Dec 2015 12:30:49 -0500 (EST) |
| X-Original-Message-ID | <alpine.LRH.2.02.1512101213420.25927@file01.intranet.prod.int.rdu2.redhat.com> |
| X-Original-References | <alpine.LRH.2.02.1512101145430.25927@file01.intranet.prod.int.rdu2.redhat.com> |
| X-Original-Sender | linux-kernel-owner@vger.kernel.org |
| Xref | csiph.com linux.kernel:1288735 |
Show key headers only | View raw
This patch implements basic copy support for device mapper core.
Individual targets can enable copy support by setting ti->copy_supported.
Device mapper device advertises copy support if at least one target
supports copy and for this target, at least one underlying device supports
copy.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
---
drivers/md/dm-table.c | 16 ++++++++++++++++
drivers/md/dm.c | 27 +++++++++++++++++++++++++++
include/linux/device-mapper.h | 5 +++++
3 files changed, 48 insertions(+)
Index: linux-4.4-rc4/drivers/md/dm.c
===================================================================
--- linux-4.4-rc4.orig/drivers/md/dm.c 2015-12-10 17:04:05.000000000 +0100
+++ linux-4.4-rc4/drivers/md/dm.c 2015-12-10 17:05:48.000000000 +0100
@@ -1679,6 +1679,31 @@ static int __send_write_same(struct clon
return __send_changing_extent_only(ci, get_num_write_same_bios, NULL);
}
+static int __send_copy(struct clone_info *ci)
+{
+ struct dm_target *ti;
+ sector_t bound;
+
+ ti = dm_table_find_target(ci->map, ci->sector);
+ if (!dm_target_is_valid(ti))
+ return -EIO;
+
+ if (!ti->copy_supported)
+ return -EOPNOTSUPP;
+
+ bound = max_io_len(ci->sector, ti);
+
+ if (unlikely(ci->sector_count > bound))
+ return -EOPNOTSUPP;
+
+ __clone_and_map_simple_bio(ci, ti, 0, NULL);
+
+ ci->sector += ci->sector_count;
+ ci->sector_count = 0;
+
+ return 0;
+}
+
/*
* Select the correct strategy for processing a non-flush bio.
*/
@@ -1692,6 +1717,8 @@ static int __split_and_process_non_flush
return __send_discard(ci);
else if (unlikely(bio->bi_rw & REQ_WRITE_SAME))
return __send_write_same(ci);
+ else if (unlikely(bio->bi_rw & REQ_COPY))
+ return __send_copy(ci);
ti = dm_table_find_target(ci->map, ci->sector);
if (!dm_target_is_valid(ti))
Index: linux-4.4-rc4/include/linux/device-mapper.h
===================================================================
--- linux-4.4-rc4.orig/include/linux/device-mapper.h 2015-12-10 17:04:05.000000000 +0100
+++ linux-4.4-rc4/include/linux/device-mapper.h 2015-12-10 17:04:59.000000000 +0100
@@ -271,6 +271,11 @@ struct dm_target {
* Set if this target does not return zeroes on discarded blocks.
*/
bool discard_zeroes_data_unsupported:1;
+
+ /*
+ * Set if the target supports XCOPY.
+ */
+ bool copy_supported:1;
};
/* Each target can link one of these into the table */
Index: linux-4.4-rc4/drivers/md/dm-table.c
===================================================================
--- linux-4.4-rc4.orig/drivers/md/dm-table.c 2015-12-10 17:03:51.000000000 +0100
+++ linux-4.4-rc4/drivers/md/dm-table.c 2015-12-10 17:04:59.000000000 +0100
@@ -286,6 +286,11 @@ static int device_area_is_invalid(struct
limits->logical_block_size >> SECTOR_SHIFT;
char b[BDEVNAME_SIZE];
+ if (ti->copy_supported)
+ limits->max_copy_sectors =
+ min_not_zero(limits->max_copy_sectors,
+ bdev_get_queue(bdev)->limits.max_copy_sectors);
+
/*
* Some devices exist without request functions,
* such as loop devices not yet bound to backing files.
@@ -1256,6 +1261,13 @@ int dm_calculate_queue_limits(struct dm_
ti = dm_table_get_target(table, i++);
+ if (ti->begin)
+ ti_limits.copy_boundary = min(ti_limits.copy_boundary,
+ (unsigned char)__ffs64(ti->begin));
+ if (ti->max_io_len)
+ ti_limits.copy_boundary = min(ti_limits.copy_boundary,
+ (unsigned char)__ffs(ti->max_io_len));
+
if (!ti->type->iterate_devices)
goto combine_limits;
@@ -1289,6 +1301,10 @@ combine_limits:
dm_device_name(table->md),
(unsigned long long) ti->begin,
(unsigned long long) ti->len);
+
+ limits->max_copy_sectors =
+ min_not_zero(limits->max_copy_sectors,
+ ti_limits.max_copy_sectors);
}
return validate_hardware_logical_block_alignment(table, limits);
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/15] copy offload patches Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 7/15] scsi xcopy: keep cache of failures Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 5/15] block copy: use asynchronous notification Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 6/15] scsi xcopy: suppress error messages Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 4/15] block copy: use a timer to fix a theoretical deadlock Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 14/15] dm kcopyd: change mutex to spinlock Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 9/15] dm: implement copy Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 1/15] block copy: initial XCOPY offload support Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
[PATCH 12/15] dm kcopyd: introduce the function submit_job Mikulas Patocka <mpatocka@redhat.com> - 2015-12-10 18:40 +0100
Re: [PATCH 0/15] copy offload patches "Martin K. Petersen" <martin.petersen@oracle.com> - 2015-12-10 23:40 +0100
Re: [dm-devel] [PATCH 0/15] copy offload patches Mike Christie <michaelc@cs.wisc.edu> - 2015-12-11 06:10 +0100
Re: [dm-devel] [PATCH 0/15] copy offload patches Christoph Hellwig <hch@infradead.org> - 2015-12-11 21:00 +0100
csiph-web