On Mon, Jun 8, 2015 at 10:43 PM, Josh Durgin <jdurgin@xxxxxxxxxx> wrote: > On 06/08/2015 11:19 AM, Alexandre DERUMIER wrote: >> >> Hi, >>>> >>>> looking at the latest version of QEMU, >> >> >> It's seem that it's was already this behaviour since the add of rbd_cache >> parsing in rbd.c by josh in 2012 >> >> >> http://git.qemu.org/?p=qemu.git;a=blobdiff;f=block/rbd.c;h=eebc3344620058322bb53ba8376af4a82388d277;hp=1280d66d3ca73e552642d7a60743a0e2ce05f664;hb=b11f38fcdf837c6ba1d4287b1c685eb3ae5351a8;hpb=166acf546f476d3594a1c1746dc265f1984c5c85 >> >> >> I'll do tests on my side tomorrow to be sure. > > > It seems like we should switch the order so ceph.conf is overridden by > qemu's cache settings. I don't remember a good reason to have it the > other way around. > > Josh > Erm, doesn`t this code *already* represent the right priorities? Cache=none setting should set a BDRV_O_NOCACHE which is effectively disabling cache in a mentioned snippet. _______________________________________________ ceph-users mailing list ceph-users@xxxxxxxxxxxxxx http://lists.ceph.com/listinfo.cgi/ceph-users-ceph.com
 |