[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v4 3/8] s390/sclp: rework sclp boundary and length checks

From: Collin Walling
Subject: Re: [PATCH v4 3/8] s390/sclp: rework sclp boundary and length checks
Date: Fri, 24 Jul 2020 11:06:11 -0400
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.9.0

On 7/23/20 2:26 AM, Cornelia Huck wrote:
> On Tue, 21 Jul 2020 14:40:14 -0400
> Collin Walling <walling@linux.ibm.com> wrote:
>> On 7/21/20 4:41 AM, David Hildenbrand wrote:
>>> The options I would support are
>>> 1. "sccb_boundary_is_valid" which returns "true" if valid
>>> 2. "sccb_boundary_is_invalid" which returns "true" if invalid
>>> 3. "sccb_boundary_validate" which returns "0" if valid and -EINVAL if not.
>>> Which makes reading this code a bit easier.
> Of these, I like option 1 best.
>> Sounds good. I'll takes this into consideration for the next round. (I
>> may wait just a little longer for that to allow more reviews to come in
>> from whoever has the time, if that's okay.)
> We have to wait for (a) QEMU to do a release and (b) the Linux changes
> to merge upstream anyway, so we're not in a hurry :)
> As said before, it already looked good from my side, but the suggested
> changes are fine with me as well.

Okay, thanks for the info.

I do want to send out a v5 of these patches. While working with someone
who is implementing the kernel support for the extended-length SCCB, we
found some snags. I'll highlight these changes/fixes in the respective
patches on the next version.



Stay safe and stay healthy

reply via email to

[Prev in Thread] Current Thread [Next in Thread]