From owner-svn-src-head@freebsd.org Mon Sep 25 21:48:44 2017 Return-Path: Delivered-To: svn-src-head@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 D9848E24FDC for ; Mon, 25 Sep 2017 21:48:44 +0000 (UTC) (envelope-from shurd@freebsd.org) Received: from fed1rmfepi101.cox.net (fed1rmfepi101.cox.net [68.230.241.132]) by mx1.freebsd.org (Postfix) with ESMTP id B57F76ADFA for ; Mon, 25 Sep 2017 21:48:44 +0000 (UTC) (envelope-from shurd@freebsd.org) Received: from fed1rmimpo306.cox.net ([68.230.241.174]) by fed1rmfepo203.cox.net (InterMail vM.8.01.05.28 201-2260-151-171-20160122) with ESMTP id <20170925214643.BFFY30763.fed1rmfepo203.cox.net@fed1rmimpo306.cox.net> for ; Mon, 25 Sep 2017 17:46:43 -0400 Received: from [192.168.0.33] ([72.194.73.141]) by fed1rmimpo306.cox.net with cox id Dxmd1w00Y32uAN001xmen4; Mon, 25 Sep 2017 17:46:39 -0400 X-CT-Class: Clean X-CT-Score: 0.00 X-CT-RefID: str=0001.0A090206.59C9793F.004D, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0 X-CT-Spam: 0 X-Authority-Analysis: v=2.2 cv=LdtM0nXi c=1 sm=1 tr=0 a=l4Y+EJuLrT/8f1z5FvEQ1g==:117 a=l4Y+EJuLrT/8f1z5FvEQ1g==:17 a=IkcTkHD0fZMA:10 a=6I5d2MoRAAAA:8 a=CPxIPFaeP-TbcODUjsoA:9 a=QEXdDO2ut3YA:10 a=IjZwj45LgO3ly-622nXo:22 X-CM-Score: 0.00 Authentication-Results: cox.net; auth=pass (PLAIN) smtp.auth=myhuge@cox.net Subject: Re: svn commit: r323944 - in head/sys: kern sys To: Gleb Smirnoff Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org References: <201709230139.v8N1dGCi063544@repo.freebsd.org> <20170925212543.GX1055@FreeBSD.org> From: Stephen Hurd Message-ID: <929758eb-c4b6-2d0f-9eb3-f2f15b545e58@FreeBSD.org> Date: Mon, 25 Sep 2017 14:46:37 -0700 User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:51.0) Gecko/20100101 Firefox/51.0 SeaMonkey/2.48 MIME-Version: 1.0 In-Reply-To: <20170925212543.GX1055@FreeBSD.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Sep 2017 21:48:45 -0000 Gleb Smirnoff wrote: > Stephen, > > On Sat, Sep 23, 2017 at 01:39:16AM +0000, Stephen Hurd wrote: > S> Author: shurd > S> Date: Sat Sep 23 01:39:16 2017 > S> New Revision: 323944 > S> URL: https://svnweb.freebsd.org/changeset/base/323944 > S> > S> Log: > S> Make struct grouptask gt_name member a char array > S> > S> Previously, it was just a pointer which was copied, but > S> some callers pass in a stack variable which will go out of scope. > S> Add GROUPTASK_NAMELEN macro (32) and snprintf() the name into it, > S> using "grouptask" if name is NULL. We can now safely include > S> gtask->gt_name in console messages. > > Have you estimated which structures include struct grouptask, so > that they now got larger? I did take a look through and didn't see anything alarming in either iflib or the linuxkpi stuff which seem to be the only consumers at this time. > Is this done only for the sake of console messages? I'm not sure yet. The member was already there and had pointers to the stack in it. Nothing appeared to be consuming it, but I noticed the error while working on console messages, so that's all it's used for now. I need to finish digging through the rest of the iflib backlog to see if there's another use for it. The safest change seemed to be making it part of the structure and reviewing the usage later.