Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-05 Thread Scott Wood
On Fri, 2017-05-05 at 08:01 +0200, Karim Eshapa wrote: > > On 5/4/2017 5:07 PM, Scott Wood wrote: > > > On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: > > > > +stop = jiffies + 1; > > > > +/* > > > > + * if MR was full and h/w had other FQRNI entries to produce, we > > > >

Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-05 Thread Scott Wood
On Fri, 2017-05-05 at 08:01 +0200, Karim Eshapa wrote: > > On 5/4/2017 5:07 PM, Scott Wood wrote: > > > On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: > > > > +stop = jiffies + 1; > > > > +/* > > > > + * if MR was full and h/w had other FQRNI entries to produce, we > > > >

RE: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-05 Thread Karim Eshapa
>On 5/4/2017 5:07 PM, Scott Wood wrote: >> On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: >>> +stop = jiffies + 1; >>> +/* >>> + * if MR was full and h/w had other FQRNI entries to produce, we >>> + * need to allow it time to produce those entries once the >>> + *

RE: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-05 Thread Karim Eshapa
>On 5/4/2017 5:07 PM, Scott Wood wrote: >> On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: >>> +stop = jiffies + 1; >>> +/* >>> + * if MR was full and h/w had other FQRNI entries to produce, we >>> + * need to allow it time to produce those entries once the >>> + *

Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-04 Thread Roy Pledge
On 5/4/2017 5:07 PM, Scott Wood wrote: > On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: >> +stop = jiffies + 1; >> +/* >> + * if MR was full and h/w had other FQRNI entries to produce, we >> + * need to allow it time to produce those entries once the >> + * existing

Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-04 Thread Roy Pledge
On 5/4/2017 5:07 PM, Scott Wood wrote: > On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: >> +stop = jiffies + 1; >> +/* >> + * if MR was full and h/w had other FQRNI entries to produce, we >> + * need to allow it time to produce those entries once the >> + * existing

Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-04 Thread Scott Wood
On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: > + stop = jiffies + 1; > + /* > +  * if MR was full and h/w had other FQRNI entries to produce, we > +  * need to allow it time to produce those entries once the > +  * existing entries are consumed. A worst-case

Re: [PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-04 Thread Scott Wood
On Thu, 2017-05-04 at 06:58 +0200, Karim Eshapa wrote: > + stop = jiffies + 1; > + /* > +  * if MR was full and h/w had other FQRNI entries to produce, we > +  * need to allow it time to produce those entries once the > +  * existing entries are consumed. A worst-case

[PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-03 Thread Karim Eshapa
Avoid stuck and hacking jiffies for a long time and using msleep() for certatin numeber of cylces without the factor of safety but using the the long delay considering the factor of safety with the while loop such that after msleep for a short period of time we check the msg if it's OK, breaking

[PATCH v2] drivers:soc:fsl:qbman:qman.c: Sleep instead of stuck hacking jiffies.

2017-05-03 Thread Karim Eshapa
Avoid stuck and hacking jiffies for a long time and using msleep() for certatin numeber of cylces without the factor of safety but using the the long delay considering the factor of safety with the while loop such that after msleep for a short period of time we check the msg if it's OK, breaking