Re: [PATCH 13/17] smartpqi: remove timeout for cache flush operations

2016-08-31 Thread Tomas Henzl
On 26.8.2016 18:45, Don Brace wrote: > From: Kevin Barnett > > Some cache flush operations can take longer than the > timeout value. Best to not impose a time limit to > handle all cases. > > Reviewed-by: Scott Teel > Reviewed-by: Scott

Re: [PATCH 13/17] smartpqi: remove timeout for cache flush operations

2016-08-30 Thread Johannes Thumshirn
On Mon, Aug 29, 2016 at 09:36:32PM +, Don Brace wrote: > > -Original Message- [...] > > > > IIRC I asked you to introduce this timeout, as you're calling > > pqi_flush_cache() from the driver's shutdown callback and I still > > doubt users like their shutdown's to hang indefinitely

RE: [PATCH 13/17] smartpqi: remove timeout for cache flush operations

2016-08-29 Thread Don Brace
> elli...@hpe.com; linux-scsi@vger.kernel.org > Subject: Re: [PATCH 13/17] smartpqi: remove timeout for cache flush > operations > > EXTERNAL EMAIL > > > On Fri, Aug 26, 2016 at 11:45:24AM -0500, Don Brace wrote: > > From: Kevin Barnett <kevin.barn...@microsem

Re: [PATCH 13/17] smartpqi: remove timeout for cache flush operations

2016-08-29 Thread Johannes Thumshirn
On Fri, Aug 26, 2016 at 11:45:24AM -0500, Don Brace wrote: > From: Kevin Barnett > > Some cache flush operations can take longer than the > timeout value. Best to not impose a time limit to > handle all cases. > > Reviewed-by: Scott Teel >

[PATCH 13/17] smartpqi: remove timeout for cache flush operations

2016-08-26 Thread Don Brace
From: Kevin Barnett Some cache flush operations can take longer than the timeout value. Best to not impose a time limit to handle all cases. Reviewed-by: Scott Teel Reviewed-by: Scott Benesh Signed-off-by: