On 01/30/2015 04:05 PM, Jan Beulich wrote:
On 30.01.15 at 15:54, <jgr...@suse.com> wrote:
On 01/30/2015 03:46 PM, Jan Beulich wrote:
On 30.01.15 at 14:52, <"jgr...@suse.com".non-mime.internet> wrote:
@@ -231,8 +242,23 @@ static int scsifront_cmd_done(struct vsc
return more_to_do;
}
+void scsifront_finish_all(struct vscsifrnt_info *info)
+{
+ unsigned i;
+ struct vscsiif_response resp;
+ scsifront_ring_drain(info);
Shouldn't you at least issue some kind of warning when this returns
non-zero?
If a warning should be issued, then this should be done after the
following loop in case of at least one request terminated there.
I'm really not sure, whether a warning is required here. If you like,
I can add one.
I'm not sure, I'm merely asking because I saw the function return
value being ignored here.
I think it can be 0 only. We are handling resume, so the ring which is
being drained will no longer be filled by the backend.
Juergen
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel