LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: linux-kernel@vger.kernel.org,
	Linux Containers <containers@lists.osdl.org>,
	linux-fsdevel@vger.kernel.org, Andrew Morton <akpm@osdl.org>
Subject: Re: [PATCH 2/2] Implement shadow directory support for device classes.
Date: Wed, 24 Jan 2007 13:28:24 -0800	[thread overview]
Message-ID: <20070124212824.GA29981@suse.de> (raw)
In-Reply-To: <m1y7nsyxsp.fsf@ebiederm.dsl.xmission.com>

On Wed, Jan 24, 2007 at 01:28:22PM -0700, Eric W. Biederman wrote:
> Greg KH <gregkh@suse.de> writes:
> 
> > On Wed, Jan 24, 2007 at 12:37:19PM -0700, Eric W. Biederman wrote:
> >> 
> >> Modify the device class code so that normal manipulations work
> >> in the presence of shadow directories.  Some of the shadow directory
> >> support still needs to be implemented in the implementation of the
> >> class but these modifications are sufficient to make that simple.
> >
> > This is a nice idea, but I'm working to get rid of 'struct class_device'
> > from the tree.  Network devices have already been converted, the patch
> > is in the -mm tree, I'm working on some firewire fixes before pushing it
> > to Linus.
> >
> > So, I think you want to do this work for 'struct device' too, as that is
> > what will be the real user of this code.
> 
> Sure.  That shouldn't be too hard.
> 
> Is any of that work in Dave Millers' networking tree?

No, it's in my tree, and in -mm.  It's all in one patch at:
	http://www.kernel.org/pub/linux/kernel/people/gregkh/gregkh-2.6/gregkh-01-driver/network-device.patch
if you want to try it out.

Or you can try it with any of the subsystems that have already been
converted over to use the struct device code, like misc, mem, and many
others.  Just look for the symlinks in /sys/class/CLASS_NAME/ instead of
real subdirectories.

So I'd really prefer to not apply this patch, and just do this kind of
work for 'struct device' instead.

The first one looks fine, I'll add it to my queue.

thanks,

greg k-h

  reply	other threads:[~2007-01-24 21:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-24 19:35 [PATCH 1/2] sysfs: Shadow directory support Eric W. Biederman
2007-01-24 19:37 ` [PATCH 2/2] Implement shadow directory support for device classes Eric W. Biederman
2007-01-24 19:50   ` Greg KH
2007-01-24 20:28     ` Eric W. Biederman
2007-01-24 21:28       ` Greg KH [this message]
2007-01-25  2:43         ` Eric W. Biederman

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=20070124212824.GA29981@suse.de \
    --to=gregkh@suse.de \
    --cc=akpm@osdl.org \
    --cc=containers@lists.osdl.org \
    --cc=ebiederm@xmission.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: [PATCH 2/2] Implement shadow directory support for device classes.' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).