netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Hemminger <shemminger@linux-foundation.org>
To: Keiichi KII <k-keiichi@bx.jp.nec.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
	mpm@selenic.com, davem@davemloft.net,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: [RFC][PATCH -mm take4 2/6] support multiple logging
Date: Tue, 24 Apr 2007 08:21:17 -0700	[thread overview]
Message-ID: <20070424082117.354341d3@oldman> (raw)
In-Reply-To: <20070424012902.1089bc02.akpm@linux-foundation.org>

On Tue, 24 Apr 2007 01:29:02 -0700
Andrew Morton <akpm@linux-foundation.org> wrote:

> On Tue, 24 Apr 2007 17:14:28 +0900 Keiichi KII <k-keiichi@bx.jp.nec.com> wrote:
> 
> > > On Fri, 20 Apr 2007 18:51:13 +0900
> > > Keiichi KII <k-keiichi@bx.jp.nec.com> wrote:
> > > 
> > >>> I started to do some cleanups and fixups here, but abandoned it when it was
> > >>> all getting a bit large.
> > >>>
> > >>> Here are some fixes against this patch:
> > >> I'm going to fix my patches by following your reviews and send new patches 
> > >> on the LKML and the netdev ML in a few days.
> > >>
> > > 
> > > Well..  before you can finish this work we need to decide upon what the
> > > interface to userspace will be.
> > > 
> > > - The miscdev isn't appropriate
> > > 
> > 
> > Why isn't miscdev appropriate? 
> > We just shouldn't use miscdev for networking conventionally?
> > 
> 
> Yes it's rather odd, especially for networking.
> 
> What does the miscdev _do_ anyway?  Is it purely a target for the ioctls?

Some other speculations:
1. Would it be possible to add ioctl's to /dev/console? This would be more in
    keeping with older Unix style model.

2. Using sysfs makes sense if there is a device object that exists to
   add the sysfs attributes to.

3. Procfs is handy for summary type tables.

4. Netlink does feel like overkill for this. Although newer generic netlink
   makes it easier.

  reply	other threads:[~2007-04-24 15:21 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-18 11:49 [RFC][PATCH -mm take4 0/6] proposal for dynamic configurable netconsole Keiichi KII
2007-04-18 12:05 ` [RFC][PATCH -mm take4 1/6] marking __init Keiichi KII
2007-04-18 12:06 ` [RFC][PATCH -mm take4 2/6] support multiple logging Keiichi KII
2007-04-18 14:40   ` Stephen Hemminger
2007-04-20  9:58     ` Keiichi KII
2007-04-20  4:08   ` Andrew Morton
2007-04-20  9:51     ` Keiichi KII
2007-04-20 18:15       ` Andrew Morton
2007-04-20 18:21         ` Matt Mackall
2007-04-20 19:00           ` Stephen Hemminger
2007-04-24  8:14             ` Keiichi KII
2007-04-24  8:14         ` Keiichi KII
2007-04-24  8:29           ` Andrew Morton
2007-04-24 15:21             ` Stephen Hemminger [this message]
2007-04-26  4:02               ` Keiichi KII
2007-04-26  4:45                 ` David Miller
2007-04-18 12:08 ` [RFC][PATCH -mm take4 3/6] add interface for netconsole using sysfs Keiichi KII
2007-04-20  4:06   ` Andrew Morton
2007-04-18 12:11 ` [RFC][PATCH -mm take4 4/6] using symlink for the net_device Keiichi KII
2007-04-20  4:12   ` Andrew Morton
2007-04-18 12:12 ` [RFC][PATCH -mm take4 5/6] switch function of netpoll Keiichi KII
2007-04-18 12:14 ` [RFC][PATCH -mm take4 6/6] add ioctls for adding/removing target Keiichi KII
2007-04-20  4:16   ` Andrew Morton
2007-04-20  5:39     ` Stephen Hemminger
2007-04-20  6:59       ` David Miller

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=20070424082117.354341d3@oldman \
    --to=shemminger@linux-foundation.org \
    --cc=akpm@linux-foundation.org \
    --cc=davem@davemloft.net \
    --cc=k-keiichi@bx.jp.nec.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mpm@selenic.com \
    --cc=netdev@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;
as well as URLs for NNTP newsgroup(s).