<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jan 23, 2021, 08:28 Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sat, Jan 23, 2021 at 12:45:20AM +0200, Nir Soffer wrote:<br>
>  @failing<br>
> -def pwrite(h, buf, offset):<br>
> +def pwrite(h, buf, offset, flags):<br>
>      http = h['http']<br>
>  <br>
>      count = len(buf)<br>
> @@ -234,7 +253,7 @@ def pwrite(h, buf, offset):<br>
>  <br>
>  <br>
>  @failing<br>
> -def zero(h, count, offset, may_trim):<br>
> +def zero(h, count, offset, flags):<br>
<br>
Assume there's no way to pass the possible FUA flag through to oVirt here?<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">We have flush=true, which send nbd flush command after the write/zero.</div><div dir="auto"><br></div><div dir="auto">Maybe we could handle flush=true as fua when using nbd backend, I'm not sure how important is this.</div><div dir="auto"><br></div><div dir="auto">I think we miss converting fua to flush=true to have full api 2 support.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
libguestfs lets you edit virtual machines.  Supports shell scripting,<br>
bindings from many languages.  <a href="http://libguestfs.org" rel="noreferrer noreferrer" target="_blank">http://libguestfs.org</a><br>
<br>
</blockquote></div></div></div>