Re: [PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-05 Thread Liu Bo
On Wed, Oct 04, 2017 at 08:23:05PM +0200, David Sterba wrote: > On Wed, Oct 04, 2017 at 11:13:51AM -0600, Liu Bo wrote: > > On Wed, Oct 04, 2017 at 04:22:28PM +0200, David Sterba wrote: > > > On Tue, Oct 03, 2017 at 07:31:10PM +0200, Goffredo Baroncelli wrote: > > > > From: Goffredo Baroncelli > >

Re: [PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-04 Thread Goffredo Baroncelli
On 10/04/2017 07:13 PM, Liu Bo wrote: > On Wed, Oct 04, 2017 at 04:22:28PM +0200, David Sterba wrote: >> On Tue, Oct 03, 2017 at 07:31:10PM +0200, Goffredo Baroncelli wrote: >>> From: Goffredo Baroncelli >>> >>> Jean-Denis Girard noticed commit c821e7f3 "pass bytes to >>> btrfs_bio_alloc" (https:/

Re: [PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-04 Thread David Sterba
On Wed, Oct 04, 2017 at 11:13:51AM -0600, Liu Bo wrote: > On Wed, Oct 04, 2017 at 04:22:28PM +0200, David Sterba wrote: > > On Tue, Oct 03, 2017 at 07:31:10PM +0200, Goffredo Baroncelli wrote: > > > From: Goffredo Baroncelli > > > > > > Jean-Denis Girard noticed commit c821e7f3 "pass bytes to > >

Re: [PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-04 Thread Liu Bo
On Wed, Oct 04, 2017 at 04:22:28PM +0200, David Sterba wrote: > On Tue, Oct 03, 2017 at 07:31:10PM +0200, Goffredo Baroncelli wrote: > > From: Goffredo Baroncelli > > > > Jean-Denis Girard noticed commit c821e7f3 "pass bytes to > > btrfs_bio_alloc" (https://patchwork.kernel.org/patch/9763081/) in

Re: [PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-04 Thread David Sterba
On Tue, Oct 03, 2017 at 07:31:10PM +0200, Goffredo Baroncelli wrote: > From: Goffredo Baroncelli > > Jean-Denis Girard noticed commit c821e7f3 "pass bytes to > btrfs_bio_alloc" (https://patchwork.kernel.org/patch/9763081/) introduces a > regression on 32 bit machines. > When CONFIG_LBDAF is _not_

[PATCH] btrfs: avoid overflow when sector_t is 32 bit

2017-10-03 Thread Goffredo Baroncelli
From: Goffredo Baroncelli Jean-Denis Girard noticed commit c821e7f3 "pass bytes to btrfs_bio_alloc" (https://patchwork.kernel.org/patch/9763081/) introduces a regression on 32 bit machines. When CONFIG_LBDAF is _not_ defined (CONFIG_LBDAF == Support for large (2TB+) block devices and files) secto