Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Message-ID: <d8a453b1-c77d-71b3-72cc-eaac51ef8cb8@hisilicon.com>
Date:   Tue, 5 Dec 2023 10:05:46 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.1.0
Subject: Re: [PATCH for-rc 0/6] Bugfixes and improvements for hns RoCE
Content-Language: en-US
To:     Leon Romanovsky <leon@kernel.org>
CC:     <jgg@ziepe.ca>, <linux-rdma@vger.kernel.org>,
        <linuxarm@huawei.com>, <linux-kernel@vger.kernel.org>
References: <20231129094434.134528-1-huangjunxian6@hisilicon.com>
 <20231204142447.GB5136@unreal>
From:   Junxian Huang <huangjunxian6@hisilicon.com>
In-Reply-To: <20231204142447.GB5136@unreal>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8bit
Precedence: bulk


On 2023/12/4 22:24, Leon Romanovsky wrote:
> On Wed, Nov 29, 2023 at 05:44:28PM +0800, Junxian Huang wrote:
>> Here are several bugfixes and improvements for hns RoCE.
>>
>> Chengchang Tang (4):
>>   RDMA/hns: Rename the interrupts
>>   RDMA/hns: Remove unnecessary checks for NULL in mtr_alloc_bufs()
>>   RDMA/hns: Fix memory leak in free_mr_init()
>>   RDMA/hns: Improve the readability of free mr uninit
> 
> 1. The series doesn't apply.
> ➜  kernel git:(wip/leon-for-next) ~/src/b4/b4.sh shazam -l -s https://lore.kernel.org/all/20231129094434.134528-1-huangjunxian6@hisilicon.com -P 1-5

Is this series going to be applied to -next?

> 2. Please drop patch #6 as you are deleting the code which you added in
> first patches without actual gain.

Is it better to drop it directly or merge it with the previous patch?

Thanks,
Junxian

> 
> Thanks
> 
>>
>> Junxian Huang (2):
>>   RDMA/hns: Response dmac to userspace
>>   RDMA/hns: Add a max length of gid table
>>
>>  drivers/infiniband/hw/hns/hns_roce_ah.c    |  7 ++
>>  drivers/infiniband/hw/hns/hns_roce_hw_v2.c | 87 +++++++++++++++-------
>>  drivers/infiniband/hw/hns/hns_roce_mr.c    |  2 +-
>>  include/uapi/rdma/hns-abi.h                |  5 ++
>>  4 files changed, 73 insertions(+), 28 deletions(-)
>>
>> --
>> 2.30.0
>>
>>
>