2005-04-17 05:20:36 +07:00
|
|
|
/*
|
|
|
|
* Copyright (C) 2003 Sistina Software
|
2007-05-09 16:33:00 +07:00
|
|
|
* Copyright (C) 2006 Red Hat GmbH
|
2005-04-17 05:20:36 +07:00
|
|
|
*
|
|
|
|
* This file is released under the GPL.
|
|
|
|
*/
|
|
|
|
|
2009-12-11 06:51:57 +07:00
|
|
|
#include "dm.h"
|
|
|
|
|
2008-10-21 23:44:59 +07:00
|
|
|
#include <linux/device-mapper.h>
|
2005-04-17 05:20:36 +07:00
|
|
|
|
|
|
|
#include <linux/bio.h>
|
2014-06-28 02:29:04 +07:00
|
|
|
#include <linux/completion.h>
|
2005-04-17 05:20:36 +07:00
|
|
|
#include <linux/mempool.h>
|
|
|
|
#include <linux/module.h>
|
|
|
|
#include <linux/sched.h>
|
|
|
|
#include <linux/slab.h>
|
2008-04-25 04:02:01 +07:00
|
|
|
#include <linux/dm-io.h>
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2009-12-11 06:51:58 +07:00
|
|
|
#define DM_MSG_PREFIX "io"
|
|
|
|
|
|
|
|
#define DM_IO_MAX_REGIONS BITS_PER_LONG
|
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
struct dm_io_client {
|
|
|
|
mempool_t *pool;
|
|
|
|
struct bio_set *bios;
|
|
|
|
};
|
|
|
|
|
2009-12-11 06:51:58 +07:00
|
|
|
/*
|
|
|
|
* Aligning 'struct io' reduces the number of bits required to store
|
|
|
|
* its address. Refer to store_io_and_region_in_bio() below.
|
|
|
|
*/
|
2005-04-17 05:20:36 +07:00
|
|
|
struct io {
|
2008-04-25 03:43:14 +07:00
|
|
|
unsigned long error_bits;
|
2005-04-17 05:20:36 +07:00
|
|
|
atomic_t count;
|
2007-05-09 16:33:00 +07:00
|
|
|
struct dm_io_client *client;
|
2005-04-17 05:20:36 +07:00
|
|
|
io_notify_fn callback;
|
|
|
|
void *context;
|
2011-08-02 18:32:01 +07:00
|
|
|
void *vma_invalidate_address;
|
|
|
|
unsigned long vma_invalidate_size;
|
2009-12-11 06:51:58 +07:00
|
|
|
} __attribute__((aligned(DM_IO_MAX_REGIONS)));
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2009-12-11 06:51:57 +07:00
|
|
|
static struct kmem_cache *_dm_io_cache;
|
|
|
|
|
2007-05-09 16:33:01 +07:00
|
|
|
/*
|
|
|
|
* Create a client with mempool and bioset.
|
|
|
|
*/
|
2011-05-29 19:03:09 +07:00
|
|
|
struct dm_io_client *dm_io_client_create(void)
|
2007-05-09 16:33:01 +07:00
|
|
|
{
|
|
|
|
struct dm_io_client *client;
|
2013-09-13 05:06:12 +07:00
|
|
|
unsigned min_ios = dm_get_reserved_bio_based_ios();
|
2007-05-09 16:33:01 +07:00
|
|
|
|
|
|
|
client = kmalloc(sizeof(*client), GFP_KERNEL);
|
|
|
|
if (!client)
|
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
|
2013-09-13 05:06:12 +07:00
|
|
|
client->pool = mempool_create_slab_pool(min_ios, _dm_io_cache);
|
2007-05-09 16:33:01 +07:00
|
|
|
if (!client->pool)
|
|
|
|
goto bad;
|
|
|
|
|
2013-09-13 05:06:12 +07:00
|
|
|
client->bios = bioset_create(min_ios, 0);
|
2007-05-09 16:33:01 +07:00
|
|
|
if (!client->bios)
|
|
|
|
goto bad;
|
|
|
|
|
|
|
|
return client;
|
|
|
|
|
|
|
|
bad:
|
|
|
|
if (client->pool)
|
|
|
|
mempool_destroy(client->pool);
|
|
|
|
kfree(client);
|
|
|
|
return ERR_PTR(-ENOMEM);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(dm_io_client_create);
|
|
|
|
|
|
|
|
void dm_io_client_destroy(struct dm_io_client *client)
|
|
|
|
{
|
|
|
|
mempool_destroy(client->pool);
|
|
|
|
bioset_free(client->bios);
|
|
|
|
kfree(client);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(dm_io_client_destroy);
|
|
|
|
|
2005-04-17 05:20:36 +07:00
|
|
|
/*-----------------------------------------------------------------
|
|
|
|
* We need to keep track of which region a bio is doing io for.
|
2009-12-11 06:51:58 +07:00
|
|
|
* To avoid a memory allocation to store just 5 or 6 bits, we
|
|
|
|
* ensure the 'struct io' pointer is aligned so enough low bits are
|
|
|
|
* always zero and then combine it with the region number directly in
|
|
|
|
* bi_private.
|
2005-04-17 05:20:36 +07:00
|
|
|
*---------------------------------------------------------------*/
|
2009-12-11 06:51:58 +07:00
|
|
|
static void store_io_and_region_in_bio(struct bio *bio, struct io *io,
|
|
|
|
unsigned region)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2009-12-11 06:51:58 +07:00
|
|
|
if (unlikely(!IS_ALIGNED((unsigned long)io, DM_IO_MAX_REGIONS))) {
|
|
|
|
DMCRIT("Unaligned struct io pointer %p", io);
|
|
|
|
BUG();
|
|
|
|
}
|
|
|
|
|
|
|
|
bio->bi_private = (void *)((unsigned long)io | region);
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2009-12-11 06:51:58 +07:00
|
|
|
static void retrieve_io_and_region_from_bio(struct bio *bio, struct io **io,
|
|
|
|
unsigned *region)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2009-12-11 06:51:58 +07:00
|
|
|
unsigned long val = (unsigned long)bio->bi_private;
|
|
|
|
|
|
|
|
*io = (void *)(val & -(unsigned long)DM_IO_MAX_REGIONS);
|
|
|
|
*region = val & (DM_IO_MAX_REGIONS - 1);
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
/*-----------------------------------------------------------------
|
|
|
|
* We need an io object to keep track of the number of bios that
|
|
|
|
* have been dispatched for a particular io.
|
|
|
|
*---------------------------------------------------------------*/
|
2014-07-01 00:26:30 +07:00
|
|
|
static void complete_io(struct io *io)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2014-07-01 00:26:30 +07:00
|
|
|
unsigned long error_bits = io->error_bits;
|
|
|
|
io_notify_fn fn = io->callback;
|
|
|
|
void *context = io->context;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
if (io->vma_invalidate_size)
|
|
|
|
invalidate_kernel_vmap_range(io->vma_invalidate_address,
|
|
|
|
io->vma_invalidate_size);
|
2011-08-02 18:32:01 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
mempool_free(io, io->client->pool);
|
|
|
|
fn(error_bits, context);
|
|
|
|
}
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
static void dec_count(struct io *io, unsigned int region, int error)
|
|
|
|
{
|
|
|
|
if (error)
|
|
|
|
set_bit(region, &io->error_bits);
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
if (atomic_dec_and_test(&io->count))
|
|
|
|
complete_io(io);
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2007-09-27 17:47:43 +07:00
|
|
|
static void endio(struct bio *bio, int error)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2007-05-09 16:33:00 +07:00
|
|
|
struct io *io;
|
|
|
|
unsigned region;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
|
|
|
if (error && bio_data_dir(bio) == READ)
|
|
|
|
zero_fill_bio(bio);
|
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
/*
|
|
|
|
* The bio destructor in bio_put() may use the io object.
|
|
|
|
*/
|
2009-12-11 06:51:58 +07:00
|
|
|
retrieve_io_and_region_from_bio(bio, &io, ®ion);
|
2007-05-09 16:33:00 +07:00
|
|
|
|
2005-04-17 05:20:36 +07:00
|
|
|
bio_put(bio);
|
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
dec_count(io, region, error);
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
/*-----------------------------------------------------------------
|
|
|
|
* These little objects provide an abstraction for getting a new
|
|
|
|
* destination page for io.
|
|
|
|
*---------------------------------------------------------------*/
|
|
|
|
struct dpages {
|
|
|
|
void (*get_page)(struct dpages *dp,
|
|
|
|
struct page **p, unsigned long *len, unsigned *offset);
|
|
|
|
void (*next_page)(struct dpages *dp);
|
|
|
|
|
|
|
|
unsigned context_u;
|
|
|
|
void *context_ptr;
|
2011-08-02 18:32:01 +07:00
|
|
|
|
|
|
|
void *vma_invalidate_address;
|
|
|
|
unsigned long vma_invalidate_size;
|
2005-04-17 05:20:36 +07:00
|
|
|
};
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Functions for getting the pages from a list.
|
|
|
|
*/
|
|
|
|
static void list_get_page(struct dpages *dp,
|
|
|
|
struct page **p, unsigned long *len, unsigned *offset)
|
|
|
|
{
|
|
|
|
unsigned o = dp->context_u;
|
|
|
|
struct page_list *pl = (struct page_list *) dp->context_ptr;
|
|
|
|
|
|
|
|
*p = pl->page;
|
|
|
|
*len = PAGE_SIZE - o;
|
|
|
|
*offset = o;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void list_next_page(struct dpages *dp)
|
|
|
|
{
|
|
|
|
struct page_list *pl = (struct page_list *) dp->context_ptr;
|
|
|
|
dp->context_ptr = pl->next;
|
|
|
|
dp->context_u = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void list_dp_init(struct dpages *dp, struct page_list *pl, unsigned offset)
|
|
|
|
{
|
|
|
|
dp->get_page = list_get_page;
|
|
|
|
dp->next_page = list_next_page;
|
|
|
|
dp->context_u = offset;
|
|
|
|
dp->context_ptr = pl;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* Functions for getting the pages from a bvec.
|
|
|
|
*/
|
dm io: fix I/O to multiple destinations
Commit 003b5c5719f159f4f4bf97511c4702a0638313dd ("block: Convert drivers
to immutable biovecs") broke dm-mirror due to dm-io breakage.
dm-io had three possible iterators (DM_IO_PAGE_LIST, DM_IO_BVEC,
DM_IO_VMA) that iterate over pages where the I/O should be performed.
The switch to immutable biovecs changed the DM_IO_BVEC iterator to
DM_IO_BIO. Before this change the iterator stored the pointer to a bio
vector in the dpages structure. The iterator incremented the pointer in
the dpages structure as it advanced over the pages. After the immutable
biovecs change, the DM_IO_BIO iterator stores a pointer to the bio in
the dpages structure and uses bio_advance to change the bio as it
advances.
The problem is that the function dispatch_io stores the content of the
dpages structure into the variable old_pages and restores it before
issuing I/O to each of the devices. Before the change, the statement
"*dp = old_pages;" restored the iterator to its starting position.
After the change, struct dpages holds a pointer to the bio, thus the
statement "*dp = old_pages;" doesn't restore the iterator.
Consequently, in the context of dm-mirror: only the first mirror leg is
written correctly, the kernel locks up when trying to write the other
mirror legs because the number of sectors to write in the where->count
variable doesn't match the number of sectors returned by the iterator.
This patch fixes the bug by partially reverting the original patch - it
changes the code so that struct dpages holds a pointer to the bio vector,
so that the statement "*dp = old_pages;" restores the iterator correctly.
The field "context_u" holds the offset from the beginning of the current
bio vector entry, just like the "bio->bi_iter.bi_bvec_done" field.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
2014-02-13 04:37:30 +07:00
|
|
|
static void bio_get_page(struct dpages *dp, struct page **p,
|
|
|
|
unsigned long *len, unsigned *offset)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
dm io: fix I/O to multiple destinations
Commit 003b5c5719f159f4f4bf97511c4702a0638313dd ("block: Convert drivers
to immutable biovecs") broke dm-mirror due to dm-io breakage.
dm-io had three possible iterators (DM_IO_PAGE_LIST, DM_IO_BVEC,
DM_IO_VMA) that iterate over pages where the I/O should be performed.
The switch to immutable biovecs changed the DM_IO_BVEC iterator to
DM_IO_BIO. Before this change the iterator stored the pointer to a bio
vector in the dpages structure. The iterator incremented the pointer in
the dpages structure as it advanced over the pages. After the immutable
biovecs change, the DM_IO_BIO iterator stores a pointer to the bio in
the dpages structure and uses bio_advance to change the bio as it
advances.
The problem is that the function dispatch_io stores the content of the
dpages structure into the variable old_pages and restores it before
issuing I/O to each of the devices. Before the change, the statement
"*dp = old_pages;" restored the iterator to its starting position.
After the change, struct dpages holds a pointer to the bio, thus the
statement "*dp = old_pages;" doesn't restore the iterator.
Consequently, in the context of dm-mirror: only the first mirror leg is
written correctly, the kernel locks up when trying to write the other
mirror legs because the number of sectors to write in the where->count
variable doesn't match the number of sectors returned by the iterator.
This patch fixes the bug by partially reverting the original patch - it
changes the code so that struct dpages holds a pointer to the bio vector,
so that the statement "*dp = old_pages;" restores the iterator correctly.
The field "context_u" holds the offset from the beginning of the current
bio vector entry, just like the "bio->bi_iter.bi_bvec_done" field.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
2014-02-13 04:37:30 +07:00
|
|
|
struct bio_vec *bvec = dp->context_ptr;
|
|
|
|
*p = bvec->bv_page;
|
|
|
|
*len = bvec->bv_len - dp->context_u;
|
|
|
|
*offset = bvec->bv_offset + dp->context_u;
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2013-10-12 05:45:43 +07:00
|
|
|
static void bio_next_page(struct dpages *dp)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
dm io: fix I/O to multiple destinations
Commit 003b5c5719f159f4f4bf97511c4702a0638313dd ("block: Convert drivers
to immutable biovecs") broke dm-mirror due to dm-io breakage.
dm-io had three possible iterators (DM_IO_PAGE_LIST, DM_IO_BVEC,
DM_IO_VMA) that iterate over pages where the I/O should be performed.
The switch to immutable biovecs changed the DM_IO_BVEC iterator to
DM_IO_BIO. Before this change the iterator stored the pointer to a bio
vector in the dpages structure. The iterator incremented the pointer in
the dpages structure as it advanced over the pages. After the immutable
biovecs change, the DM_IO_BIO iterator stores a pointer to the bio in
the dpages structure and uses bio_advance to change the bio as it
advances.
The problem is that the function dispatch_io stores the content of the
dpages structure into the variable old_pages and restores it before
issuing I/O to each of the devices. Before the change, the statement
"*dp = old_pages;" restored the iterator to its starting position.
After the change, struct dpages holds a pointer to the bio, thus the
statement "*dp = old_pages;" doesn't restore the iterator.
Consequently, in the context of dm-mirror: only the first mirror leg is
written correctly, the kernel locks up when trying to write the other
mirror legs because the number of sectors to write in the where->count
variable doesn't match the number of sectors returned by the iterator.
This patch fixes the bug by partially reverting the original patch - it
changes the code so that struct dpages holds a pointer to the bio vector,
so that the statement "*dp = old_pages;" restores the iterator correctly.
The field "context_u" holds the offset from the beginning of the current
bio vector entry, just like the "bio->bi_iter.bi_bvec_done" field.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
2014-02-13 04:37:30 +07:00
|
|
|
struct bio_vec *bvec = dp->context_ptr;
|
|
|
|
dp->context_ptr = bvec + 1;
|
|
|
|
dp->context_u = 0;
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2013-10-12 05:45:43 +07:00
|
|
|
static void bio_dp_init(struct dpages *dp, struct bio *bio)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2013-10-12 05:45:43 +07:00
|
|
|
dp->get_page = bio_get_page;
|
|
|
|
dp->next_page = bio_next_page;
|
dm io: fix I/O to multiple destinations
Commit 003b5c5719f159f4f4bf97511c4702a0638313dd ("block: Convert drivers
to immutable biovecs") broke dm-mirror due to dm-io breakage.
dm-io had three possible iterators (DM_IO_PAGE_LIST, DM_IO_BVEC,
DM_IO_VMA) that iterate over pages where the I/O should be performed.
The switch to immutable biovecs changed the DM_IO_BVEC iterator to
DM_IO_BIO. Before this change the iterator stored the pointer to a bio
vector in the dpages structure. The iterator incremented the pointer in
the dpages structure as it advanced over the pages. After the immutable
biovecs change, the DM_IO_BIO iterator stores a pointer to the bio in
the dpages structure and uses bio_advance to change the bio as it
advances.
The problem is that the function dispatch_io stores the content of the
dpages structure into the variable old_pages and restores it before
issuing I/O to each of the devices. Before the change, the statement
"*dp = old_pages;" restored the iterator to its starting position.
After the change, struct dpages holds a pointer to the bio, thus the
statement "*dp = old_pages;" doesn't restore the iterator.
Consequently, in the context of dm-mirror: only the first mirror leg is
written correctly, the kernel locks up when trying to write the other
mirror legs because the number of sectors to write in the where->count
variable doesn't match the number of sectors returned by the iterator.
This patch fixes the bug by partially reverting the original patch - it
changes the code so that struct dpages holds a pointer to the bio vector,
so that the statement "*dp = old_pages;" restores the iterator correctly.
The field "context_u" holds the offset from the beginning of the current
bio vector entry, just like the "bio->bi_iter.bi_bvec_done" field.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
2014-02-13 04:37:30 +07:00
|
|
|
dp->context_ptr = __bvec_iter_bvec(bio->bi_io_vec, bio->bi_iter);
|
|
|
|
dp->context_u = bio->bi_iter.bi_bvec_done;
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2007-05-09 16:33:01 +07:00
|
|
|
/*
|
|
|
|
* Functions for getting the pages from a VMA.
|
|
|
|
*/
|
2005-04-17 05:20:36 +07:00
|
|
|
static void vm_get_page(struct dpages *dp,
|
|
|
|
struct page **p, unsigned long *len, unsigned *offset)
|
|
|
|
{
|
|
|
|
*p = vmalloc_to_page(dp->context_ptr);
|
|
|
|
*offset = dp->context_u;
|
|
|
|
*len = PAGE_SIZE - dp->context_u;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void vm_next_page(struct dpages *dp)
|
|
|
|
{
|
|
|
|
dp->context_ptr += PAGE_SIZE - dp->context_u;
|
|
|
|
dp->context_u = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void vm_dp_init(struct dpages *dp, void *data)
|
|
|
|
{
|
|
|
|
dp->get_page = vm_get_page;
|
|
|
|
dp->next_page = vm_next_page;
|
|
|
|
dp->context_u = ((unsigned long) data) & (PAGE_SIZE - 1);
|
|
|
|
dp->context_ptr = data;
|
|
|
|
}
|
|
|
|
|
2007-05-09 16:33:01 +07:00
|
|
|
/*
|
|
|
|
* Functions for getting the pages from kernel memory.
|
|
|
|
*/
|
|
|
|
static void km_get_page(struct dpages *dp, struct page **p, unsigned long *len,
|
|
|
|
unsigned *offset)
|
|
|
|
{
|
|
|
|
*p = virt_to_page(dp->context_ptr);
|
|
|
|
*offset = dp->context_u;
|
|
|
|
*len = PAGE_SIZE - dp->context_u;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void km_next_page(struct dpages *dp)
|
|
|
|
{
|
|
|
|
dp->context_ptr += PAGE_SIZE - dp->context_u;
|
|
|
|
dp->context_u = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void km_dp_init(struct dpages *dp, void *data)
|
|
|
|
{
|
|
|
|
dp->get_page = km_get_page;
|
|
|
|
dp->next_page = km_next_page;
|
|
|
|
dp->context_u = ((unsigned long) data) & (PAGE_SIZE - 1);
|
|
|
|
dp->context_ptr = data;
|
|
|
|
}
|
|
|
|
|
2005-04-17 05:20:36 +07:00
|
|
|
/*-----------------------------------------------------------------
|
|
|
|
* IO routines that accept a list of pages.
|
|
|
|
*---------------------------------------------------------------*/
|
2008-04-25 03:43:17 +07:00
|
|
|
static void do_region(int rw, unsigned region, struct dm_io_region *where,
|
2005-04-17 05:20:36 +07:00
|
|
|
struct dpages *dp, struct io *io)
|
|
|
|
{
|
|
|
|
struct bio *bio;
|
|
|
|
struct page *page;
|
|
|
|
unsigned long len;
|
|
|
|
unsigned offset;
|
|
|
|
unsigned num_bvecs;
|
|
|
|
sector_t remaining = where->count;
|
2012-03-08 02:09:37 +07:00
|
|
|
struct request_queue *q = bdev_get_queue(where->bdev);
|
2012-12-22 03:23:37 +07:00
|
|
|
unsigned short logical_block_size = queue_logical_block_size(q);
|
|
|
|
sector_t num_sectors;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2009-12-11 06:52:22 +07:00
|
|
|
/*
|
2010-09-03 16:56:19 +07:00
|
|
|
* where->count may be zero if rw holds a flush and we need to
|
|
|
|
* send a zero-sized flush.
|
2009-12-11 06:52:22 +07:00
|
|
|
*/
|
|
|
|
do {
|
2005-04-17 05:20:36 +07:00
|
|
|
/*
|
2009-12-11 06:51:58 +07:00
|
|
|
* Allocate a suitably sized-bio.
|
2005-04-17 05:20:36 +07:00
|
|
|
*/
|
2012-12-22 03:23:37 +07:00
|
|
|
if ((rw & REQ_DISCARD) || (rw & REQ_WRITE_SAME))
|
2012-03-08 02:09:37 +07:00
|
|
|
num_bvecs = 1;
|
|
|
|
else
|
|
|
|
num_bvecs = min_t(int, bio_get_nr_vecs(where->bdev),
|
|
|
|
dm_sector_div_up(remaining, (PAGE_SIZE >> SECTOR_SHIFT)));
|
|
|
|
|
2007-05-09 16:33:05 +07:00
|
|
|
bio = bio_alloc_bioset(GFP_NOIO, num_bvecs, io->client->bios);
|
2013-10-12 05:44:27 +07:00
|
|
|
bio->bi_iter.bi_sector = where->sector + (where->count - remaining);
|
2005-04-17 05:20:36 +07:00
|
|
|
bio->bi_bdev = where->bdev;
|
|
|
|
bio->bi_end_io = endio;
|
2009-12-11 06:51:58 +07:00
|
|
|
store_io_and_region_in_bio(bio, io, region);
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2012-03-08 02:09:37 +07:00
|
|
|
if (rw & REQ_DISCARD) {
|
2012-12-22 03:23:37 +07:00
|
|
|
num_sectors = min_t(sector_t, q->limits.max_discard_sectors, remaining);
|
2013-10-12 05:44:27 +07:00
|
|
|
bio->bi_iter.bi_size = num_sectors << SECTOR_SHIFT;
|
2012-12-22 03:23:37 +07:00
|
|
|
remaining -= num_sectors;
|
|
|
|
} else if (rw & REQ_WRITE_SAME) {
|
|
|
|
/*
|
|
|
|
* WRITE SAME only uses a single page.
|
|
|
|
*/
|
|
|
|
dp->get_page(dp, &page, &len, &offset);
|
|
|
|
bio_add_page(bio, page, logical_block_size, offset);
|
|
|
|
num_sectors = min_t(sector_t, q->limits.max_write_same_sectors, remaining);
|
2013-10-12 05:44:27 +07:00
|
|
|
bio->bi_iter.bi_size = num_sectors << SECTOR_SHIFT;
|
2012-12-22 03:23:37 +07:00
|
|
|
|
|
|
|
offset = 0;
|
|
|
|
remaining -= num_sectors;
|
|
|
|
dp->next_page(dp);
|
2012-03-08 02:09:37 +07:00
|
|
|
} else while (remaining) {
|
|
|
|
/*
|
|
|
|
* Try and add as many pages as possible.
|
|
|
|
*/
|
2005-04-17 05:20:36 +07:00
|
|
|
dp->get_page(dp, &page, &len, &offset);
|
|
|
|
len = min(len, to_bytes(remaining));
|
|
|
|
if (!bio_add_page(bio, page, len, offset))
|
|
|
|
break;
|
|
|
|
|
|
|
|
offset = 0;
|
|
|
|
remaining -= to_sector(len);
|
|
|
|
dp->next_page(dp);
|
|
|
|
}
|
|
|
|
|
|
|
|
atomic_inc(&io->count);
|
|
|
|
submit_bio(rw, bio);
|
2009-12-11 06:52:22 +07:00
|
|
|
} while (remaining);
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
|
|
|
static void dispatch_io(int rw, unsigned int num_regions,
|
2008-04-25 03:43:17 +07:00
|
|
|
struct dm_io_region *where, struct dpages *dp,
|
2005-04-17 05:20:36 +07:00
|
|
|
struct io *io, int sync)
|
|
|
|
{
|
|
|
|
int i;
|
|
|
|
struct dpages old_pages = *dp;
|
|
|
|
|
2009-12-11 06:51:58 +07:00
|
|
|
BUG_ON(num_regions > DM_IO_MAX_REGIONS);
|
|
|
|
|
2005-04-17 05:20:36 +07:00
|
|
|
if (sync)
|
2011-03-09 17:56:30 +07:00
|
|
|
rw |= REQ_SYNC;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
|
|
|
/*
|
|
|
|
* For multiple regions we need to be careful to rewind
|
|
|
|
* the dp object for each call to do_region.
|
|
|
|
*/
|
|
|
|
for (i = 0; i < num_regions; i++) {
|
|
|
|
*dp = old_pages;
|
2010-09-03 16:56:19 +07:00
|
|
|
if (where[i].count || (rw & REQ_FLUSH))
|
2005-04-17 05:20:36 +07:00
|
|
|
do_region(rw, i, where + i, dp, io);
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
2006-12-08 17:41:01 +07:00
|
|
|
* Drop the extra reference that we were holding to avoid
|
2005-04-17 05:20:36 +07:00
|
|
|
* the io being completed too early.
|
|
|
|
*/
|
|
|
|
dec_count(io, 0, 0);
|
|
|
|
}
|
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
struct sync_io {
|
|
|
|
unsigned long error_bits;
|
|
|
|
struct completion wait;
|
|
|
|
};
|
|
|
|
|
|
|
|
static void sync_io_complete(unsigned long error, void *context)
|
|
|
|
{
|
|
|
|
struct sync_io *sio = context;
|
|
|
|
|
|
|
|
sio->error_bits = error;
|
|
|
|
complete(&sio->wait);
|
|
|
|
}
|
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
static int sync_io(struct dm_io_client *client, unsigned int num_regions,
|
2008-04-25 03:43:17 +07:00
|
|
|
struct dm_io_region *where, int rw, struct dpages *dp,
|
2007-05-09 16:33:00 +07:00
|
|
|
unsigned long *error_bits)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
2014-07-01 00:26:30 +07:00
|
|
|
struct io *io;
|
|
|
|
struct sync_io sio;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
dm: unplug queues in threads
Remove an avoidable 3ms delay on some dm-raid1 and kcopyd I/O.
It is specified that any submitted bio without BIO_RW_SYNC flag may plug the
queue (i.e. block the requests from being dispatched to the physical device).
The queue is unplugged when the caller calls blk_unplug() function. Usually, the
sequence is that someone calls submit_bh to submit IO on a buffer. The IO plugs
the queue and waits (to be possibly joined with other adjacent bios). Then, when
the caller calls wait_on_buffer(), it unplugs the queue and submits the IOs to
the disk.
This was happenning:
When doing O_SYNC writes, function fsync_buffers_list() submits a list of
bios to dm_raid1, the bios are added to dm_raid1 write queue and kmirrord is
woken up.
fsync_buffers_list() calls wait_on_buffer(). That unplugs the queue, but
there are no bios on the device queue as they are still in the dm_raid1 queue.
wait_on_buffer() starts waiting until the IO is finished.
kmirrord is scheduled, kmirrord takes bios and submits them to the devices.
The submitted bio plugs the harddisk queue but there is no one to unplug it.
(The process that called wait_on_buffer() is already sleeping.)
So there is a 3ms timeout, after which the queues on the harddisks are
unplugged and requests are processed.
This 3ms timeout meant that in certain workloads (e.g. O_SYNC, 8kb writes),
dm-raid1 is 10 times slower than md raid1.
Every time we submit something asynchronously via dm_io, we must unplug the
queue actually to send the request to the device.
This patch adds an unplug call to kmirrord - while processing requests, it keeps
the queue plugged (so that adjacent bios can be merged); when it finishes
processing all the bios, it unplugs the queue to submit the bios.
It also fixes kcopyd which has the same potential problem. All kcopyd requests
are submitted with BIO_RW_SYNC.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Acked-by: Jens Axboe <jens.axboe@oracle.com>
2008-04-25 04:10:47 +07:00
|
|
|
if (num_regions > 1 && (rw & RW_MASK) != WRITE) {
|
2005-04-17 05:20:36 +07:00
|
|
|
WARN_ON(1);
|
|
|
|
return -EIO;
|
|
|
|
}
|
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
init_completion(&sio.wait);
|
|
|
|
|
|
|
|
io = mempool_alloc(client->pool, GFP_NOIO);
|
2009-12-11 06:51:58 +07:00
|
|
|
io->error_bits = 0;
|
|
|
|
atomic_set(&io->count, 1); /* see dispatch_io() */
|
|
|
|
io->client = client;
|
2014-07-01 00:26:30 +07:00
|
|
|
io->callback = sync_io_complete;
|
|
|
|
io->context = &sio;
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2011-08-02 18:32:01 +07:00
|
|
|
io->vma_invalidate_address = dp->vma_invalidate_address;
|
|
|
|
io->vma_invalidate_size = dp->vma_invalidate_size;
|
|
|
|
|
2009-12-11 06:51:58 +07:00
|
|
|
dispatch_io(rw, num_regions, where, dp, io, 1);
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
wait_for_completion_io(&sio.wait);
|
2005-04-17 05:20:36 +07:00
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
if (error_bits)
|
2014-07-01 00:26:30 +07:00
|
|
|
*error_bits = sio.error_bits;
|
2007-05-09 16:33:00 +07:00
|
|
|
|
2014-07-01 00:26:30 +07:00
|
|
|
return sio.error_bits ? -EIO : 0;
|
2005-04-17 05:20:36 +07:00
|
|
|
}
|
|
|
|
|
2007-05-09 16:33:00 +07:00
|
|
|
static int async_io(struct dm_io_client *client, unsigned int num_regions,
|
2008-04-25 03:43:17 +07:00
|
|
|
struct dm_io_region *where, int rw, struct dpages *dp,
|
2007-05-09 16:33:00 +07:00
|
|
|
io_notify_fn fn, void *context)
|
2005-04-17 05:20:36 +07:00
|
|
|
{
|
|
|
|
struct io *io;
|
|
|
|
|
dm: unplug queues in threads
Remove an avoidable 3ms delay on some dm-raid1 and kcopyd I/O.
It is specified that any submitted bio without BIO_RW_SYNC flag may plug the
queue (i.e. block the requests from being dispatched to the physical device).
The queue is unplugged when the caller calls blk_unplug() function. Usually, the
sequence is that someone calls submit_bh to submit IO on a buffer. The IO plugs
the queue and waits (to be possibly joined with other adjacent bios). Then, when
the caller calls wait_on_buffer(), it unplugs the queue and submits the IOs to
the disk.
This was happenning:
When doing O_SYNC writes, function fsync_buffers_list() submits a list of
bios to dm_raid1, the bios are added to dm_raid1 write queue and kmirrord is
woken up.
fsync_buffers_list() calls wait_on_buffer(). That unplugs the queue, but
there are no bios on the device queue as they are still in the dm_raid1 queue.
wait_on_buffer() starts waiting until the IO is finished.
kmirrord is scheduled, kmirrord takes bios and submits them to the devices.
The submitted bio plugs the harddisk queue but there is no one to unplug it.
(The process that called wait_on_buffer() is already sleeping.)
So there is a 3ms timeout, after which the queues on the harddisks are
unplugged and requests are processed.
This 3ms timeout meant that in certain workloads (e.g. O_SYNC, 8kb writes),
dm-raid1 is 10 times slower than md raid1.
Every time we submit something asynchronously via dm_io, we must unplug the
queue actually to send the request to the device.
This patch adds an unplug call to kmirrord - while processing requests, it keeps
the queue plugged (so that adjacent bios can be merged); when it finishes
processing all the bios, it unplugs the queue to submit the bios.
It also fixes kcopyd which has the same potential problem. All kcopyd requests
are submitted with BIO_RW_SYNC.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Acked-by: Jens Axboe <jens.axboe@oracle.com>
2008-04-25 04:10:47 +07:00
|
|
|
if (num_regions > 1 && (rw & RW_MASK) != WRITE) {
|
2005-04-17 05:20:36 +07:00
|
|
|
WARN_ON(1);
|
|
|
|
fn(1, context);
|
|
|
|
return -EIO;
|
|
|
|
}
|
|
|
|
|
2007-05-09 16:33:05 +07:00
|
|
|
io = mempool_alloc(client->pool, GFP_NOIO);
|
2008-04-25 03:43:14 +07:00
|
|
|
io->error_bits = 0;
|
2005-04-17 05:20:36 +07:00
|
|
|
atomic_set(&io->count, 1); /* see dispatch_io() */
|
2007-05-09 16:33:00 +07:00
|
|
|
io->client = client;
|
2005-04-17 05:20:36 +07:00
|
|
|
io->callback = fn;
|
|
|
|
io->context = context;
|
|
|
|
|
2011-08-02 18:32:01 +07:00
|
|
|
io->vma_invalidate_address = dp->vma_invalidate_address;
|
|
|
|
io->vma_invalidate_size = dp->vma_invalidate_size;
|
|
|
|
|
2005-04-17 05:20:36 +07:00
|
|
|
dispatch_io(rw, num_regions, where, dp, io, 0);
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
2011-08-02 18:32:01 +07:00
|
|
|
static int dp_init(struct dm_io_request *io_req, struct dpages *dp,
|
|
|
|
unsigned long size)
|
2007-05-09 16:33:01 +07:00
|
|
|
{
|
|
|
|
/* Set up dpages based on memory type */
|
2011-08-02 18:32:01 +07:00
|
|
|
|
|
|
|
dp->vma_invalidate_address = NULL;
|
|
|
|
dp->vma_invalidate_size = 0;
|
|
|
|
|
2007-05-09 16:33:01 +07:00
|
|
|
switch (io_req->mem.type) {
|
|
|
|
case DM_IO_PAGE_LIST:
|
|
|
|
list_dp_init(dp, io_req->mem.ptr.pl, io_req->mem.offset);
|
|
|
|
break;
|
|
|
|
|
2013-10-12 05:45:43 +07:00
|
|
|
case DM_IO_BIO:
|
|
|
|
bio_dp_init(dp, io_req->mem.ptr.bio);
|
2007-05-09 16:33:01 +07:00
|
|
|
break;
|
|
|
|
|
|
|
|
case DM_IO_VMA:
|
2011-08-02 18:32:01 +07:00
|
|
|
flush_kernel_vmap_range(io_req->mem.ptr.vma, size);
|
|
|
|
if ((io_req->bi_rw & RW_MASK) == READ) {
|
|
|
|
dp->vma_invalidate_address = io_req->mem.ptr.vma;
|
|
|
|
dp->vma_invalidate_size = size;
|
|
|
|
}
|
2007-05-09 16:33:01 +07:00
|
|
|
vm_dp_init(dp, io_req->mem.ptr.vma);
|
|
|
|
break;
|
|
|
|
|
|
|
|
case DM_IO_KMEM:
|
|
|
|
km_dp_init(dp, io_req->mem.ptr.addr);
|
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
|
|
|
return -EINVAL;
|
|
|
|
}
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
/*
|
dm: unplug queues in threads
Remove an avoidable 3ms delay on some dm-raid1 and kcopyd I/O.
It is specified that any submitted bio without BIO_RW_SYNC flag may plug the
queue (i.e. block the requests from being dispatched to the physical device).
The queue is unplugged when the caller calls blk_unplug() function. Usually, the
sequence is that someone calls submit_bh to submit IO on a buffer. The IO plugs
the queue and waits (to be possibly joined with other adjacent bios). Then, when
the caller calls wait_on_buffer(), it unplugs the queue and submits the IOs to
the disk.
This was happenning:
When doing O_SYNC writes, function fsync_buffers_list() submits a list of
bios to dm_raid1, the bios are added to dm_raid1 write queue and kmirrord is
woken up.
fsync_buffers_list() calls wait_on_buffer(). That unplugs the queue, but
there are no bios on the device queue as they are still in the dm_raid1 queue.
wait_on_buffer() starts waiting until the IO is finished.
kmirrord is scheduled, kmirrord takes bios and submits them to the devices.
The submitted bio plugs the harddisk queue but there is no one to unplug it.
(The process that called wait_on_buffer() is already sleeping.)
So there is a 3ms timeout, after which the queues on the harddisks are
unplugged and requests are processed.
This 3ms timeout meant that in certain workloads (e.g. O_SYNC, 8kb writes),
dm-raid1 is 10 times slower than md raid1.
Every time we submit something asynchronously via dm_io, we must unplug the
queue actually to send the request to the device.
This patch adds an unplug call to kmirrord - while processing requests, it keeps
the queue plugged (so that adjacent bios can be merged); when it finishes
processing all the bios, it unplugs the queue to submit the bios.
It also fixes kcopyd which has the same potential problem. All kcopyd requests
are submitted with BIO_RW_SYNC.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Acked-by: Jens Axboe <jens.axboe@oracle.com>
2008-04-25 04:10:47 +07:00
|
|
|
* New collapsed (a)synchronous interface.
|
|
|
|
*
|
|
|
|
* If the IO is asynchronous (i.e. it has notify.fn), you must either unplug
|
2014-07-01 00:26:30 +07:00
|
|
|
* the queue with blk_unplug() some time later or set REQ_SYNC in io_req->bi_rw.
|
|
|
|
* If you fail to do one of these, the IO will be submitted to the disk after
|
|
|
|
* q->unplug_delay, which defaults to 3ms in blk-settings.c.
|
2007-05-09 16:33:01 +07:00
|
|
|
*/
|
|
|
|
int dm_io(struct dm_io_request *io_req, unsigned num_regions,
|
2008-04-25 03:43:17 +07:00
|
|
|
struct dm_io_region *where, unsigned long *sync_error_bits)
|
2007-05-09 16:33:01 +07:00
|
|
|
{
|
|
|
|
int r;
|
|
|
|
struct dpages dp;
|
|
|
|
|
2011-08-02 18:32:01 +07:00
|
|
|
r = dp_init(io_req, &dp, (unsigned long)where->count << SECTOR_SHIFT);
|
2007-05-09 16:33:01 +07:00
|
|
|
if (r)
|
|
|
|
return r;
|
|
|
|
|
|
|
|
if (!io_req->notify.fn)
|
|
|
|
return sync_io(io_req->client, num_regions, where,
|
|
|
|
io_req->bi_rw, &dp, sync_error_bits);
|
|
|
|
|
|
|
|
return async_io(io_req->client, num_regions, where, io_req->bi_rw,
|
|
|
|
&dp, io_req->notify.fn, io_req->notify.context);
|
|
|
|
}
|
|
|
|
EXPORT_SYMBOL(dm_io);
|
2009-12-11 06:51:57 +07:00
|
|
|
|
|
|
|
int __init dm_io_init(void)
|
|
|
|
{
|
|
|
|
_dm_io_cache = KMEM_CACHE(io, 0);
|
|
|
|
if (!_dm_io_cache)
|
|
|
|
return -ENOMEM;
|
|
|
|
|
|
|
|
return 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
void dm_io_exit(void)
|
|
|
|
{
|
|
|
|
kmem_cache_destroy(_dm_io_cache);
|
|
|
|
_dm_io_cache = NULL;
|
|
|
|
}
|