Re: [PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-11-06 Thread Thomas Huth
On 28/10/2022 22.22, Eric Farman wrote: On Thu, 2022-10-27 at 23:23 +0200, Peter Jin wrote: Revert the control and flag bits in the subchannel status word in case the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH). According to POPS, the control and flag bits are only changed if

Re: [PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-10-31 Thread Peter Jin
To everyone else on the mailing lists, I apologize for any confusion caused by this being a "v2" of the original "v1" patch and not submitting a changelog. I am rather new to these kinds of mailing lists, though Eric and Matt here have been really helpful in getting this correct. I will rememb

Re: [PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-10-31 Thread Matthew Rosato
On 10/28/22 4:22 PM, Eric Farman wrote: > On Thu, 2022-10-27 at 23:23 +0200, Peter Jin wrote: >> Revert the control and flag bits in the subchannel status word in >> case >> the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH). >> According to POPS, the control and flag bits are only

Re: [PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-10-28 Thread Peter Jin
On 10/28/22 16:22, Eric Farman wrote: On Thu, 2022-10-27 at 23:23 +0200, Peter Jin wrote: Revert the control and flag bits in the subchannel status word in case the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH). According to POPS, the control and flag bits are only changed if

Re: [PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-10-28 Thread Eric Farman
On Thu, 2022-10-27 at 23:23 +0200, Peter Jin wrote: > Revert the control and flag bits in the subchannel status word in > case > the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH). > According to POPS, the control and flag bits are only changed if > SSCH, > CSCH, and HSCH return CC

[PATCH v2] s390x/css: revert SCSW ctrl/flag bits on error

2022-10-27 Thread Peter Jin
Revert the control and flag bits in the subchannel status word in case the SSCH operation fails with non-zero CC (ditto for CSCH and HSCH). According to POPS, the control and flag bits are only changed if SSCH, CSCH, and HSCH return CC 0, and no other action should be taken otherwise. In order to s