From: "H. Peter Anvin" <hpa@zytor.com>
To: Dmitry Torokhov <dtor@vmware.com>
Cc: "mingo@redhat.com" <mingo@redhat.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"greg@kroah.com" <greg@kroah.com>,
"hjanssen@microsoft.com" <hjanssen@microsoft.com>,
"ksrinivasan@novell.com" <ksrinivasan@novell.com>,
"tglx@linutronix.de" <tglx@linutronix.de>,
Alok Kataria <akataria@vmware.com>,
"linux-tip-commits@vger.kernel.org"
<linux-tip-commits@vger.kernel.org>
Subject: Re: [tip:x86/cpu] Modify the VMware balloon driver for the new x86_hyper API
Date: Mon, 10 May 2010 10:30:43 -0700 [thread overview]
Message-ID: <4BE842C3.5040300@zytor.com> (raw)
In-Reply-To: <201005100917.28980.dtor@vmware.com>
On 05/10/2010 09:17 AM, Dmitry Torokhov wrote:
> On Monday 10 May 2010 08:23:28 am H. Peter Anvin wrote:
>> On 05/10/2010 01:06 AM, Dmitry Torokhov wrote:
>>> Source please? I was not aware that there was a standard governing
>>> returns code for module init methods.
>>
>> ENODEV means "not a device node."
>> ENXIO means "hardware not present."
>>
>
> There is no device node in question. Again, could you please point me to
> the list of allowed error codes for init methods? According to SUS,
> we need to follow ERROR section of the appropriate function, and I do
> not believe that spec covers cases if driver binding and module loading.
>
> FWIW -ENODEV is explicitly allowed in our device core and means "device
> not found", especially in context of platform devices. I do not see the
> need of changing that.
>
The problem with this abuse of ENODEV is that people start using it
elsewhere, where it *DOES* matter than ENXIO is the proper return code.
Yes, we handle incorrect uses of ENODEV, but the right thing to do is
to teach people to use ENXIO properly.
-hpa
next prev parent reply other threads:[~2010-05-10 17:31 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-07 22:43 RFC - Cleaned up hypervisor layer H. Peter Anvin
2010-05-07 23:02 ` Greg KH
2010-05-07 23:08 ` H. Peter Anvin
2010-05-07 23:12 ` Greg KH
2010-05-07 23:18 ` H. Peter Anvin
2010-05-07 23:55 ` [PATCH] HyperV: fix up the license to mshyperv.c Greg KH
2010-05-08 1:58 ` [tip:x86/cpu] x86, " tip-bot for Greg Kroah-Hartman
2010-05-08 1:58 ` [tip:x86/cpu] x86: Clean up the hypervisor layer tip-bot for H. Peter Anvin
2010-05-08 8:16 ` Ingo Molnar
2010-05-19 16:54 ` [PATCH] Hyperv: Export the symbol that tracks hyperv features and recommendations Ky Srinivasan
2010-05-19 17:03 ` Greg KH
2010-05-08 5:54 ` RFC - Cleaned up hypervisor layer Dmitry Torokhov
2010-05-09 8:18 ` [tip:x86/cpu] x86, hypervisor: Export the x86_hyper* symbols tip-bot for H. Peter Anvin
2010-05-09 8:19 ` [tip:x86/cpu] Modify the VMware balloon driver for the new x86_hyper API tip-bot for H. Peter Anvin
2010-05-09 8:20 ` H. Peter Anvin
2010-05-10 8:06 ` Dmitry Torokhov
2010-05-10 15:23 ` H. Peter Anvin
2010-05-10 16:17 ` Dmitry Torokhov
2010-05-10 17:30 ` H. Peter Anvin [this message]
2010-05-10 17:52 ` Dmitry Torokhov
2010-05-10 18:05 ` H. Peter Anvin
2010-05-09 10:47 ` Ingo Molnar
2010-05-10 5:56 ` [tip:x86/cpu] x86, hypervisor: add missing <linux/module.h> tip-bot for H. Peter Anvin
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=4BE842C3.5040300@zytor.com \
--to=hpa@zytor.com \
--cc=akataria@vmware.com \
--cc=dtor@vmware.com \
--cc=greg@kroah.com \
--cc=hjanssen@microsoft.com \
--cc=ksrinivasan@novell.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tip-commits@vger.kernel.org \
--cc=mingo@redhat.com \
--cc=tglx@linutronix.de \
/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.