[Cake] allocate_src allocate_dst

Dave Taht dave.taht at gmail.com
Sat Nov 25 11:48:39 EST 2017


On Sat, Nov 25, 2017 at 8:28 AM, Pete Heist <peteheist at gmail.com> wrote:
>
>> On Nov 25, 2017, at 3:08 PM, Kevin Darbyshire-Bryant <kevin at darbyshire-bryant.me.uk> wrote:
>>
>>> On 25 Nov 2017, at 03:59, Dave Taht <dave.taht at gmail.com> wrote:
>>>
>>> there is no place in the current code base where these are not both
>>> true or both false. Thus redundant.
>>>
>>> ?
>>
>> Yes, interesting that.  Been there since introduced in 5a6da2ba Add triple-isolation support.  *EXPERIMENTAL*
>>
>> Have done a PR https://github.com/dtaht/sch_cake/pull/72 with a minor cleanup for that - been running on my router with no explosions :-)
>
> Woah, nice find. I’d been working through that function (looking at setting srchost_hash and dsthost_hash to a packet mark value for custom isolation) and between the gotos and ifdefs for kernel version support, it takes some work to look at it.
>
> Could this function be refactored a bit for readability? I’d offer to do it, but this is an important piece of code with little room for error and more ways to plant hard to find problems, better played with by someone who’s more familiar with it...

I am unfond of the gotos as well. But I'm gonna go do laundry. :) If
someone else beats me to it?



-- 

Dave Täht
CEO, TekLibre, LLC
http://www.teklibre.com
Tel: 1-669-226-2619


More information about the Cake mailing list