From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751362AbdI1Eez convert rfc822-to-8bit (ORCPT ); Thu, 28 Sep 2017 00:34:55 -0400 Received: from smtprz15.163.net ([106.3.154.248]:62415 "HELO smtp.tom.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with SMTP id S1750731AbdI1Eey (ORCPT ); Thu, 28 Sep 2017 00:34:54 -0400 Authentication-Results: my-fxspam01 smtp.user=xavier.huwei@tom.com; auth=pass (LOGIN) Subject: Re: [PATCH for-next 05/20] RDMA/hns: Add command queue support for hip08 RoCE driver To: Doug Ledford Cc: Leon Romanovsky , "Wei Hu (Xavier)" , linux-rdma@vger.kernel.org, lijun_nudt@163.com, oulijun@huawei.com, charles.chenxin@huawei.com, liuyixian@huawei.com, xushaobo2@huawei.com, zhangxiping3@huawei.com, linuxarm@huawei.com, linux-kernel@vger.kernel.org References: <1504084998-64397-1-git-send-email-xavier.huwei@huawei.com> <1504084998-64397-6-git-send-email-xavier.huwei@huawei.com> <1506359213.120853.75.camel@redhat.com> <20170925171821.GQ25094@mtr-leonro.local> <1506361015.120853.81.camel@redhat.com> <59CA5261.80209@huawei.com> <81dd332d-e060-d7e3-bec9-1791511c5470@redhat.com> <9172f8c5-3dd6-a573-8e28-1b3ae4b1726b@tom.com> <1b8bda3b-c514-7e46-08bf-3ea50ea68096@redhat.com> <1506516105.82712.0.camel@redhat.com> From: "Wei Hu (Xavier)" Message-ID: <05bebc4c-eabf-a21b-cf5a-79a09fe71e02@tom.com> Date: Thu, 28 Sep 2017 12:34:38 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <1506516105.82712.0.camel@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2017/9/27 20:41, Doug Ledford wrote: > On Wed, 2017-09-27 at 08:21 -0400, Doug Ledford wrote: >>> And if you argee, after this patchset has been accepted we will >> send a >>> following up patch : >>> In hns_roce_cmq_send function, replace >>> usleep_range(1000, 2000); >>> with the following statement: >>> udelay(1); >>> And if so, we can avoid using usleep_range function in >> spin_lock_bh >>> spin region, >>> because it probally cause calltrace. >> Ok, I'm fine with that. I'll pull these in. > OK, these are in my tree, thanks. Thanks a lot.     Regards Wei Hu