<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 14, 2013 at 10:40 PM, MORITA Kazutaka <span dir="ltr"><<a href="mailto:morita.kazutaka@gmail.com" target="_blank">morita.kazutaka@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">At Wed, 14 Aug 2013 17:22:53 +0800,<br>
<div class="im">Liu Yuan wrote:<br>
><br>
> On Tue, Aug 13, 2013 at 01:37:36PM +0900, MORITA Kazutaka wrote:<br>
> > From: MORITA Kazutaka <<a href="mailto:morita.kazutaka@lab.ntt.co.jp">morita.kazutaka@lab.ntt.co.jp</a>><br>
> ><br>
> > v2:<br>
> >  - rebased onto the devel branch<br>
> >  - added necessary flush after _vdi_write()<br>
> >  - call _filter_qemu_io() in _qemu_io()<br>
> ><br>
> > This series enables us to test object cache by running the following<br>
> > commands.<br>
> ><br>
> >  # SHEEP_EXTRA_OPTIONS="-w size=100" ./check<br>
> >  # SHEEP_EXTRA_OPTIONS="-w size=100" ./check -writethrough<br>
> >  # SHEEP_EXTRA_OPTIONS="-w size=100" ./check -direct<br>
><br>
> our vdi read/write causes many false alarms because<br>
>  - write -w doesn't flush the cache<br>
>  - read doesn't release the cache<br>
><br>
> Maybe we should use qemu-io which release the cache and support vected io,<br>
> to replace all the 'vdi write/read'.<br>
<br>
</div>qemu-io cannot be alternative because it works as if VM starts before<br>
doing I/Os and shutdowns after that.  Running qemu-io multiple times<br>
doesn't emulate multiple VM I/Os correctly.<br>
<br>
I guess the only way to go is adding "collie vdi cache flush" and<br>
"collie vdi cache delete" appropriately to our test cases.</blockquote><div><br></div><div>Manual add flush & delete is very hard, especially for new comers to write test cases. I think create helpers,</div>
<div>_vdi_write_flush, _vdi_{wrtie, read}_release(flush & delete cache) might help, but I am not sure.</div><div><br></div><div>Thanks</div><div>Yuan </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">
><br>
> 'SHEEP_EXTRA_OPTIONS="-w size=100" ./check' is cumbersome.<br>
><br>
> ./check -writeback is better. I think we only need to test writeback. direct<br>
> and writethrough only need one test script to test functionality because<br>
> writethrough is pretty much the same as writeback, and direct is same as no<br>
> cache.<br>
<br>
</div>Agreed.<br>
<br>
However, I think adding SHEEP_EXTRA_OPTIONS is still useful to test<br>
the journaling feature.<br>
<br>
Thanks,<br>
<br>
Kazutaka<br>
</blockquote></div><br></div></div>