[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] virtio_blk: fix defaults for max_hw_sectors and max_seg
From: |
Mike Snitzer |
Subject: |
Re: [Qemu-devel] virtio_blk: fix defaults for max_hw_sectors and max_segment_size |
Date: |
Wed, 26 Nov 2014 16:51:08 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, Nov 26 2014 at 3:54pm -0500,
Jens Axboe <address@hidden> wrote:
> On 11/26/2014 01:51 PM, Mike Snitzer wrote:
> > On Wed, Nov 26 2014 at 2:48pm -0500,
> > Jens Axboe <address@hidden> wrote:
> >
> >>
> >> That code isn't even in mainline, as far as I can tell...
> >
> > Right, it is old RHEL6 code.
> >
> > But I've yet to determine what changed upstream that enables this to
> > "just work" with a really large max_sectors (I haven't been looking
> > either).
>
> Kind of hard for the rest of us to say, since it's triggering a BUG in
> code we don't have :-)
I never asked you or others to weigh in on old RHEL6 code. Once I
realized upstream worked even if max_sectors is _really_ high I said
"sorry for the noise".
But while you're here, I wouldn't mind getting your take on virtio-blk
setting max_hw_sectors to -1U.
As I said in my original reply to mst: it only makes sense to set a
really high initial upper bound like that in a driver if that driver
goes on to stack an underlying device's limit.