Re: [PATCHv2]brd: set max_discard_sectors properly

On Mon, Apr 18, 2016 at 10:34 AM, Jinpu Wang wrote: > On Sat, Apr 16, 2016 at 3:50 AM, Christoph Hellwig wrote: >>> - blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX); >>> + blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX >> 9); >> >> Shouldn't we fix the issue by capping to

Re: [PATCHv2]brd: set max_discard_sectors properly

On Sat, Apr 16, 2016 at 3:50 AM, Christoph Hellwig wrote: >> - blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX); >> + blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX >> 9); > > Shouldn't we fix the issue by capping to UINT_MAX >> 9 inside > blk_queue_max_discard_sectors? Tha

Re: [PATCHv2]brd: set max_discard_sectors properly

> - blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX); > + blk_queue_max_discard_sectors(brd->brd_queue, UINT_MAX >> 9); Shouldn't we fix the issue by capping to UINT_MAX >> 9 inside blk_queue_max_discard_sectors? That way we'll prevent against having issues like this in any other d