| To: | Patrick McHardy <kaber@xxxxxxxxx> |
|---|---|
| Subject: | Re: path: module replay |
| From: | jamal <hadi@xxxxxxxxxx> |
| Date: | 19 Jan 2005 08:35:07 -0500 |
| Cc: | Thomas Graf <tgraf@xxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, netdev@xxxxxxxxxxx |
| In-reply-to: | <41EDE15B.40300@xxxxxxxxx> |
| Organization: | jamalopolous |
| References: | <1105900522.1090.798.camel@xxxxxxxxxxxxxxxx> <41EAB74F.6060507@xxxxxxxxx> <20050116185630.GZ26856@xxxxxxxxxxxxxx> <1105903033.1097.829.camel@xxxxxxxxxxxxxxxx> <1105903960.1097.836.camel@xxxxxxxxxxxxxxxx> <41EDE15B.40300@xxxxxxxxx> |
| Reply-to: | hadi@xxxxxxxxxx |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Tue, 2005-01-18 at 23:26, Patrick McHardy wrote:
> jamal wrote:
>
> >Ok, here is the last patch. I think module replay should be done from
> >that spot and to be consistent as well from cls_api.c for legacy stuff.
> >I also fixed a module ref count leak.
> >the act_api piece is dependent on what i sent earlier for namsiz.
> >the cls_api change i believe conflicts with what Thomas sent yesterday.
> >
> +replay:
> act = tcf_action_init_1(tb[i], est, name, ovr, bind, err);
> - if (act == NULL)
> - goto err;
> + if (act == NULL) {
> + if (*err == -EAGAIN)
> + goto replay;
> + goto err_out;
> + }
>
> This part is wrong.
And we exchanged emails on this topic already in private and you said
you will fix and send out the patch;-> So please do just that or you can
wait until i have more time and my hardware at the same spot.
cheers,
jamal
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH PKT_SCHED 4/4]: fix CONFIG_NET_CLS_ACT skb leaks in HFSC/CBQ, Patrick McHardy |
|---|---|
| Next by Date: | Re: [Spam:********* SpamScore] Re: excel document, Eric J. Korpela |
| Previous by Thread: | Re: path: module replay, Patrick McHardy |
| Next by Thread: | Re: path: module replay, Patrick McHardy |
| Indexes: | [Date] [Thread] [Top] [All Lists] |