All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Lord <kernel@teksavvy.com>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Linux 3.3-rc5
Date: Thu, 01 Mar 2012 19:17:48 -0500	[thread overview]
Message-ID: <4F5011AC.2020909@teksavvy.com> (raw)
In-Reply-To: <CA+55aFyij_ORw+BE-Sbkt9Y9qcUQSmW_4e-r7p3t0DNA4Sg9pA@mail.gmail.com>

On 12-03-01 11:22 AM, Linus Torvalds wrote:
> On Thu, Mar 1, 2012 at 6:47 AM, Mark Lord <kernel@teksavvy.com> wrote:
>>
>> It's close (after adding a missing left-paren), but not 100% working yet.
>> In particular, this command fails to get valid data:  i8kctl bios
>>
>>    ioctl(3, I8K_BIOS_VERSION, 0xbfc543c8)  = -1 EINVAL (Invalid argument)
>
> Hmm. That makes no sense.

No, that was my misinterpretation.
The BIOS_VERSION call is broken for different reasons,
and my original all-inclusive patch has a fix for it too.  :)

> Does *this* work? And if not, can you do an strace, but use the "-e
> raw=ioctl" option to make it show the ioctl numbers as raw hex to
> verify..

Here's a go, 64-bit kernel (3.2.8), 32-bit userspace:

Dell laptop SMM driver v1.14 21/02/2005 Massimo Dal Zotto (dz@debian.org)
ioctl32(i8kctl:1474): Unknown cmd fd(3) cmd(80046980){t:'i';sz:4} arg(ff8b1fd8)
on /proc/i8k
ioctl32(i8kctl:1475): Unknown cmd fd(3) cmd(c0046986){t:'i';sz:4} arg(ff90ed5c)
on /proc/i8k
ioctl32(i8kctl:1475): Unknown cmd fd(3) cmd(c0046986){t:'i';sz:4} arg(ff90ed5c)
on /proc/i8k

I'll reboot/redo that again with strace shortly.

  reply	other threads:[~2012-03-02  0:17 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-25 20:34 Linux 3.3-rc5 Linus Torvalds
2012-02-27 15:23 ` Mark Lord
2012-02-28 18:30   ` Linus Torvalds
2012-02-28 19:29     ` Andreas Schwab
2012-02-28 19:39       ` Linus Torvalds
2012-03-01 14:47     ` Mark Lord
2012-03-01 15:26       ` [PATCH] i8k: fix ioctl handing (was Re: Linux 3.3-rc5) Mark Lord
2012-03-02  3:25         ` Linus Torvalds
2012-03-01 16:22       ` Linux 3.3-rc5 Linus Torvalds
2012-03-02  0:17         ` Mark Lord [this message]
2012-03-02  0:21           ` Mark Lord
2012-03-02  0:31             ` Linus Torvalds
2012-03-02  3:48               ` Mark Lord
2012-02-27 21:59 ` Michael S. Tsirkin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4F5011AC.2020909@teksavvy.com \
    --to=kernel@teksavvy.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.