diff options
| author | Alasdair G Kergon | 2008-03-28 22:16:10 +0100 |
|---|---|---|
| committer | Linus Torvalds | 2008-03-28 22:45:23 +0100 |
| commit | 4cdc1d1fa5c5ac14dc21be19832f02fd0b83867e (patch) | |
| tree | a8c93c90dd088b18b9307bb50413478dbc48f5b5 /drivers/md/kcopyd.c | |
| parent | memstick: suppress uninitialized-var warning (diff) | |
| download | kernel-qcow2-linux-4cdc1d1fa5c5ac14dc21be19832f02fd0b83867e.tar.gz kernel-qcow2-linux-4cdc1d1fa5c5ac14dc21be19832f02fd0b83867e.tar.xz kernel-qcow2-linux-4cdc1d1fa5c5ac14dc21be19832f02fd0b83867e.zip | |
dm io: write error bits form long not int
write_err is an unsigned long used with set_bit() so should not be passed
around as unsigned int.
http://bugzilla.kernel.org/show_bug.cgi?id=10271
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Cc: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/md/kcopyd.c')
| -rw-r--r-- | drivers/md/kcopyd.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/md/kcopyd.c b/drivers/md/kcopyd.c index f3831f31223e..e76b52ade690 100644 --- a/drivers/md/kcopyd.c +++ b/drivers/md/kcopyd.c @@ -169,7 +169,7 @@ struct kcopyd_job { * Error state of the job. */ int read_err; - unsigned int write_err; + unsigned long write_err; /* * Either READ or WRITE @@ -293,7 +293,7 @@ static int run_complete_job(struct kcopyd_job *job) { void *context = job->context; int read_err = job->read_err; - unsigned int write_err = job->write_err; + unsigned long write_err = job->write_err; kcopyd_notify_fn fn = job->fn; struct kcopyd_client *kc = job->kc; @@ -396,7 +396,7 @@ static int process_jobs(struct list_head *jobs, int (*fn) (struct kcopyd_job *)) if (r < 0) { /* error this rogue job */ if (job->rw == WRITE) - job->write_err = (unsigned int) -1; + job->write_err = (unsigned long) -1L; else job->read_err = 1; push(&_complete_jobs, job); @@ -448,8 +448,8 @@ static void dispatch_job(struct kcopyd_job *job) } #define SUB_JOB_SIZE 128 -static void segment_complete(int read_err, - unsigned int write_err, void *context) +static void segment_complete(int read_err, unsigned long write_err, + void *context) { /* FIXME: tidy this function */ sector_t progress = 0; |