RE: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-04-01 Thread Praveen Kannoju
chanamatlu ; Jeffery Yoder Subject: Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update On Thu, Mar 25, 2021 at 11:39:28AM -0300, Jason Gunthorpe wrote: > On Tue, Mar 23, 2021 at 09:27:38PM -0700, Aruna Ramakrishna wrote: > > > > Do you have benchmarks

Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-31 Thread Jason Gunthorpe
On Thu, Mar 25, 2021 at 11:39:28AM -0300, Jason Gunthorpe wrote: > On Tue, Mar 23, 2021 at 09:27:38PM -0700, Aruna Ramakrishna wrote: > > > > Do you have benchmarks that show the performance of the high order > > > pages is not relavent? I'm a bit surprised to hear that > > > > > > > I guess my

Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-25 Thread Jason Gunthorpe
On Tue, Mar 23, 2021 at 09:27:38PM -0700, Aruna Ramakrishna wrote: > > Do you have benchmarks that show the performance of the high order > > pages is not relavent? I'm a bit surprised to hear that > > > > I guess my point was more to the effect that an order-8 alloc will > fail more often than

Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-23 Thread Aruna Ramakrishna
> On Mar 23, 2021, at 4:13 PM, Jason Gunthorpe wrote: > > On Tue, Mar 23, 2021 at 12:41:51PM -0700, Aruna Ramakrishna wrote: >> There is a far greater possibility of an order-8 allocation failing, >> esp. with the addition of __GFP_NORETRY , and the code would have to >> fall back to a

Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-23 Thread Jason Gunthorpe
On Tue, Mar 23, 2021 at 12:41:51PM -0700, Aruna Ramakrishna wrote: >There is a far greater possibility of an order-8 allocation failing, >esp. with the addition of __GFP_NORETRY , and the code would have to >fall back to a lower order allocation more often than not (esp. on a >long

Re: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-23 Thread Jason Gunthorpe
On Tue, Mar 16, 2021 at 01:09:01PM +, Praveen Kumar Kannoju wrote: > To update xlt (during mlx5_ib_reg_user_mr()), the driver can request up to > 1 MB (order-8) memory, depending on the size of the MR. This costly > allocation can sometimes take very long to return (a few seconds), >

RE: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-23 Thread Praveen Kannoju
; j...@ziepe.ca; linux-r...@vger.kernel.org; linux-kernel@vger.kernel.org Cc: Rajesh Sivaramasubramaniom ; Rama Nichanamatlu ; Aruna Ramakrishna ; Jeffery Yoder ; Praveen Kannoju Subject: [PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update To update xlt (during

[PATCH v2] IB/mlx5: Reduce max order of memory allocated for xlt update

2021-03-16 Thread Praveen Kumar Kannoju
To update xlt (during mlx5_ib_reg_user_mr()), the driver can request up to 1 MB (order-8) memory, depending on the size of the MR. This costly allocation can sometimes take very long to return (a few seconds), especially if the system is fragmented and does not have any free chunks for orders >=