Page 2 of 9 FirstFirst 1234 ... LastLast
Results 11 to 20 of 86

Thread: [SOLVED] MPB4,1/MBA 'wait status failed: c != 8' solved?

  1. #11
    Join Date
    May 2008
    Beans
    745

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    _mario_,

    thanks for the report! I have a comment, a question and a script:

    1. Occasional read errors are quite natural, given that there is a finite probability (p) for every read to fail (and it will stay that way unless the applesmc sensor interface introduces buffered reading). The probability of *all* 12 sensors being ok in a read is (1-p)^12, which quickly becomes small if p is only slightly greater than zero.

    2. The diagnostics suggests some sensors truly are not readable, but to make sure, one can try reading a single value, like
    Code:
    cat /sys/devices/platform/applesmc.768/temp1_input
    3. To make sure exactly what sensors are there, try running the attached script as root (courtesy of Nicolas Boichat).
    Attached Files Attached Files
    Last edited by kosumi68; October 19th, 2008 at 07:55 PM.

  2. #12
    Join Date
    Aug 2005
    Location
    Huntsville, AL, USA
    Beans
    7,526
    Distro
    Ubuntu

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    Quote Originally Posted by kosumi68 View Post
    1. Occasional read errors are quite natural, given that there is a finite probability (p) for every read to fail (and it will stay that way unless the applesmc sensor interface introduces buffered reading). The probability of *all* 12 sensors being ok in a read is (1-p)^12, which quickly becomes small if p is only slightly less than unity.
    free stats lesson today...

  3. #13
    Join Date
    Jul 2008
    Beans
    245
    Distro
    Ubuntu 10.10 Maverick Meerkat

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    Quote Originally Posted by kosumi68 View Post
    2. The diagnostics suggests some sensors truly are not readable, but to make sure, one can try reading a single value.
    Yes, as stated in my previous post the sensors 1, 6, 7 & 12 are always not readable.

    Quote Originally Posted by kosumi68 View Post
    3. To make sure exactly what sensors are there, try running the attached script as root (courtesy of Nicolas Boichat).
    I attached the output of this command. Btw: What does this mean? Is there any documentation one can read?

    thanks & ciao,
    Mario
    Attached Files Attached Files
    Last edited by _mario_; September 22nd, 2008 at 10:41 AM. Reason: wrong qoute tag

  4. #14
    Join Date
    May 2008
    Beans
    745

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    I attached the output of this command. Btw: What does this mean? Is there any documentation one can read?
    Great! The sensors list shows that MBP4,1 has 12 sensors, but four of them have different names from the older MBPs, precisely the ones you reported as failing. I will submit a patch for it shortly.

    Documentation? This script was given to me by the applesmc maintainer. Together with the kernel code, it is about the extent of what I know. There might be some general documentation regarding the sensor codes out there, but Apple-specific details are few and far between.

  5. #15
    Join Date
    May 2008
    Beans
    745

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    _mario_,

    Please find enclosed a patch to fix the temperature sensor list on MBP4,1, and a precompiled 2.6.27-3-generic i386 binary including that patch.

    Hope this helps!
    Attached Files Attached Files

  6. #16
    Join Date
    Jul 2008
    Beans
    245
    Distro
    Ubuntu 10.10 Maverick Meerkat

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    Quote Originally Posted by kosumi68 View Post
    Please find enclosed a patch to fix the temperature sensor list on MBP4,1, and a precompiled 2.6.27-3-generic i386 binary including that patch.
    Hope this helps!
    Thanks. I applied the patch to the source file and compiled myself, so now it seems to work:
    Code:
    performance:   reading all temperatures
     fail frequency:        .262
     reads per second:      71
    and:
    Code:
    applesmc-isa-0300
    Adapter: ISA adapter
    Left side  :1006 RPM  (min = 1000 RPM)
    Right side :1002 RPM  (min = 1000 RPM)
    temp1:       +31.8°C                                    
    temp2:       +50.0°C                                    
    temp3:       +47.5°C                                    
    temp4:       +53.0°C                                    
    temp5:       +49.5°C                                    
    temp6:       +70.0°C                                    
    temp7:       +38.5°C                                    
    temp8:       +48.5°C                                    
    temp9:       +41.0°C                                    
    temp10:      +42.0°C                                    
    temp11:      +43.8°C                                    
    temp12:      +33.5°C
    There are still some messages left (as you already pointed out) we shouldn't be concerned about:
    Code:
    Sep 22 15:35:01 snoopy kernel: [ 1468.433884] applesmc: wait status failed: 5 != 0
    Sep 22 15:35:01 snoopy kernel: [ 1468.460291] applesmc: wait status failed: 5 != 1
    I asked for documentation to better understand how this chip works, and to get to know which temperature sensor is where inside the machine...

    ciao,
    Mario

  7. #17
    Join Date
    May 2008
    Beans
    745

    Re: MPB4,1/MBA 'wait status failed: c != 8' solved?

    Quote Originally Posted by _mario_ View Post
    Code:
    performance:   reading all temperatures
     fail frequency:        .262
     reads per second:      71
    Excellent, I will send the patch to kernel.org.

    Thanks!

    PS. I am switching this thread to solved - always wanted to do that

  8. #18
    Join Date
    Jul 2008
    Beans
    245
    Distro
    Ubuntu 10.10 Maverick Meerkat

    Re: [SOLVED] MPB4,1/MBA 'wait status failed: c != 8' solved?

    Hi again,

    maybe we should also change the SYSFS directory from /sys/class/leds/smc::kbd_backlight to /sys/class/leds/smc:kbd_backlight. At least this is where pommed (version 1.16~mactel1) searchs the files. Or is it some bug in pommed? What is right? The current stable upstream kernel version still uses the former path.

    ciao,
    Mario

  9. #19
    Join Date
    Aug 2005
    Location
    Huntsville, AL, USA
    Beans
    7,526
    Distro
    Ubuntu

    Re: [SOLVED] MPB4,1/MBA 'wait status failed: c != 8' solved?

    Quote Originally Posted by _mario_ View Post
    Hi again,

    maybe we should also change the SYSFS directory from /sys/class/leds/smc::kbd_backlight to /sys/class/leds/smc:kbd_backlight. At least this is where pommed (version 1.16~mactel1) searchs the files. Or is it some bug in pommed? What is right? The current stable upstream kernel version still uses the former path.

    ciao,
    Mario
    the mactel compile is old. Try this one:
    http://packages.debian.org/lenny/pommed
    Last edited by cyberdork33; September 29th, 2008 at 06:18 PM.

  10. #20
    Join Date
    May 2008
    Beans
    745

    Re: [SOLVED] MPB4,1/MBA 'wait status failed: c != 8' solved?

    Not only did the name change from leds/smc:kbd_backlight to leds/smc::kbd_backlight as you noticed, but it lately (intrepid) changed to leds:smc::kbd_backlight... Somewhat confusing.

    Btw, with the help of Bob McElrath, the applesmc reading errors have lately been further improved; I am attaching the latest version for Intrepid (applesmc.c.zip at the bottom). To get a version usable in Hardy, simply apply the trivial patch below.

    Cheers!

    Code:
    --- upstream-2.6/drivers/hwmon/applesmc.c       2008-09-29 17:50:30.000000000 +0200
    +++ hardy/drivers/hwmon/applesmc.c      2008-09-27 17:07:33.000000000 +0200
    @@ -565,7 +565,7 @@ static ssize_t applesmc_light_show(struc
                    ret = applesmc_get_key_type(LIGHT_SENSOR_LEFT_KEY, query);
                    if (ret)
                            goto out;
    -               data_length = clamp_val(query[0], 0, 10);
    +               data_length = query[0];
                    printk(KERN_INFO "applesmc: light sensor data length set to "
                            "%d\n", data_length);
            }
    @@ -951,7 +951,7 @@ static ssize_t applesmc_key_at_index_sto
     }
     
     static struct led_classdev applesmc_backlight = {
    -       .name                   = "smc::kbd_backlight",
    +       .name                   = "smc:kbd_backlight",
            .default_trigger        = "nand-disk",
            .brightness_set         = applesmc_brightness_set,
     };
    Attached Files Attached Files
    Last edited by kosumi68; October 13th, 2008 at 10:48 PM.

Page 2 of 9 FirstFirst 1234 ... LastLast

Tags for this Thread

Bookmarks

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •