From: Evgeniy Polyakov <johnpol@2ka.mipt.ru>
To: "David S. Miller" <davem@davemloft.net>
Cc: akpm@osdl.org, herbert@gondor.apana.org.au, netdev@oss.sgi.com
Subject: Re: 2.6.12-rc4-mm[12] - ULOG problem
Date: Fri, 20 May 2005 10:39:38 +0400 [thread overview]
Message-ID: <1116571178.21310.124.camel@uganda> (raw)
In-Reply-To: <20050519.114425.18307286.davem@davemloft.net>
[-- Attachment #1: Type: text/plain, Size: 1992 bytes --]
On Thu, 2005-05-19 at 11:44 -0700, David S. Miller wrote:
> From: Andrew Morton <akpm@osdl.org>
> Date: Wed, 18 May 2005 23:45:52 -0700
>
> > > Should I ask David Miller to allocate new one specially for connector?
> >
> > Dave?
>
> Giving the connector a single value 10 is OK, the rest of
> what the connector is doing in this area is not.
>
> I wish you hadn't put the connector into -mm, it's really
> in need of several fixups still. The bogus reuse of the
> ULOG netlink number is simply indicative of that.
And how does it indicate that it is bogus? :)
It is one netlink user that can use any socket number - and now
it is called bogus.
What other people suggest is just create some wrapper over
netlink_broadcast() and alloc_skb().
if you do think it will solve all problems, then I even will not
say anything. Point.
I just want to note, that if you want message bus, you require
at least oppsite direction, and thus input callback, and
either several socket number for each user [kobject, audit, iscsi,
xfrm - all they implement what connector already does] or
some header and thus parser in input callback, and thus
some registration mechanism. Magically it is what connector does.
And nothing more, btw.
It only allows not to rewrite the whole driver just to move
to new transport layer, only register new cn_dev device
with different method.
Ok, Dave, we all see your point, feel free to remove connector,
I will not bother people to include it yet another time...
But I believe in a month or two someone will reinvent the wheel
and create somthing similar. If it will be simple wrapper, then
it just can not solve message bus problem, only notification,
and thus someone will create it again.
Thank you.
P.S. smtp server from my old university years seems to still drop
your messages, sorry for that, I will kick admins again.
--
Evgeniy Polyakov
Crash is better than data corruption -- Arthur Grabowski
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]
next prev parent reply other threads:[~2005-05-20 6:39 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-19 3:24 Fw: 2.6.12-rc4-mm[12] - ULOG problem Andrew Morton
2005-05-19 4:49 ` Herbert Xu
2005-05-19 5:27 ` Andrew Morton
2005-05-19 6:31 ` Evgeniy Polyakov
2005-05-19 6:45 ` Andrew Morton
2005-05-19 7:06 ` Evgeniy Polyakov
2005-05-19 7:08 ` Andrew Morton
2005-05-19 7:27 ` Evgeniy Polyakov
2005-05-19 13:19 ` James Morris
2005-05-19 16:53 ` Andrew Morton
2005-05-19 17:09 ` James Morris
2005-05-19 18:54 ` David S. Miller
2005-05-19 18:48 ` David S. Miller
2005-05-20 6:27 ` Evgeniy Polyakov
2005-05-19 18:42 ` David S. Miller
2005-05-20 6:27 ` Evgeniy Polyakov
2005-05-19 18:44 ` David S. Miller
2005-05-20 6:39 ` Evgeniy Polyakov [this message]
2005-05-20 11:42 ` jamal
2005-05-20 14:15 ` Evgeniy Polyakov
2005-05-19 13:16 ` Fw: " James Morris
2005-05-19 13:46 ` jamal
2005-05-19 18:49 ` David S. Miller
2005-05-20 6:25 ` Evgeniy Polyakov
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=1116571178.21310.124.camel@uganda \
--to=johnpol@2ka.mipt.ru \
--cc=akpm@osdl.org \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=netdev@oss.sgi.com \
/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).