Re: [PATCH v4] sd: Check for unaligned partial completion

2017-02-14 Thread Ram Pai
On Wed, Feb 15, 2017 at 03:48:52PM +0900, Damien Le Moal wrote: > Christoph, > > On 2/15/17 15:34, Christoph Hellwig wrote: > > this looks reasonable, but we should ask Guilherme and Ram to confirm > > it fixes their originally reported issue. I've added them to Cc. > > Thank you. > > Guilherme

Re: [PATCH v2] mpt3sas: Force request partial completion alignment

2017-01-26 Thread Ram Pai
xfer_cnt, sector_sz); > > + xfer_cnt = (xfer_cnt / sector_sz) * sector_sz; > > > > Not so keen on divisions. xfer_cnt = round_down(xfer_cnt, sector_sz), maybe? > > > > Martin, I might be completely wrong here (please correct me if this is > the