Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-10 Thread Leon Romanovsky
On Tue, Jul 10, 2018 at 03:08:38PM +, Bart Van Assche wrote: > On Tue, 2018-07-10 at 08:39 +0300, Leon Romanovsky wrote: > > On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > > > An array of pointers to SRPT contexts in ib_device is over 30KiB even > > > in default case, in

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-10 Thread Leon Romanovsky
On Tue, Jul 10, 2018 at 03:08:38PM +, Bart Van Assche wrote: > On Tue, 2018-07-10 at 08:39 +0300, Leon Romanovsky wrote: > > On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > > > An array of pointers to SRPT contexts in ib_device is over 30KiB even > > > in default case, in

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-10 Thread Bart Van Assche
On Tue, 2018-07-10 at 08:39 +0300, Leon Romanovsky wrote: > On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > > An array of pointers to SRPT contexts in ib_device is over 30KiB even > > in default case, in which an amount of contexts is 4095. The patch > > is intended to weed out

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-10 Thread Bart Van Assche
On Tue, 2018-07-10 at 08:39 +0300, Leon Romanovsky wrote: > On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > > An array of pointers to SRPT contexts in ib_device is over 30KiB even > > in default case, in which an amount of contexts is 4095. The patch > > is intended to weed out

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Leon Romanovsky
On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. kvmalloc* doesn't

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Leon Romanovsky
On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. kvmalloc* doesn't

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Jason Gunthorpe
On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. > > Signed-off-by:

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Jason Gunthorpe
On Mon, Jul 09, 2018 at 04:51:08PM +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. > > Signed-off-by:

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Bart Van Assche
On Mon, 2018-07-09 at 16:51 +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. Reviewed-by: Bart Van

Re: [PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Bart Van Assche
On Mon, 2018-07-09 at 16:51 +0300, Jan Dakinevich wrote: > An array of pointers to SRPT contexts in ib_device is over 30KiB even > in default case, in which an amount of contexts is 4095. The patch > is intended to weed out large contigous allocation for non-DMA memory. Reviewed-by: Bart Van

[PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Jan Dakinevich
An array of pointers to SRPT contexts in ib_device is over 30KiB even in default case, in which an amount of contexts is 4095. The patch is intended to weed out large contigous allocation for non-DMA memory. Signed-off-by: Jan Dakinevich --- drivers/infiniband/ulp/srpt/ib_srpt.c | 6 +++--- 1

[PATCH] ib_srpt: use kvmalloc to allocate ring pointers

2018-07-09 Thread Jan Dakinevich
An array of pointers to SRPT contexts in ib_device is over 30KiB even in default case, in which an amount of contexts is 4095. The patch is intended to weed out large contigous allocation for non-DMA memory. Signed-off-by: Jan Dakinevich --- drivers/infiniband/ulp/srpt/ib_srpt.c | 6 +++--- 1