Re: [PATCH 1/6] cxlflash: Reset command ioasc

2018-01-08 Thread Uma Krishnan
> On Jan 7, 2018, at 1:36 PM, Matthew R. Ochs wrote: > > On Thu, Jan 04, 2018 at 05:33:48PM +1100, Andrew Donnellan wrote: >> On 04/01/18 09:54, Uma Krishnan wrote: >>> In the event of a command failure, cxlflash returns the failure to the >>> upper layers to process. After processing the error,

Re: [PATCH 1/6] cxlflash: Reset command ioasc

2018-01-07 Thread Matthew R. Ochs
On Thu, Jan 04, 2018 at 05:33:48PM +1100, Andrew Donnellan wrote: > On 04/01/18 09:54, Uma Krishnan wrote: > >In the event of a command failure, cxlflash returns the failure to the > >upper layers to process. After processing the error, when the command is > >queued again, the private command struc

Re: [PATCH 1/6] cxlflash: Reset command ioasc

2018-01-07 Thread Matthew R. Ochs
On Wed, Jan 03, 2018 at 04:54:02PM -0600, Uma Krishnan wrote: > In the event of a command failure, cxlflash returns the failure to the > upper layers to process. After processing the error, when the command is > queued again, the private command structure will not be zeroed and the > ioasc could be

Re: [PATCH 1/6] cxlflash: Reset command ioasc

2018-01-03 Thread Andrew Donnellan
On 04/01/18 09:54, Uma Krishnan wrote: In the event of a command failure, cxlflash returns the failure to the upper layers to process. After processing the error, when the command is queued again, the private command structure will not be zeroed and the ioasc could be stale. Per the SISLite speci

[PATCH 1/6] cxlflash: Reset command ioasc

2018-01-03 Thread Uma Krishnan
In the event of a command failure, cxlflash returns the failure to the upper layers to process. After processing the error, when the command is queued again, the private command structure will not be zeroed and the ioasc could be stale. Per the SISLite specification, the AFU only sets the ioasc in