staging: lustre: include: fix one off errors in lustre_id.h
During inspection of another patch Dan Carpenter noticed some one off errors in lustre_id.h. Fix the condition test for OBIF_MAX_OID. Signed-off-by: James Simmons <jsimmons@infradead.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
3147b26840
commit
a5c954eff6
|
@ -659,7 +659,7 @@ static inline void ostid_set_id(struct ost_id *oi, __u64 oid)
|
||||||
oi->oi_fid.f_oid = oid;
|
oi->oi_fid.f_oid = oid;
|
||||||
oi->oi_fid.f_ver = oid >> 48;
|
oi->oi_fid.f_ver = oid >> 48;
|
||||||
} else {
|
} else {
|
||||||
if (oid > OBIF_MAX_OID) {
|
if (oid >= OBIF_MAX_OID) {
|
||||||
CERROR("Bad %llu to set " DOSTID "\n", oid, POSTID(oi));
|
CERROR("Bad %llu to set " DOSTID "\n", oid, POSTID(oi));
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
@ -684,7 +684,7 @@ static inline int fid_set_id(struct lu_fid *fid, __u64 oid)
|
||||||
fid->f_oid = oid;
|
fid->f_oid = oid;
|
||||||
fid->f_ver = oid >> 48;
|
fid->f_ver = oid >> 48;
|
||||||
} else {
|
} else {
|
||||||
if (oid > OBIF_MAX_OID) {
|
if (oid >= OBIF_MAX_OID) {
|
||||||
CERROR("Too large OID %#llx to set REG "DFID"\n",
|
CERROR("Too large OID %#llx to set REG "DFID"\n",
|
||||||
(unsigned long long)oid, PFID(fid));
|
(unsigned long long)oid, PFID(fid));
|
||||||
return -EBADF;
|
return -EBADF;
|
||||||
|
|
Loading…
Reference in New Issue