All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: hadi@cyberus.ca
Cc: netdev <netdev@oss.sgi.com>, "David S. Miller" <davem@davemloft.net>
Subject: Re: patch: Action repeat
Date: Sat, 30 Apr 2005 20:13:11 +0200	[thread overview]
Message-ID: <4273CAB7.6080403@trash.net> (raw)
In-Reply-To: <1114882045.8929.123.camel@localhost.localdomain>

jamal wrote:
> Actions can set it. 
> Theres one in particular that i havent submitted yet that sets it
> amongst other items (depending on policy). We discussed this a while
> back when talking about ematches etc.

Since it only has such a short lifetime (action function sets it,
tcf_action_exec() clears it after changing classification result),
it would be less wasteful to just pass the classification result
to the actions. This would also avoid that skbs with tc_classid
already set can reach tcf_action_exec() (for example through mirred).

What do you think?

Regards
Patrick

  reply	other threads:[~2005-04-30 18:13 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-04-30 16:50 patch: Action repeat jamal
2005-04-30 17:06 ` Patrick McHardy
2005-04-30 17:08   ` Patrick McHardy
2005-04-30 17:27     ` jamal
2005-04-30 18:13       ` Patrick McHardy [this message]
2005-04-30 19:51         ` jamal
2005-04-30 20:08           ` Thomas Graf
2005-04-30 20:50             ` jamal
2005-04-30 21:55               ` Thomas Graf
2005-04-30 22:34                 ` jamal
2005-04-30 23:58                   ` Thomas Graf
2005-05-02 12:10                     ` jamal
2005-05-02 15:06                       ` Thomas Graf
2005-05-04 11:46                         ` jamal
2005-05-04 12:15                           ` Patrick McHardy
2005-05-04 12:31                           ` Thomas Graf
2005-05-04 12:59                             ` jamal
2005-05-04 13:28                               ` Thomas Graf
2005-05-04 13:33                                 ` Thomas Graf
2005-05-04 13:33                                 ` jamal
2005-05-04 13:48                                   ` Thomas Graf
2005-05-04 13:53                                     ` jamal
2005-05-04 14:05                                       ` Thomas Graf
2005-05-04 14:23                                         ` jamal
2005-05-04 14:53                                           ` Thomas Graf
2005-05-05 13:06                                             ` jamal
2005-05-01  0:08             ` Patrick McHardy
2005-05-01  0:06           ` Patrick McHardy
2005-05-03 23:28 ` David S. 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=4273CAB7.6080403@trash.net \
    --to=kaber@trash.net \
    --cc=davem@davemloft.net \
    --cc=hadi@cyberus.ca \
    --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 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.