On 11/21/18 1:25 AM, Nilesh Javali wrote:
> Default packet size is 0x400.
> For jumbo packets set to 0x2400.
> 
> Signed-off-by: Nilesh Javali <nilesh.jav...@cavium.com>
> ---
>  drivers/scsi/qedi/qedi.h      |  1 -
>  drivers/scsi/qedi/qedi_main.c | 13 +++++++++----
>  2 files changed, 9 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/scsi/qedi/qedi.h b/drivers/scsi/qedi/qedi.h
> index 6fa02c5..a26bb506 100644
> --- a/drivers/scsi/qedi/qedi.h
> +++ b/drivers/scsi/qedi/qedi.h
> @@ -63,7 +63,6 @@
>  #define QEDI_LOCAL_PORT_INVALID      0xffff
>  #define TX_RX_RING           16
>  #define RX_RING                      (TX_RX_RING - 1)
> -#define LL2_SINGLE_BUF_SIZE  0x400
>  #define QEDI_PAGE_ALIGN(addr)        ALIGN(addr, QEDI_PAGE_SIZE)
>  #define QEDI_PAGE_MASK               (~((QEDI_PAGE_SIZE) - 1))
>  
> diff --git a/drivers/scsi/qedi/qedi_main.c b/drivers/scsi/qedi/qedi_main.c
> index 2621dee..8942f62 100644
> --- a/drivers/scsi/qedi/qedi_main.c
> +++ b/drivers/scsi/qedi/qedi_main.c
> @@ -44,6 +44,11 @@
>  MODULE_PARM_DESC(qedi_io_tracing,
>                " Enable logging of SCSI requests/completions into trace 
> buffer. (default off).");
>  
> +uint qedi_ll2_buf_size = 0x400;
> +module_param(qedi_ll2_buf_size, uint, 0644);
> +MODULE_PARM_DESC(qedi_ll2_buf_size,
> +              "parameter to set ping packet size, default - 0x400, Jumbo 
> packets - 0x2400.");
> +
>  const struct qed_iscsi_ops *qedi_ops;
>  static struct scsi_transport_template *qedi_scsi_transport;
>  static struct pci_driver qedi_pci_driver;
> @@ -228,7 +233,7 @@ static int __qedi_alloc_uio_rings(struct qedi_uio_dev 
> *udev)
>       }
>  
>       /* Allocating memory for Tx/Rx pkt buffer */
> -     udev->ll2_buf_size = TX_RX_RING * LL2_SINGLE_BUF_SIZE;
> +     udev->ll2_buf_size = TX_RX_RING * qedi_ll2_buf_size;
>       udev->ll2_buf_size = QEDI_PAGE_ALIGN(udev->ll2_buf_size);
>       udev->ll2_buf = (void *)__get_free_pages(GFP_KERNEL | __GFP_COMP |
>                                                __GFP_ZERO, 2);
> @@ -283,7 +288,7 @@ static int qedi_alloc_uio_rings(struct qedi_ctx *qedi)
>       qedi->udev = udev;
>  
>       udev->tx_pkt = udev->ll2_buf;
> -     udev->rx_pkt = udev->ll2_buf + LL2_SINGLE_BUF_SIZE;
> +     udev->rx_pkt = udev->ll2_buf + qedi_ll2_buf_size;
>       return 0;
>  
>   err_uctrl:
> @@ -752,8 +757,8 @@ static int qedi_ll2_process_skb(struct qedi_ctx *qedi, 
> struct sk_buff *skb,
>  
>       udev = qedi->udev;
>       uctrl = udev->uctrl;
> -     pkt = udev->rx_pkt + (uctrl->hw_rx_prod * LL2_SINGLE_BUF_SIZE);
> -     len = min_t(u32, skb->len, (u32)LL2_SINGLE_BUF_SIZE);
> +     pkt = udev->rx_pkt + (uctrl->hw_rx_prod * qedi_ll2_buf_size);
> +     len = min_t(u32, skb->len, (u32)qedi_ll2_buf_size);
>       memcpy(pkt, skb->data, len);
>  
>       memset(&rxbd, 0, sizeof(rxbd));
> 


Reviewed-by: Lee Duncan <ldun...@suse.com>

Reply via email to