diff options
author | Alex Elder <elder@inktank.com> | 2013-02-15 23:10:17 -0500 |
---|---|---|
committer | Sage Weil <sage@inktank.com> | 2013-05-02 00:14:23 -0400 |
commit | 07c09b725543ff2958c11522d583f90f7fdba735 (patch) | |
tree | e59ccdb0798fb56c4d2d34bbe1952b1de9cfdf6c /include/linux/ceph/messenger.h | |
parent | 3ff5f385b1449a07372d51fb89ca94dbfb6a3be2 (diff) |
libceph: make ceph_msg->bio_seg be unsigned
The bio_seg field is used by the ceph messenger in iterating through
a bio. It should never have a negative value, so make it an
unsigned. (I contemplated making it unsigned short to match the
struct bio definition, but it offered no benefit.)
Change variables used to hold bio_seg values to all be unsigned as
well. Change two variable names in init_bio_iter() to match the
convention used everywhere else.
Signed-off-by: Alex Elder <elder@inktank.com>
Reviewed-by: Josh Durgin <josh.durgin@inktank.com>
Diffstat (limited to 'include/linux/ceph/messenger.h')
-rw-r--r-- | include/linux/ceph/messenger.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/ceph/messenger.h b/include/linux/ceph/messenger.h index 60903e0f665c..8297288a66e0 100644 --- a/include/linux/ceph/messenger.h +++ b/include/linux/ceph/messenger.h | |||
@@ -86,7 +86,7 @@ struct ceph_msg { | |||
86 | #ifdef CONFIG_BLOCK | 86 | #ifdef CONFIG_BLOCK |
87 | struct bio *bio; /* instead of pages/pagelist */ | 87 | struct bio *bio; /* instead of pages/pagelist */ |
88 | struct bio *bio_iter; /* bio iterator */ | 88 | struct bio *bio_iter; /* bio iterator */ |
89 | int bio_seg; /* current bio segment */ | 89 | unsigned int bio_seg; /* current bio segment */ |
90 | #endif /* CONFIG_BLOCK */ | 90 | #endif /* CONFIG_BLOCK */ |
91 | struct ceph_pagelist *trail; /* the trailing part of the data */ | 91 | struct ceph_pagelist *trail; /* the trailing part of the data */ |
92 | bool front_is_vmalloc; | 92 | bool front_is_vmalloc; |