[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 1/2] s390x: Add sclp boundary check and fix error priority
|
From: |
Janosch Frank |
|
Subject: |
Re: [PATCH v1 1/2] s390x: Add sclp boundary check and fix error priority |
|
Date: |
Fri, 27 Sep 2019 11:20:53 +0200 |
|
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
On 9/27/19 11:17 AM, David Hildenbrand wrote:
> On 27.09.19 11:14, Janosch Frank wrote:
>> On 9/27/19 10:51 AM, David Hildenbrand wrote:
>>> On 26.09.19 13:33, Claudio Imbrenda wrote:
>>>> From: Janosch Frank <address@hidden>
>>>>
>>>> * All sclp codes need to be checked for page boundary violations.
>>>> * Requests over 4k are not a spec exception.
>>>> * Invalid command checking has to be done before the boundary check.
>>>
>>> Can we split this patch up so we fix one thing at a time?
>>
>> Sure, but we would end up with very small patches.
>> Do you want that?
>
> Why should I say no to easy-to-review, logically consistent, small
> chunks? I have shortcuts for my RB's and ACK's, so I don't have to type
> much ;)
>
Higher patch count for me, win - win :-)
signature.asc
Description: OpenPGP digital signature
[PATCH v1 2/2] s390x: Fix SCLP return code when buffer too small, Claudio Imbrenda, 2019/09/26
Re: [PATCH v1 0/2] s390x: SCLP error cleanup, David Hildenbrand, 2019/09/27