[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [win-pv-devel] [PATCH 0/4] Use StorPort queues



> -----Original Message-----
> From: win-pv-devel [mailto:win-pv-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On
> Behalf Of owen.smith@xxxxxxxxxx
> Sent: 09 April 2018 09:48
> To: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: Owen Smith <owen.smith@xxxxxxxxxx>
> Subject: [win-pv-devel] [PATCH 0/4] Use StorPort queues
> 
> From: Owen Smith <owen.smith@xxxxxxxxxx>
> 
> Maintaining a queue of SRBs inside xenvbd is not ideal, as its possible
> (though rare) for SRBs to time out while in the fresh queue.
> These patches moves the blkif_request preparation and an attempt to
> submit all prepared requests code to the StartIo call. Requests that cannot
> be prepared (e.g. due to limited resources) are returned to StorPort with
> SRB_STATUS_BUSY, which will reinsert the SRB into StorPorts queues.
> This reorganization reduces the amount of work required in the DPC routine.
> 
> An exploratory WHQL test passed using Windows10 (x64) 1709
> IOMeter results (backed by local spinning disk) showed some small
> improvments
> across all scenarios
> 
> Owen Smith (4):
>   Rationalize WasQueued semantics
>   Pass SrbExt to RingPrepare* calls
>   Remove FreshSrbs queue
>   Abort any partially completed SRBs
>

All applied. Thanks,

  Paul
 
>  src/xenvbd/ring.c   | 144 
> ++++++++++++++++------------------------------------
>  src/xenvbd/ring.h   |   2 +-
>  src/xenvbd/target.c |  28 ++++------
>  3 files changed, 55 insertions(+), 119 deletions(-)
> 
> --
> 2.8.3
> 
> 
> _______________________________________________
> win-pv-devel mailing list
> win-pv-devel@xxxxxxxxxxxxxxxxxxxx
> https://lists.xenproject.org/mailman/listinfo/win-pv-devel
_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.