All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: rpjday@mindspring.com
Cc: jengelh@linux01.gwdg.de, linux-kernel@vger.kernel.org
Subject: Re: stuff ready to be deleted?
Date: Tue, 29 May 2007 01:46:10 -0700 (PDT)	[thread overview]
Message-ID: <20070529.014610.23017882.davem@davemloft.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0705290419370.23055@localhost.localdomain>

From: "Robert P. J. Day" <rpjday@mindspring.com>
Date: Tue, 29 May 2007 04:20:24 -0400 (EDT)

> On Tue, 29 May 2007, Jan Engelhardt wrote:
> 
> >
> > On May 28 2007 18:25, Robert P. J. Day wrote:
> > >On Mon, 28 May 2007, Jan Engelhardt wrote:
> >
> > >> >currently, the list contains the items:
> > >> >
> > >> >    * 4 Traffic policing
> > >>
> > >> CONFIG_NET_SCHED? CONFIG_NET_ACT_POLICE? Neither seems
> > >> marked as deprecated/obsoleted in Kconfig.
> > >
> > >the "traffic policing" entry in that list refers specifically to the
> > >config option "CLS_NET_POLICE" in net/sched/Kconfig:
> > >
> > >config NET_CLS_POLICE
> > >        bool "Traffic Policing (obsolete)"
> > >        depends on NET_CLS_ACT!=y
> > >
> > >so ... does "obsolete" really mean "obsolete"?
> >
> > Have you taken a look at the Makefile? See this oddity:
> >
> > obj-$(CONFIG_NET_ACT_POLICE)    += act_police.o
> > obj-$(CONFIG_NET_CLS_POLICE)    += act_police.o
> >
> > So is the Kconfig entry all that is obsolete?
> 
> ah, i hadn't noticed that.  how odd.  now i don't know *what* to
> think.

Look at what tests that config option in the code, not just
what the makefiles do.

  reply	other threads:[~2007-05-29  8:46 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-28 20:15 stuff ready to be deleted? Robert P. J. Day
2007-05-28 20:24 ` Jesper Juhl
2007-05-28 21:14   ` John Anthony Kazos Jr.
2007-05-28 21:23     ` Jesper Juhl
2007-05-28 20:44 ` Oliver Pinter
2007-05-28 20:52   ` Robert P. J. Day
2007-05-28 23:49     ` Daniel Hazelton
2007-05-28 22:24   ` Bill Davidsen
2007-05-28 22:29     ` Jeff Garzik
2007-05-29  8:34     ` Alan Cox
2007-05-29  8:47       ` David Miller
2007-05-28 21:18 ` Jiri Slaby
2007-05-28 21:30 ` Jan Engelhardt
2007-05-28 22:25   ` Robert P. J. Day
2007-05-28 22:41     ` Robert P. J. Day
2007-05-29  6:09     ` Jan Engelhardt
2007-05-29  8:20       ` Robert P. J. Day
2007-05-29  8:46         ` David Miller [this message]
2007-05-29  7:56 ` Geert Uytterhoeven
2007-05-29 13:04   ` Finn Thain

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=20070529.014610.23017882.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=jengelh@linux01.gwdg.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rpjday@mindspring.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 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.