linux-kernel - Re: [PULL] IIO fixes for 4.10 set 3

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Tue, 14 Feb 2017 15:29:24 +0100
From:   Peter Rosin <peda@...ntia.se>
To:     Jonathan Cameron <jic23@...nel.org>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
CC:     <linux-kernel@...r.kernel.org>, Ken Lin <ken.lin@...antech.com>,
        <linux-iio@...r.kernel.org>
Subject: Re: [PULL] IIO fixes for 4.10 set 3 - a couple of regression fixes.

On 2017-02-11 10:16, Jonathan Cameron wrote:
> On 11/02/17 07:17, Greg Kroah-Hartman wrote:
>> On Fri, Feb 10, 2017 at 11:35:35PM +0100, Peter Rosin wrote:
>>>> On Sun, Feb 05, 2017 at 10:35:02AM +0000, Jonathan Cameron wrote:
>>>>> The following changes since commit 5c113b5e0082e90d2e1c7b12e96a7b8cf0623e27:
>>>>>
>>>>>   iio: dht11: Use usleep_range instead of msleep for start signal (2017-01-22 13:35:40 +0000)
>>>>>
>>>>> are available in the git repository at:
>>>>>
>>>>>   git://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git tags/iio-fixes-for-4.10c
>>>>
>>>> It's a bit late for 4.10 for me, can I just pull this into my -next
>>>> branch and will they get to 4.10.1 properly?  Meaning, do that have cc:
>>>> stable markings on them?  Or do you want to fix that up and resend this
>>>> request?
>>>
>>> Hi Greg,
>>>
>>> You should ask Ken Lin who has the HW and who is apparently affected.
>>> I think it's bad that you are willing to have a known regression hit
>>> v4.10 when all was fine in v4.9. Or perhaps you didn't realize that
>>> the regression was from this cycle?
>>>
>>> The fixes are obvious. I don't understand your hesitation.
>>
>> My "hesitation" is that I'm about to get on a plane for a day or so and
>> don't have the time to get this to Linus before 4.10-final is out this
>> Sunday.  Getting it in a week later should be ok, we all make mistakes,
>> as long as we fix them it's all good, and for 4.10.1 should be ok.
> Who knows, maybe Linus will delay another week anyway ;)  Wasn't looking
> that clear when I sent these out.
> 
> From my point of view, sure 4.10.1 should be fine. I'll put stable CCs
> on them. Will send the pull out shortly so I don't forget about them.
> 
> Pull request clearly now becomes first set of fixes for 4.11.

I would like to point out that one of the commit messages was destroyed
in the rebase. I.e. commit 9cf6cdba586c ("iio: pressure: mpl3115: do not
rely on structure field ordering") is now missing this line:

# cat /sys/bus/iio/devices/iio\:device1/in_pressure_scale_available

which was present before the "Segmentation fault" line before the patch
was rebased. Without the missing line, the commit message makes much
less sense...

Can this please be fixed? Perhaps there is even time for it to make v4.10?

Cheers,
peda