public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <tj@kernel.org>
To: Kay Sievers <kay.sievers@vrfy.org>
Cc: Greg KH <greg@kroah.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] uevent: handle duplicate uevent_var keys properly
Date: Fri, 29 Aug 2008 10:02:25 +0200	[thread overview]
Message-ID: <48B7AD11.2070906@kernel.org> (raw)
In-Reply-To: <3ae72650808290048v1a5d7e51pc68270b2a8d6faa@mail.gmail.com>

Kay Sievers wrote:
> Not sure if I understand that correctly. Remember, that there is a
> symlink "subsystem" at each device, and udev, HAL, DeviceKit reads it.
> If the uevent environment key "SUBSYSTEM" does not match the symlink
> target, things will break horribly. So no device can be a member of
> class "cuse" but carry a SUBSYSTEM value of a different class.
>
> If that is how it works, I guess that must be solved differently, by
> hooking into the subsystem code and create "virtual devices" at the
> original class they fake, instead of their own "cuse" class. Possibly
> by making cuse a "bus", and use the cuse device as a parent for the
> "real" class device.

For OSS emulation, it didn't really matter.  For cases where it matters,
I think easier path to take would be let the userland emulation set up a
directory containing pseudo files and just override DEVPATH to it.
Would that work?

Thanks.

-- 
tejun

  reply	other threads:[~2008-08-29  8:04 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-28 16:30 [PATCH 1/2] uevent: don't pass envp_ext[] as format string in kobject_uevent_env() Tejun Heo
2008-08-28 16:31 ` [PATCH 2/2] uevent: handle duplicate uevent_var keys properly Tejun Heo
2008-08-28 16:49   ` Greg KH
2008-08-28 17:00     ` Tejun Heo
2008-08-28 17:33       ` Greg KH
2008-08-29  7:48       ` Kay Sievers
2008-08-29  8:02         ` Tejun Heo [this message]
2008-08-29 13:27           ` Kay Sievers
2008-08-29 13:34             ` Tejun Heo
2008-08-29 13:54               ` Kay Sievers
2008-08-29 15:00                 ` Tejun Heo

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=48B7AD11.2070906@kernel.org \
    --to=tj@kernel.org \
    --cc=greg@kroah.com \
    --cc=kay.sievers@vrfy.org \
    --cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox