2013-08-20 17:35:01

by Joseph Salisbury

[permalink] [raw]
Subject: [PATCH][v3.2.y] inetpeer: Invalidate the inetpeer tree along with the routing cache

Hello,

Please consider including mainline commit 5faa5df in the next v3.2.y
release. It was included in the mainline tree as of v3.3-rc7. It has
been tested and confirmed to resolve
http://bugs.launchpad.net/bugs/1205741 .


commit 5faa5df1fa2024bd750089ff21dcc4191798263d
Author: Steffen Klassert <[email protected]>
Date: Tue Mar 6 21:20:26 2012 +0000

inetpeer: Invalidate the inetpeer tree along with the routing cache


Also note that commit 5faa5df introduced a race condition that is fixed
by mainline commit 55432d2, so that commit would also be required:

commit 55432d2b543a4b6dfae54f5c432a566877a85d90
Author: Eric Dumazet <[email protected]>
Date: Tue Jun 5 03:00:18 2012 +0000

inetpeer: fix a race in inetpeer_gc_worker()



Sincerely,
Joseph Salisbury


2013-08-31 22:04:32

by Ben Hutchings

[permalink] [raw]
Subject: Re: [PATCH][v3.2.y] inetpeer: Invalidate the inetpeer tree along with the routing cache

On Tue, 2013-08-20 at 13:34 -0400, Joseph Salisbury wrote:
> Hello,
>
> Please consider including mainline commit 5faa5df in the next v3.2.y
> release. It was included in the mainline tree as of v3.3-rc7. It has
> been tested and confirmed to resolve
> http://bugs.launchpad.net/bugs/1205741 .
>
>
> commit 5faa5df1fa2024bd750089ff21dcc4191798263d
> Author: Steffen Klassert <[email protected]>
> Date: Tue Mar 6 21:20:26 2012 +0000
>
> inetpeer: Invalidate the inetpeer tree along with the routing cache
>
>
> Also note that commit 5faa5df introduced a race condition that is fixed
> by mainline commit 55432d2, so that commit would also be required:
>
> commit 55432d2b543a4b6dfae54f5c432a566877a85d90
> Author: Eric Dumazet <[email protected]>
> Date: Tue Jun 5 03:00:18 2012 +0000
>
> inetpeer: fix a race in inetpeer_gc_worker()

David, could you ack/nak these for 3.2 (maybe 3.0 as well?).

Ben.

--
Ben Hutchings
If God had intended Man to program,
we'd have been born with serial I/O ports.


Attachments:
signature.asc (828.00 B)
This is a digitally signed message part

2013-09-10 18:49:28

by David Miller

[permalink] [raw]
Subject: Re: [PATCH][v3.2.y] inetpeer: Invalidate the inetpeer tree along with the routing cache

From: Joseph Salisbury <[email protected]>
Date: Tue, 20 Aug 2013 13:34:51 -0400

> Please consider including mainline commit 5faa5df in the next v3.2.y
> release. It was included in the mainline tree as of v3.3-rc7. It has
> been tested and confirmed to resolve
> http://bugs.launchpad.net/bugs/1205741 .
>
> commit 5faa5df1fa2024bd750089ff21dcc4191798263d
> Author: Steffen Klassert <[email protected]>
> Date: Tue Mar 6 21:20:26 2012 +0000
>
> inetpeer: Invalidate the inetpeer tree along with the routing cache
>
>
> Also note that commit 5faa5df introduced a race condition that is fixed
> by mainline commit 55432d2, so that commit would also be required:
>
> commit 55432d2b543a4b6dfae54f5c432a566877a85d90
> Author: Eric Dumazet <[email protected]>
> Date: Tue Jun 5 03:00:18 2012 +0000
>
> inetpeer: fix a race in inetpeer_gc_worker()

Queued up, thanks.

2013-09-10 18:53:43

by David Miller

[permalink] [raw]
Subject: Re: [PATCH][v3.2.y] inetpeer: Invalidate the inetpeer tree along with the routing cache

From: Ben Hutchings <[email protected]>
Date: Sat, 31 Aug 2013 23:04:18 +0100

> On Tue, 2013-08-20 at 13:34 -0400, Joseph Salisbury wrote:
>> Hello,
>>
>> Please consider including mainline commit 5faa5df in the next v3.2.y
>> release. It was included in the mainline tree as of v3.3-rc7. It has
>> been tested and confirmed to resolve
>> http://bugs.launchpad.net/bugs/1205741 .
>>
>>
>> commit 5faa5df1fa2024bd750089ff21dcc4191798263d
>> Author: Steffen Klassert <[email protected]>
>> Date: Tue Mar 6 21:20:26 2012 +0000
>>
>> inetpeer: Invalidate the inetpeer tree along with the routing cache
>>
>>
>> Also note that commit 5faa5df introduced a race condition that is fixed
>> by mainline commit 55432d2, so that commit would also be required:
>>
>> commit 55432d2b543a4b6dfae54f5c432a566877a85d90
>> Author: Eric Dumazet <[email protected]>
>> Date: Tue Jun 5 03:00:18 2012 +0000
>>
>> inetpeer: fix a race in inetpeer_gc_worker()
>
> David, could you ack/nak these for 3.2 (maybe 3.0 as well?).

I'm queueing these two patches up for 3.2 only, the
inetpeer/metrics-cache situation in 3.0 has so many deeper problems
than can be reasonably dealt with using backports. Just look at the
layout differences of struct inetpeer between 3.0 and 3.2, and the
reasons for that, for the tip of the iceberg.