From owner-freebsd-current@FreeBSD.ORG Sun Aug 3 08:02:11 2014 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id C1E96225 for ; Sun, 3 Aug 2014 08:02:11 +0000 (UTC) Received: from mail-qg0-x22a.google.com (mail-qg0-x22a.google.com [IPv6:2607:f8b0:400d:c04::22a]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 7FD4129E5 for ; Sun, 3 Aug 2014 08:02:11 +0000 (UTC) Received: by mail-qg0-f42.google.com with SMTP id j5so7662747qga.15 for ; Sun, 03 Aug 2014 01:02:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=lTSg1GEmdWZiWj4/RZ4LAZT5P03Rq5m90s9PE+688j0=; b=FYS/fjQAGvsgh/0mLVhkVJWzbPOfkSXrISDggNhwXVe94hlQJ6ZgjwT+/hRJqbZubR yGVkDRs8d5gSjhtaOYorLtl6iDJOjWSmXVLSW0/xR8BeKMB8XpHd7lQcUAhF3n76RPBJ q5kzaiRcMELYRaxqMDxsjLHvNr64jPLDotD9/+REGwrL1tfbVZcQKV3PAN792/0vnkJz 0VcAVMw/qqcEjhi+A6UpCWFBe5vEp66uV7Ds6os4MQ4BXTfJueNBFd+RSS0qDXN41mgb cjUWtqIttU2EnrNG6iBeaKKqbDU3T4+eDRqFHwl53Hk6Fq0JS2PkXZVraDx43g+m+htg utgA== MIME-Version: 1.0 X-Received: by 10.140.16.67 with SMTP id 61mr23905886qga.28.1407052930649; Sun, 03 Aug 2014 01:02:10 -0700 (PDT) Received: by 10.140.48.2 with HTTP; Sun, 3 Aug 2014 01:02:10 -0700 (PDT) In-Reply-To: <20140802200149.GO50495@funkthat.com> References: <20140801193403.GE50495@funkthat.com> <20140801222118.GH50495@funkthat.com> <20140802183259.GL50495@funkthat.com> <20140802200149.GO50495@funkthat.com> Date: Sun, 3 Aug 2014 10:02:10 +0200 Message-ID: Subject: Re: domain_add(xxx) after domainfinalize... From: Svatopluk Kraus To: John-Mark Gurney Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.18 Cc: freebsd-current@freebsd.org X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 03 Aug 2014 08:02:11 -0000 On Sat, Aug 2, 2014 at 10:01 PM, John-Mark Gurney wrote: > Svatopluk Kraus wrote this message on Sat, Aug 02, 2014 at 21:27 +0200: > > On Sat, Aug 2, 2014 at 8:32 PM, John-Mark Gurney > wrote: > > > > > Svatopluk Kraus wrote this message on Sat, Aug 02, 2014 at 15:20 +0200: > > > > Well, I did not look at network stack for long time, so the following > > > > things could be obsolete now, but at least: > > > > > > > > (1) There is some room left in mbuf header on its allocation, so > > > protocols > > > > (link) headers can be added in the front of data without need of > > > > reallocation or data copying. The size of the room is evaluated from > all > > > > domains at boot time. > > > > > > > > (2) All network address masks are shared among domains and protocols > in > > > > mask radix tree. The tree is created with particular key size, which > is > > > > again evaluated from all domains as maximal size of all known > addresses. > > > > > > > > So, if new domain added after these evaluations does not break both > them > > > > and some other things, there is no problem. Otherwise, some warnings > are > > > > always nice if things go bad. > > > > > > But would anyone who sees these issues know that this is the problem? > > > And what would they do, report the problem? We'd say, yeh, we know, > > > but do you hit this every day? Nope, ok, we won't fix it... > > > > > > > Maybe some function which examine new domain and say there is no > risk to > > > > add it would be nice. > > > > > > I agree... These things should have been done in the first place, but > > > clearly this warning wasn't enough to cause anyone to fix it.. :) > > > > You are right, but the warning is remainder that there is something > > unresolved. Remove the warning and forget, imho, it's not well. > > Sorry, nine years[1] of the warning existing yet no fix clearly > demonstrates that it's already been forgoten, otherwise it would have > been fixed by now.. > > Also, a bug report or XXX comment is enough IMO, and will probably > receive the same amount of attention over the next nine years... > > [1] https://svnweb.freebsd.org/changeset/base/138239 > > I agree that the warning is pointless stres for anyone but network domain developer. And such developer knows if he/she is careful enough to ignore it. However, I still remember time when I was starting with FreeBSD kernel development. So much things one should be aware of. In those time, any however small glue, which could help, was appreciated. So, put the warning under INVARIANTS or something like that as FreeBSD is open source and without rookies will die. And no one wants to make kernel development harder than it is. Svata > -- > John-Mark Gurney Voice: +1 415 225 5579 > > "All that I will do, has been done, All that I have, has not." >