[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: |
David Hildenbrand |
Subject: |
Re: [PATCH v1 1/2] s390x: Add sclp boundary check and fix error priority |
Date: |
Fri, 27 Sep 2019 11:29:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 |
On 27.09.19 11:20, Janosch Frank wrote:
> 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 :-)
>
Now that's the spirit :)
--
Thanks,
David / dhildenb
[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