[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 5/9] qcow2: Assign the L2 cache relatively to
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v7 5/9] qcow2: Assign the L2 cache relatively to the image size |
Date: |
Mon, 13 Aug 2018 13:33:31 +0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
Am 11.08.2018 um 21:19 hat Leonid Bloch geschrieben:
> > > @item refcount-cache-size
> > > The maximum size of the refcount block cache in bytes
> > > diff --git a/tests/qemu-iotests/137 b/tests/qemu-iotests/137
> > > index 87965625d8..e3fb078588 100755
> > > --- a/tests/qemu-iotests/137
> > > +++ b/tests/qemu-iotests/137
> > > @@ -109,7 +109,6 @@ $QEMU_IO \
> > > -c "reopen -o
> > > cache-size=1M,l2-cache-size=64k,refcount-cache-size=64k" \
> > > -c "reopen -o cache-size=1M,l2-cache-size=2M" \
> > > -c "reopen -o cache-size=1M,refcount-cache-size=2M" \
> > > - -c "reopen -o l2-cache-size=256T" \
> >
> > The "L2 cache size too big" error can still be tested, but you will need
> > to create an image large enough to allow such a big cache.
> >
> > $ qemu-img create -f qcow2 -o cluster_size=256k hd.qcow2 32P
> > $ $QEMU -drive file=hd.qcow2,l2-cache-entry-size=512,l2-cache-size=1T
>
> * 32P qcow2 will take 33M - is it OK to create it just for a test?
> * Is it worth to create a special test scenario, with a separate image
> creation, just for that case?
We're creating larger images than that during tests, so I think this is
fine. You don't have to create a new separate test file or anything,
just increase the size of the used test image from 64M to 32P or
whatever is necessary.
Kevin
- [Qemu-devel] [PATCH v7 2/9] qcow2: Cosmetic changes, (continued)
[Qemu-devel] [PATCH v7 6/9] qcow2: Increase the default upper limit on the L2 cache size, Leonid Bloch, 2018/08/10
Re: [Qemu-devel] [PATCH v7 6/9] qcow2: Increase the default upper limit on the L2 cache size, Kevin Wolf, 2018/08/13
Re: [Qemu-devel] [PATCH v7 6/9] qcow2: Increase the default upper limit on the L2 cache size, Max Reitz, 2018/08/13
Re: [Qemu-devel] [PATCH v7 6/9] qcow2: Increase the default upper limit on the L2 cache size, Kevin Wolf, 2018/08/13