Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1720510 > unrolled thread

Re: [PATCH 3.16 119/134] ipv4: restore rt->fi for reference counting

Started byBen Hutchings <ben@decadent.org.uk>
First post2017-08-26 02:40 +0200
Last post2017-08-26 02:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 3.16 119/134] ipv4: restore rt->fi for reference counting Ben Hutchings <ben@decadent.org.uk> - 2017-08-26 02:40 +0200

#1720510 — Re: [PATCH 3.16 119/134] ipv4: restore rt->fi for reference counting

FromBen Hutchings <ben@decadent.org.uk>
Date2017-08-26 02:40 +0200
SubjectRe: [PATCH 3.16 119/134] ipv4: restore rt->fi for reference counting
Message-ID<uixiO-2At-5@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

On Fri, 2017-08-18 at 06:53 -0700, Eric Dumazet wrote:
> > On Fri, Aug 18, 2017 at 6:13 AM, Ben Hutchings <ben@decadent.org.uk> wrote:
> > 3.16.47-rc1 review patch.  If anyone has any objections, please let me know.
> > 
> > ------------------
> > 
> > From: WANG Cong <xiyou.wangcong@gmail.com>
> > 
> > commit 82486aa6f1b9bc8145e6d0fa2bc0b44307f3b875 upstream.
[...]
> I thought we refined this later with :

In fact it was reverted upstream, then replaced by the commits you
point out.  Thanks; I don't know why I missed that.

> commit 3fb07daff8e99243366a081e5129560734de4ada
> Author: Eric Dumazet <edumazet@google.com>
> Date:   Thu May 25 14:27:35 2017 -0700
> 
>     ipv4: add reference counting to metrics
[...]

It's not obvious how to backport this, as it adds:

+#define DST_METRICS_REFCOUNTED		0x2UL

but 3.16 already has a flag with value 2.  I don't think we can take
another bit from the pointer as metrics are only 32-bit aligned.

Ben.

-- 
Ben Hutchings
One of the nice things about standards is that there are so many of
them.

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web