Re: [PATCH v2 1/4] ntb: Fix leakage of ntb_device::msix_entries[] array

2014-02-28 Thread Alexander Gordeev
On Fri, Feb 21, 2014 at 04:33:25PM -0700, Jon Mason wrote:
> On Fri, Feb 21, 2014 at 04:49:29PM +0100, Alexander Gordeev wrote:
> > Signed-off-by: Alexander Gordeev 
> > Cc: Jon Mason 
> > Cc: linux-...@vger.kernel.org
> 
> Good catch.  Applied.

Hi Jon,

If the other three patches are okay?

-- 
Regards,
Alexander Gordeev
agord...@redhat.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v2 1/4] ntb: Fix leakage of ntb_device::msix_entries[] array

2014-02-28 Thread Alexander Gordeev
On Fri, Feb 21, 2014 at 04:33:25PM -0700, Jon Mason wrote:
 On Fri, Feb 21, 2014 at 04:49:29PM +0100, Alexander Gordeev wrote:
  Signed-off-by: Alexander Gordeev agord...@redhat.com
  Cc: Jon Mason jon.ma...@intel.com
  Cc: linux-...@vger.kernel.org
 
 Good catch.  Applied.

Hi Jon,

If the other three patches are okay?

-- 
Regards,
Alexander Gordeev
agord...@redhat.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v2 1/4] ntb: Fix leakage of ntb_device::msix_entries[] array

2014-02-21 Thread Jon Mason
On Fri, Feb 21, 2014 at 04:49:29PM +0100, Alexander Gordeev wrote:
> Signed-off-by: Alexander Gordeev 
> Cc: Jon Mason 
> Cc: linux-...@vger.kernel.org

Good catch.  Applied.

> ---
>  drivers/ntb/ntb_hw.c |1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ntb/ntb_hw.c b/drivers/ntb/ntb_hw.c
> index 170e8e6..487169dd 100644
> --- a/drivers/ntb/ntb_hw.c
> +++ b/drivers/ntb/ntb_hw.c
> @@ -1281,6 +1281,7 @@ static void ntb_free_interrupts(struct ntb_device *ndev)
>   free_irq(msix->vector, >db_cb[i]);
>   }
>   pci_disable_msix(pdev);
> + kfree(ndev->msix_entries);
>   } else {
>   free_irq(pdev->irq, ndev);
>  
> -- 
> 1.7.7.6
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH v2 1/4] ntb: Fix leakage of ntb_device::msix_entries[] array

2014-02-21 Thread Jon Mason
On Fri, Feb 21, 2014 at 04:49:29PM +0100, Alexander Gordeev wrote:
 Signed-off-by: Alexander Gordeev agord...@redhat.com
 Cc: Jon Mason jon.ma...@intel.com
 Cc: linux-...@vger.kernel.org

Good catch.  Applied.

 ---
  drivers/ntb/ntb_hw.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)
 
 diff --git a/drivers/ntb/ntb_hw.c b/drivers/ntb/ntb_hw.c
 index 170e8e6..487169dd 100644
 --- a/drivers/ntb/ntb_hw.c
 +++ b/drivers/ntb/ntb_hw.c
 @@ -1281,6 +1281,7 @@ static void ntb_free_interrupts(struct ntb_device *ndev)
   free_irq(msix-vector, ndev-db_cb[i]);
   }
   pci_disable_msix(pdev);
 + kfree(ndev-msix_entries);
   } else {
   free_irq(pdev-irq, ndev);
  
 -- 
 1.7.7.6
 
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/