Re: [regression] linux318, linux41 - kernel stack is corrupted

From	Sasha Levin <sasha.levin@oracle.com>
Newsgroups	linux.kernel
Subject	Re: [regression] linux318, linux41 - kernel stack is corrupted
Date	2016-04-23 04:10 +0200
Message-ID	<rqVbc-3Az-7@gated-at.bofh.it> (permalink)
References	(2 earlier) <rqEan-6Su-23@gated-at.bofh.it> <rqEan-6Su-21@gated-at.bofh.it> <rqEtJ-7kx-17@gated-at.bofh.it> <rqEDo-7qZ-9@gated-at.bofh.it> <rqGlP-lQ-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 04/22/2016 06:16 AM, Mike Galbraith wrote:
> On Fri, 2016-04-22 at 17:23 +0900, Greg Kroah-Hartman wrote:
>> On Fri, Apr 22, 2016 at 10:10:59AM +0200, Sebastian M. Bobrecki wrote:
>>> W dniu 22.04.2016 o 09:55, Greg Kroah-Hartman pisze:
>>>> On Fri, Apr 22, 2016 at 09:47:04AM +0200, Sebastian M. Bobrecki wrote:
>>>>> Hi,
>>>>>
>>>>> I just hit the same with 4.1.22 on Gentoo. 4.1.21 are working fine.
>>>>>
>>>>> On 22.04.2016 at 08:46, Greg Kroah-Hartman wrote:
>>>>>> ...
>>>>>> You are going to have to be a bit more specific here...
>>>>>> What is the oops message?  How do you reproduce this?  Does it also
>>>>>> happen on 4.6-rc4?
>>>>>>
>>>>>> Can you run 'git bisect' to find the offending patch?
>>>>>>
>>>>> Greg have you seen screenshots linked by Philip?
>>>> I saw no such screenshots in the email.
>>> They are here: https://github.com/manjaro/packages-core/issues/36
>>
>> Looks like an acpi thermal patch got backported incorrectly, again, 'git
>> bisect' is going to help out the best here.
> 
> That'll work, but requires repeatedly ignoring the big-fat-warning :)
> 
> Backport of 81ad4276b505e987dd8ebbdf63605f92cd172b52 failed to adjust
> for intervening ->get_trip_temp() argument type change, thus causing
> stack protector to panic.
> 
> drivers/thermal/thermal_core.c: In function ‘thermal_zone_device_register’:
> drivers/thermal/thermal_core.c:1569:41: warning: passing argument 3 of
> ‘tz->ops->get_trip_temp’ from incompatible pointer type [-Wincompatible-pointer-types]
>    if (tz->ops->get_trip_temp(tz, count, &trip_temp))
>                                          ^
> drivers/thermal/thermal_core.c:1569:41: note: expected ‘long unsigned int *’
> but argument is of type ‘int *’
> 
> CC: <stable@vger.kernel.org> #3.18,#4.1
> Signed-off-by: Mike Galbraith <umgwanakikbuti@gmail.com>
> ---
>  drivers/thermal/thermal_core.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -1491,7 +1491,7 @@ struct thermal_zone_device *thermal_zone
>  {
>  	struct thermal_zone_device *tz;
>  	enum thermal_trip_type trip_type;
> -	int trip_temp;
> +	unsigned long trip_temp;
>  	int result;
>  	int count;
>  	int passive = 0;
> 

Thanks!

I'll put it on both 3.18 and 4.1, and will try to ship it within a day or
two once all tests have gone through.


Thanks,
Sasha

Thread

[regression] linux318, linux41 - kernel stack is corrupted Philip Müller <philm@manjaro.org> - 2016-04-22 08:40 +0200
  Re: [regression] linux318, linux41 - kernel stack is corrupted Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-22 08:50 +0200
    Re: [regression] linux318, linux41 - kernel stack is corrupted Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-22 10:00 +0200
      Re: [regression] linux318, linux41 - kernel stack is corrupted "Sebastian M. Bobrecki" <sebastian@bobrecki.pl> - 2016-04-22 10:20 +0200
        Re: [regression] linux318, linux41 - kernel stack is corrupted Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2016-04-22 10:30 +0200
          Re: [regression] linux318, linux41 - kernel stack is corrupted Mike Galbraith <umgwanakikbuti@gmail.com> - 2016-04-22 12:20 +0200
            Re: [regression] linux318, linux41 - kernel stack is corrupted Sasha Levin <sasha.levin@oracle.com> - 2016-04-23 04:10 +0200
    Re: [regression] linux318, linux41 - kernel stack is corrupted "Sebastian M. Bobrecki" <sebastian@bobrecki.pl> - 2016-04-22 10:00 +0200

csiph-web