RE: [v3] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.
> -Original Message- > From: Scott Wood [mailto:o...@buserror.net] > Sent: Saturday, June 24, 2017 9:47 PM > To: Karim Eshapa <karim.esh...@gmail.com> > Cc: Roy Pledge <roy.ple...@nxp.com>; linux-ker...@vger.kernel.org; > Claudiu Manoil <claudiu.man...@nxp.com>; colin.k...@canonical.com; > linuxppc-dev@lists.ozlabs.org; linux-arm-ker...@lists.infradead.org; Leo Li > <leoyang...@nxp.com> > Subject: Re: [v3] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck > hacking jiffies. > > On Fri, May 05, 2017 at 07:45:18AM +0200, Karim Eshapa wrote: > > Use msleep() instead of stucking with > > long delay will be more efficient. > > > > Signed-off-by: Karim Eshapa <karim.esh...@gmail.com> > > --- > > drivers/soc/fsl/qbman/qman.c | 6 +- > > 1 file changed, 1 insertion(+), 5 deletions(-) > > Acked-by: Scott Wood <o...@buserror.net> > > (though the subject line should be "soc/qman: ...") > > Leo, are you going to send this patch (and other qman patches) via arm-soc? Yes. I can take it through the pull request for soc/fsl via arm-soc. As mentioned in the feedback from David in another email, probably we should update the comment and commit message to mention how 1 cycles becomes 1ms. Regards, Leo
Re: [v3] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.
On Fri, May 05, 2017 at 07:45:18AM +0200, Karim Eshapa wrote: > Use msleep() instead of stucking with > long delay will be more efficient. > > Signed-off-by: Karim Eshapa> --- > drivers/soc/fsl/qbman/qman.c | 6 +- > 1 file changed, 1 insertion(+), 5 deletions(-) Acked-by: Scott Wood (though the subject line should be "soc/qman: ...") Leo, are you going to send this patch (and other qman patches) via arm-soc? -Scott
RE: [PATCH v3] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.
Use msleep() instead of stucking with long delay will be more efficient. Signed-off-by: Karim Eshapa--- drivers/soc/fsl/qbman/qman.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c b/drivers/soc/fsl/qbman/qman.c index 3d891db..18d391e 100644 --- a/drivers/soc/fsl/qbman/qman.c +++ b/drivers/soc/fsl/qbman/qman.c @@ -1084,11 +1084,7 @@ static int drain_mr_fqrni(struct qm_portal *p) * entries well before the ring has been fully consumed, so * we're being *really* paranoid here. */ - u64 now, then = jiffies; - - do { - now = jiffies; - } while ((then + 1) > now); + msleep(1); msg = qm_mr_current(p); if (!msg) return 0; -- 2.7.4