From owner-svn-src-all@freebsd.org Fri Jan 5 18:03:54 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E0C0AEB7454 for ; Fri, 5 Jan 2018 18:03:54 +0000 (UTC) (envelope-from steven@multiplay.co.uk) Received: from mail-wm0-x22b.google.com (mail-wm0-x22b.google.com [IPv6:2a00:1450:400c:c09::22b]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 6D31B758F2 for ; Fri, 5 Jan 2018 18:03:54 +0000 (UTC) (envelope-from steven@multiplay.co.uk) Received: by mail-wm0-x22b.google.com with SMTP id b141so3945112wme.1 for ; Fri, 05 Jan 2018 10:03:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=multiplay-co-uk.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language; bh=aE3q1XdywvaG2+ckibxZh2Hm4Nccj2mSJkBndLT8qCI=; b=APqEvZqzUPNZyO1Cw0Khfe4cfdXQw0eovwHsdPsvkOtTDegGOlLoat8Fm7WQF5eUyt Tkwlh5sJ4Rt0Ex9BF5Lmmsswxt8ANcPTsoDUt0fI+rXRcvJKWHXI8ye7VgGxE+ZFhOf9 n6n/JDadNeTC5PvjYFtOwcI9A7CS7JBgmiAD7dIjS4ogsR/QN6TmCbMqiSZ1tAiGLhLt mCCB4rfp6jY364JYaD4/2WPRMNd7F9YVOcM/v1mvsk2OUPJ/UVyKE0C80Le+L+6Bc1Fn ygHg0cbqd4onDmq3TgZM31wpsEaTyz8r2I5boDlg/R9VDd/U+mRZuWUPCyq/2MwnO9i+ kUlg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language; bh=aE3q1XdywvaG2+ckibxZh2Hm4Nccj2mSJkBndLT8qCI=; b=rdu+l2cgTKfWsUVSLN4ZPjICAMke3YRSY1Lw2KAbIKzsk2Dy2pOhIQK/vn6U3yB1DJ 1q6750xkijWMEYiT9T3/4ZlNv+9cQsdf6kEtRDtw60YsVWaNPC0FR5PlHJ4Yh2ptgrzH uV49QmSG0VRd9QGaSK0m4rsPOLRJVY1T91H9n+cNKHn4dXrZ6onqPIPP7zmADLMAxD07 6l8OC9gpymjOgo/KJCRzCbzqh8G46aRXCpybK23HXdQwFIibbfUzenuDjxzRma3f6++V Ig0B0qcYwUQWOutMZbxlsKfSWhcS+KlAP22brDopOKg0KQUrWTqj5Vwdo9jfnv24hvIf joQQ== X-Gm-Message-State: AKGB3mJBDDGOACtE+vh9cGucy5OMjD1H5aXTmpHqHToEmd40QuZGuLoa ausdtguzFFvq1GCUK+gxEPR/qQ== X-Google-Smtp-Source: ACJfBouF7c/BroBKdOscenw4yrftRoDbDg8K2fu3LKzcM5z0bx0+z/bNZzX1xb24AiTHspAI0237xw== X-Received: by 10.80.225.195 with SMTP id m3mr5329762edl.222.1515175432983; Fri, 05 Jan 2018 10:03:52 -0800 (PST) Received: from [10.10.1.111] ([185.97.61.1]) by smtp.gmail.com with ESMTPSA id y1sm4250290edl.39.2018.01.05.10.03.51 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 05 Jan 2018 10:03:52 -0800 (PST) Subject: Re: svn commit: r327559 - in head: . sys/net To: Eugene Grosbein , hiren panchasara Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201801042005.w04K5liB049411@repo.freebsd.org> <5A4E9397.9000308@grosbein.net> <20180104224214.GD18879@strugglingcoder.info> <63c3c450-aeaf-bdd5-5e16-414146c9bb3a@multiplay.co.uk> <20180105094126.GE18879@strugglingcoder.info> <1854b421-3c09-c040-ba86-738b9c621eae@multiplay.co.uk> <5A4FB0B2.5070108@grosbein.net> From: Steven Hartland Message-ID: <4a3927f8-1ea8-36be-2fbd-e0ddc31563df@multiplay.co.uk> Date: Fri, 5 Jan 2018 18:03:53 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: <5A4FB0B2.5070108@grosbein.net> Content-Language: en-US Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 05 Jan 2018 18:03:55 -0000 On 05/01/2018 17:06, Eugene Grosbein wrote: > 05.01.2018 23:11, Steven Hartland wrote: > >> What do others think, am I missing something? > You still consider only TCP case missing IP forwarning case when all IP packets > are transit coming from lagg0 and going out via lagg1. Just going out via a laggX > > IP forwarding case benefits from pre-computed RSS flowid since 8.0-RELEASE > and your change breaks it. Is there a way to determine if the mbuf is a forwarded mbuf of not? I know I've said it before but just to be totally clear, changing the default was done to prevent broken behavior, if you're not concerned about the issue or you know you're not effected you can enable use_flowid to restore the original behavior. This doesn't have to be the final fix, if there are improvements that can be made to make the default more intelligent for example and use flowid if its known to be good then that can be looked into. In the mean time the new "default" will prevent others from configuring lagg(4) with LACP or loadbalance and ending up with problems; yes this may mean that IP forwarding in HEAD will use manual hashing hence will perform a little worse for now but that's the lesser of two evils.     Regards     Steve