From: Cheng Xu <chengyou@linux.alibaba.com>
To: Leon Romanovsky <leon@kernel.org>
Cc: jgg@ziepe.ca, linux-rdma@vger.kernel.org, KaiShen@linux.alibaba.com
Subject: Re: [PATCH for-next 0/3] RDMA/erdma: Add atomic operations support
Date: Mon, 7 Nov 2022 10:02:42 +0800 [thread overview]
Message-ID: <cf92c599-5078-1464-eed0-8ba72afb4824@linux.alibaba.com> (raw)
In-Reply-To: <Y2e1pr36v3tm0l5A@unreal>
On 11/6/22 9:24 PM, Leon Romanovsky wrote:
> On Thu, Oct 27, 2022 at 05:57:38PM +0800, Cheng Xu wrote:
>> Hi,
>>
>> This series introcuces atomic operations support for erdma driver:
>> - #1 extends access rights field in FRMR and REG MR commands to support
>> IB_ACCESS_REMOTE_ATOMIC.
>> - #2 gets atomic capacity from hardware, and reports the cap to core.
>> - #3 implements the IO-path of atomic WR processing.
>>
>> Thanks,
>> Cheng Xu
>>
>> Cheng Xu (3):
>> RDMA/erdma: Extend access right field of FRMR and REG MR to support
>> atomic
>> RDMA/erdma: Report atomic capacity when hardware supports atomic
>> feature
>> RDMA/erdma: Implement atomic operations support
>
> It doesn't pass my static analyzer checks.
>
> ➜ kernel git:(wip/leon-for-next) mkt ci
> 3f69c46621e3 (HEAD -> build) RDMA/erdma: Implement atomic operations support
> drivers/infiniband/hw/erdma/erdma_qp.c:365:26: warning: incorrect type in assignment (different base types)
> drivers/infiniband/hw/erdma/erdma_qp.c:365:26: expected restricted __le32 [usertype] key
> drivers/infiniband/hw/erdma/erdma_qp.c:365:26: got unsigned int [usertype] rkey
>
I'm sorry for forgetting to run sparse for the code.
I will send v2.
Thanks,
Cheng Xu
prev parent reply other threads:[~2022-11-07 2:02 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-27 9:57 [PATCH for-next 0/3] RDMA/erdma: Add atomic operations support Cheng Xu
2022-10-27 9:57 ` [PATCH for-next 1/3] RDMA/erdma: Extend access right field of FRMR and REG MR to support atomic Cheng Xu
2022-10-27 9:57 ` [PATCH for-next 2/3] RDMA/erdma: Report atomic capacity when hardware supports atomic feature Cheng Xu
2022-10-27 9:57 ` [PATCH for-next 3/3] RDMA/erdma: Implement atomic operations support Cheng Xu
2022-11-06 13:24 ` [PATCH for-next 0/3] RDMA/erdma: Add " Leon Romanovsky
2022-11-07 2:02 ` Cheng Xu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cf92c599-5078-1464-eed0-8ba72afb4824@linux.alibaba.com \
--to=chengyou@linux.alibaba.com \
--cc=KaiShen@linux.alibaba.com \
--cc=jgg@ziepe.ca \
--cc=leon@kernel.org \
--cc=linux-rdma@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox