backport of IB/isert commit to '5.10.y' stable release

backport of IB/isert commit to '5.10.y' stable release

[Krishnamraju Eraparaju]

Hi Greg,

Could you please backport the below commit to "5.10.y" stable release.

Looks like this commit was already pulled to "5.10.y" stable tree weeks
ago.

This fix is required for Chelsio adapters. Without this fix the number
of connections supported by isert(over Chelsio adapter) will be significantly less.

--------------------------------------------------------
commit ID: dae7a75f1f19bffb579daf148f8d8addd2726772
IB/isert: add module param to set sg_tablesize for IO cmd
Author: Max Gurtovoy <mgurtovoy@nvidia.com>
Fixes: 317000b ("IB/isert: allocate RW ctxs according to max IO size")


Thanks,
Krishnamraju.

Re: backport of IB/isert commit to '5.10.y' stable release

[Sasha Levin]

On Thu, Feb 18, 2021 at 12:49:16PM +0530, Krishnamraju Eraparaju wrote:
>
>Hi Greg,
>
>Could you please backport the below commit to "5.10.y" stable release.
>
>Looks like this commit was already pulled to "5.10.y" stable tree weeks
>ago.
>
>This fix is required for Chelsio adapters. Without this fix the number
>of connections supported by isert(over Chelsio adapter) will be significantly less.

This reads like a performane improvement rather than a bug?

-- 
Thanks,
Sasha

Re: backport of IB/isert commit to '5.10.y' stable release

[Krishnamraju Eraparaju]

On Thursday, February 02/18/21, 2021 at 11:08:57 -0500, Sasha Levin wrote:
> On Thu, Feb 18, 2021 at 12:49:16PM +0530, Krishnamraju Eraparaju wrote:
> >
> >Hi Greg,
> >
> >Could you please backport the below commit to "5.10.y" stable release.
> >
> >Looks like this commit was already pulled to "5.10.y" stable tree weeks
> >ago.
> >
> >This fix is required for Chelsio adapters. Without this fix the number
> >of connections supported by isert(over Chelsio adapter) will be significantly less.
> 
> This reads like a performane improvement rather than a bug?
Hi Sasha Levin,

This is not a performance improvement.
Earlier, commit:317000b926b07c has introduced a side effect, where no.of
supported isert connections, over Chelsio adapters, had come down to
9(prior to this commit it was 250).

And this side effect got fixed with commit:dae7a75f1f19bf

Hence, please apply this commit(dae7a75f1f19bf) to '5.10.y' stable release.

Thanks,
Krishnamraju.
> 
> -- 
> Thanks,
> Sasha

Re: backport of IB/isert commit to '5.10.y' stable release

[Sasha Levin]

On Thu, Feb 18, 2021 at 11:11:56PM +0530, Krishnamraju Eraparaju wrote:
>On Thursday, February 02/18/21, 2021 at 11:08:57 -0500, Sasha Levin wrote:
>> On Thu, Feb 18, 2021 at 12:49:16PM +0530, Krishnamraju Eraparaju wrote:
>> >
>> >Hi Greg,
>> >
>> >Could you please backport the below commit to "5.10.y" stable release.
>> >
>> >Looks like this commit was already pulled to "5.10.y" stable tree weeks
>> >ago.
>> >
>> >This fix is required for Chelsio adapters. Without this fix the number
>> >of connections supported by isert(over Chelsio adapter) will be significantly less.
>>
>> This reads like a performane improvement rather than a bug?
>Hi Sasha Levin,
>
>This is not a performance improvement.
>Earlier, commit:317000b926b07c has introduced a side effect, where no.of
>supported isert connections, over Chelsio adapters, had come down to
>9(prior to this commit it was 250).
>
>And this side effect got fixed with commit:dae7a75f1f19bf
>
>Hence, please apply this commit(dae7a75f1f19bf) to '5.10.y' stable release.

Okay, I've queued it up, thanks!

-- 
Thanks,
Sasha