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

Re: [Xen-devel] [PATCH] vscsiif: minor cleanup


  • To: Jan Beulich <JBeulich@xxxxxxxx>, xen-devel <xen-devel@xxxxxxxxxxxxx>
  • From: Keir Fraser <keir@xxxxxxx>
  • Date: Fri, 16 Nov 2012 15:03:56 +0000
  • Delivery-date: Fri, 16 Nov 2012 15:04:20 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xen.org>
  • Thread-index: Ac3EC5lYm1/+XIvgb0ub9RtV69X8XA==
  • Thread-topic: [Xen-devel] [PATCH] vscsiif: minor cleanup

On 16/11/2012 13:14, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:

> Remove a definition not belonging into the interface, and correct a
> few typos in comments.
> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>

Acked-by: Keir Fraser <keir@xxxxxxx>

> --- a/xen/include/public/io/vscsiif.h
> +++ b/xen/include/public/io/vscsiif.h
> @@ -30,25 +30,22 @@
>  #include "ring.h"
>  #include "../grant_table.h"
>  
> -/* command between backend and frontend */
> +/* commands between backend and frontend */
>  #define VSCSIIF_ACT_SCSI_CDB         1    /* SCSI CDB command */
>  #define VSCSIIF_ACT_SCSI_ABORT       2    /* SCSI Device(Lun) Abort*/
>  #define VSCSIIF_ACT_SCSI_RESET       3    /* SCSI Device(Lun) Reset*/
>  
> -
> -#define VSCSIIF_BACK_MAX_PENDING_REQS    128
> -
>  /*
>   * Maximum scatter/gather segments per request.
>   *
> - * Considering balance between allocating al least 16 "vscsiif_request"
> - * structures on one page (4096bytes) and number of scatter gather
> - * needed, we decided to use 26 as a magic number.
> + * Considering balance between allocating at least 16 "vscsiif_request"
> + * structures on one page (4096 bytes) and the number of scatter/gather
> + * elements needed, we decided to use 26 as a magic number.
>   */
>  #define VSCSIIF_SG_TABLESIZE             26
>  
>  /*
> - * base on linux kernel 2.6.18
> + * based on Linux kernel 2.6.18
>   */
>  #define VSCSIIF_MAX_COMMAND_SIZE         16
>  #define VSCSIIF_SENSE_BUFFERSIZE         96
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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