On Tue, 2018-02-13 at 17:54 +0900, Sergey Senozhatsky wrote:
> On (02/12/18 11:05), Bart Van Assche wrote:
> [..]
> > diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
> > index ac4740cf74be..cf17626604c2 100644
> > --- a/include/linux/blkdev.h
> > +++ b/include/linux/blkdev.h
> > @@ -1026,14 +1026,25 @@ static inline int blk_rq_cur_bytes(const struct request *rq)
> >
> > extern unsigned int blk_rq_err_bytes(const struct request *rq);
> >
> > +/*
> > + * Variables of type sector_t represent an offset or size that is a multiple of
> > + * 2**9 bytes. Hence these two constants.
> > + */
> > +#ifndef SECTOR_SHIFT
> > +enum { SECTOR_SHIFT = 9 };
> > +#endif
> > +#ifndef SECTOR_SIZE
> > +enum { SECTOR_SIZE = 512 };
> > +#endif
>
> Shouldn't SECTOR_SIZE depend on SECTOR_SHIFT?
>
> 1 << SECTOR_SHIFT
Not sure if that change will really make a difference. Anyway, I will make that change.
Bart.
