Re: [PATCH v3 4/5] libceph: add sparse read support to OSD client

[Xiubo Li <xiubli@xxxxxxxxxx>]

On 3/22/22 9:58 AM, Xiubo Li wrote:
> On 3/18/22 9:50 PM, Jeff Layton wrote:
> >
...
> > +
> > +#ifdef __BIG_ENDIAN
> > +static inline void convert_extent_map(struct ceph_sparse_read *sr)
> > +{
> > +    int i;
> > +
> > +    for (i = 0; i < sr->sr_count; i++) {
> > +        struct ceph_sparse_extent *ext = sr->sr_extent[i];
> > +
> > +        ext->off = le64_to_cpu((__force __le32)ext->off);
> > +        ext->len = le64_to_cpu((__force __le32)ext->len);
>
> Why '__le32' ? Shouldn't it be '__le64' ?
Please ignore this, I just received your new patch series after this and 
found you have fixed it.

I will check the new series today.

Thanks.

-- XIubo