Commit 094f394d authored by Heinz Mauelshagen's avatar Heinz Mauelshagen Committed by Mike Snitzer

dm raid: address checkpatch.pl complaints

Use 'unsigned int' where appropriate.
Return negative errors.
Correct an indentation.
Signed-off-by: default avatarHeinz Mauelshagen <heinzm@redhat.com>
Signed-off-by: default avatarMike Snitzer <snitzer@redhat.com>
parent 9c72bad1
...@@ -256,10 +256,10 @@ static void rs_config_restore(struct raid_set *rs, struct rs_layout *l) ...@@ -256,10 +256,10 @@ static void rs_config_restore(struct raid_set *rs, struct rs_layout *l)
static struct raid_type { static struct raid_type {
const char *name; /* RAID algorithm. */ const char *name; /* RAID algorithm. */
const char *descr; /* Descriptor text for logging. */ const char *descr; /* Descriptor text for logging. */
const unsigned parity_devs; /* # of parity devices. */ const unsigned int parity_devs; /* # of parity devices. */
const unsigned minimal_devs; /* minimal # of devices in set. */ const unsigned int minimal_devs;/* minimal # of devices in set. */
const unsigned level; /* RAID level. */ const unsigned int level; /* RAID level. */
const unsigned algorithm; /* RAID algorithm. */ const unsigned int algorithm; /* RAID algorithm. */
} raid_types[] = { } raid_types[] = {
{"raid0", "raid0 (striping)", 0, 2, 0, 0 /* NONE */}, {"raid0", "raid0 (striping)", 0, 2, 0, 0 /* NONE */},
{"raid1", "raid1 (mirroring)", 0, 2, 1, 0 /* NONE */}, {"raid1", "raid1 (mirroring)", 0, 2, 1, 0 /* NONE */},
...@@ -665,9 +665,9 @@ static void rs_set_new(struct raid_set *rs) ...@@ -665,9 +665,9 @@ static void rs_set_new(struct raid_set *rs)
} }
static struct raid_set *raid_set_alloc(struct dm_target *ti, struct raid_type *raid_type, static struct raid_set *raid_set_alloc(struct dm_target *ti, struct raid_type *raid_type,
unsigned raid_devs) unsigned int raid_devs)
{ {
unsigned i; unsigned int i;
struct raid_set *rs; struct raid_set *rs;
if (raid_devs <= raid_type->parity_devs) { if (raid_devs <= raid_type->parity_devs) {
...@@ -920,9 +920,9 @@ static int validate_region_size(struct raid_set *rs, unsigned long region_size) ...@@ -920,9 +920,9 @@ static int validate_region_size(struct raid_set *rs, unsigned long region_size)
*/ */
static int validate_raid_redundancy(struct raid_set *rs) static int validate_raid_redundancy(struct raid_set *rs)
{ {
unsigned i, rebuild_cnt = 0; unsigned int i, rebuild_cnt = 0;
unsigned rebuilds_per_group = 0, copies; unsigned int rebuilds_per_group = 0, copies;
unsigned group_size, last_group_start; unsigned int group_size, last_group_start;
for (i = 0; i < rs->md.raid_disks; i++) for (i = 0; i < rs->md.raid_disks; i++)
if (!test_bit(In_sync, &rs->dev[i].rdev.flags) || if (!test_bit(In_sync, &rs->dev[i].rdev.flags) ||
...@@ -1030,12 +1030,12 @@ static int validate_raid_redundancy(struct raid_set *rs) ...@@ -1030,12 +1030,12 @@ static int validate_raid_redundancy(struct raid_set *rs)
* [raid10_format <near|far|offset>] Layout algorithm. (Default: near) * [raid10_format <near|far|offset>] Layout algorithm. (Default: near)
*/ */
static int parse_raid_params(struct raid_set *rs, struct dm_arg_set *as, static int parse_raid_params(struct raid_set *rs, struct dm_arg_set *as,
unsigned num_raid_params) unsigned int num_raid_params)
{ {
int value, raid10_format = ALGORITHM_RAID10_DEFAULT; int value, raid10_format = ALGORITHM_RAID10_DEFAULT;
unsigned raid10_copies = 2; unsigned int raid10_copies = 2;
unsigned i, write_mostly = 0; unsigned int i, write_mostly = 0;
unsigned region_size = 0; unsigned int region_size = 0;
sector_t max_io_len; sector_t max_io_len;
const char *arg, *key; const char *arg, *key;
struct raid_dev *rd; struct raid_dev *rd;
...@@ -1447,7 +1447,7 @@ static int rs_set_dev_and_array_sectors(struct raid_set *rs, bool use_mddev) ...@@ -1447,7 +1447,7 @@ static int rs_set_dev_and_array_sectors(struct raid_set *rs, bool use_mddev)
if (rs->raid10_copies < 2 || if (rs->raid10_copies < 2 ||
delta_disks < 0) { delta_disks < 0) {
rs->ti->error = "Bogus raid10 data copies or delta disks"; rs->ti->error = "Bogus raid10 data copies or delta disks";
return EINVAL; return -EINVAL;
} }
dev_sectors *= rs->raid10_copies; dev_sectors *= rs->raid10_copies;
...@@ -1474,7 +1474,7 @@ static int rs_set_dev_and_array_sectors(struct raid_set *rs, bool use_mddev) ...@@ -1474,7 +1474,7 @@ static int rs_set_dev_and_array_sectors(struct raid_set *rs, bool use_mddev)
return 0; return 0;
bad: bad:
rs->ti->error = "Target length not divisible by number of data devices"; rs->ti->error = "Target length not divisible by number of data devices";
return EINVAL; return -EINVAL;
} }
/* Setup recovery on @rs */ /* Setup recovery on @rs */
...@@ -2511,7 +2511,7 @@ static int rs_setup_takeover(struct raid_set *rs) ...@@ -2511,7 +2511,7 @@ static int rs_setup_takeover(struct raid_set *rs)
/* raid1 -> raid10_near layout */ /* raid1 -> raid10_near layout */
mddev->layout = raid10_format_to_md_layout(rs, ALGORITHM_RAID10_NEAR, mddev->layout = raid10_format_to_md_layout(rs, ALGORITHM_RAID10_NEAR,
rs->raid_disks); rs->raid_disks);
else else
return -EINVAL; return -EINVAL;
} }
...@@ -2758,12 +2758,12 @@ static void configure_discard_support(struct raid_set *rs) ...@@ -2758,12 +2758,12 @@ static void configure_discard_support(struct raid_set *rs)
* enforce recreation based on the passed in table parameters. * enforce recreation based on the passed in table parameters.
* *
*/ */
static int raid_ctr(struct dm_target *ti, unsigned argc, char **argv) static int raid_ctr(struct dm_target *ti, unsigned int argc, char **argv)
{ {
int r; int r;
bool resize; bool resize;
struct raid_type *rt; struct raid_type *rt;
unsigned num_raid_params, num_raid_devs; unsigned int num_raid_params, num_raid_devs;
sector_t calculated_dev_sectors; sector_t calculated_dev_sectors;
struct raid_set *rs = NULL; struct raid_set *rs = NULL;
const char *arg; const char *arg;
...@@ -3299,7 +3299,7 @@ static void raid_status(struct dm_target *ti, status_type_t type, ...@@ -3299,7 +3299,7 @@ static void raid_status(struct dm_target *ti, status_type_t type,
} }
} }
static int raid_message(struct dm_target *ti, unsigned argc, char **argv) static int raid_message(struct dm_target *ti, unsigned int argc, char **argv)
{ {
struct raid_set *rs = ti->private; struct raid_set *rs = ti->private;
struct mddev *mddev = &rs->md; struct mddev *mddev = &rs->md;
...@@ -3351,7 +3351,7 @@ static int raid_iterate_devices(struct dm_target *ti, ...@@ -3351,7 +3351,7 @@ static int raid_iterate_devices(struct dm_target *ti,
iterate_devices_callout_fn fn, void *data) iterate_devices_callout_fn fn, void *data)
{ {
struct raid_set *rs = ti->private; struct raid_set *rs = ti->private;
unsigned i; unsigned int i;
int r = 0; int r = 0;
for (i = 0; !r && i < rs->md.raid_disks; i++) for (i = 0; !r && i < rs->md.raid_disks; i++)
...@@ -3368,7 +3368,7 @@ static int raid_iterate_devices(struct dm_target *ti, ...@@ -3368,7 +3368,7 @@ static int raid_iterate_devices(struct dm_target *ti,
static void raid_io_hints(struct dm_target *ti, struct queue_limits *limits) static void raid_io_hints(struct dm_target *ti, struct queue_limits *limits)
{ {
struct raid_set *rs = ti->private; struct raid_set *rs = ti->private;
unsigned chunk_size = rs->md.chunk_sectors << 9; unsigned int chunk_size = rs->md.chunk_sectors << 9;
struct r5conf *conf = rs->md.private; struct r5conf *conf = rs->md.private;
blk_limits_io_min(limits, chunk_size); blk_limits_io_min(limits, chunk_size);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment