On Sat, Mar 13, 2021 at 10:30:11AM +0800, Weihang Li wrote: > From: Lang Cheng <chenglang@xxxxxxxxxx> > > When reloading driver, the head/tail pointer of CMDQ may be not at position > 0. Then during initialization of CMDQ, if head is reset first, the firmware > will start to handle CMDQ because the head is not equal to the tail. The > driver can reset tail first since the firmware will be triggerred only by > head. This bug is introduced by changing macros of head/tail register > without changing the order of initialization. > > Fixes: 292b3352bd5b ("RDMA/hns: Adjust fields and variables about CMDQ tail/head") > Signed-off-by: Lang Cheng <chenglang@xxxxxxxxxx> > Signed-off-by: Weihang Li <liweihang@xxxxxxxxxx> > --- > Changes since v1: > - Only retain the bugfix part for -rc branch. > - Link: https://patchwork.kernel.org/project/linux-rdma/patch/1615541933-35798-1-git-send-email-liweihang@xxxxxxxxxx/ > > drivers/infiniband/hw/hns/hns_roce_hw_v2.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) Applied to for-rc, thanks Jason