From owner-svn-src-all@freebsd.org Mon Apr 9 15:19:05 2018 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id DA1B0F88232; Mon, 9 Apr 2018 15:19:04 +0000 (UTC) (envelope-from byond.lenox@gmail.com) Received: from mail-lf0-f46.google.com (mail-lf0-f46.google.com [209.85.215.46]) (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 478CD79370; Mon, 9 Apr 2018 15:19:04 +0000 (UTC) (envelope-from byond.lenox@gmail.com) Received: by mail-lf0-f46.google.com with SMTP id v207-v6so7130537lfa.10; Mon, 09 Apr 2018 08:19:04 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=30g27/wpeTpHfBYaLd2HMQVMSQ7oQOvmsWp5vEUCArg=; b=FU8OWa/7PdQK9StUIAt6J0IfIfdTp3uj+bzBBMOsSbpGG5AWdjvVygLNgss+unZMo+ 9AKKO8SfV0un9AMRuEO4mb0mMeycgBlrjwRXZ+dwPpuAL1Ye4TBpqWM5QaRYQKqIcFF7 rjqBAkTYwubmhR/u/qg8B/o3kIL3hQLJIHD26yN6H+3yQdS3y9G3RNtICio03OJJsYQr VcnNghW3KktXJizawRgSPJs25Yq11YoVuDAvSuFPCwpCCECiYumXMm3jiBw6tVZRUNR2 xMpx9qCtvqPK7FZguK29ZQyZl6NoH7/ipKxKc/30SmC8hYUHeBnyP117mED+nRr2Bl+y nevQ== X-Gm-Message-State: ALQs6tCEbuHwN+PGQ3Q7FIsnYnCHaRcx76e/HQ8IuQhGRMYxp6WIpRF9 Uo5QIoblDkAfkuk5X1hrmGD7RE7K X-Google-Smtp-Source: AIpwx48O8KFmZQMagSk83kJM8DVi6MnYfYFBY0ZwuprpOqlygOmOPw7oOm8yntXWCKzEcxW13W6ZeA== X-Received: by 10.46.135.134 with SMTP id n6mr23334599lji.52.1523287136763; Mon, 09 Apr 2018 08:18:56 -0700 (PDT) Received: from mail-lf0-f52.google.com (mail-lf0-f52.google.com. [209.85.215.52]) by smtp.gmail.com with ESMTPSA id m127-v6sm118096lfm.79.2018.04.09.08.18.56 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 09 Apr 2018 08:18:56 -0700 (PDT) Received: by mail-lf0-f52.google.com with SMTP id m200-v6so7135570lfm.4; Mon, 09 Apr 2018 08:18:56 -0700 (PDT) X-Received: by 10.46.91.21 with SMTP id p21mr23629606ljb.38.1523287136434; Mon, 09 Apr 2018 08:18:56 -0700 (PDT) MIME-Version: 1.0 Received: by 10.46.129.90 with HTTP; Mon, 9 Apr 2018 08:18:35 -0700 (PDT) In-Reply-To: <201804091514.w39FEPRv019139@pdx.rh.CN85.dnsmgr.net> References: <201804091514.w39FEPRv019139@pdx.rh.CN85.dnsmgr.net> From: Kyle Evans Date: Mon, 9 Apr 2018 10:18:35 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: svn commit: r331880 - stable/11/etc To: "Rodney W. Grimes" Cc: Kyle Evans , Niclas Zeising , svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers , svn-src-stable-11@freebsd.org Content-Type: text/plain; charset="UTF-8" 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: Mon, 09 Apr 2018 15:19:05 -0000 On Mon, Apr 9, 2018 at 10:14 AM, Rodney W. Grimes wrote: > [ Charset UTF-8 unsupported, converting... ] >> On Mon, Apr 9, 2018 at 9:46 AM, Rodney W. Grimes >> wrote: >> >> On 04/02/18 17:39, Rodney W. Grimes wrote: >> >> >> Author: kevans >> >> >> Date: Mon Apr 2 15:28:48 2018 >> >> >> New Revision: 331880 >> >> >> URL: https://svnweb.freebsd.org/changeset/base/331880 >> >> >> >> >> >> Log: >> >> >> MFC r328331: Support configuring arbitrary limits(1) for any rc.conf daemon >> >> >> >> >> >> Usage is ${name}_limits, and the argument is any flags accepted by >> >> >> limits(1), such as `-n 100' (e.g. only allow 100 open files). >> >> >> >> >> >> Modified: >> >> >> stable/11/etc/rc.subr >> >> >> Directory Properties: >> >> >> stable/11/ (props changed) >> >> >> >> >> >> Modified: stable/11/etc/rc.subr >> >> >> ============================================================================== >> >> >> --- stable/11/etc/rc.subr Mon Apr 2 15:07:41 2018 (r331879) >> >> >> +++ stable/11/etc/rc.subr Mon Apr 2 15:28:48 2018 (r331880) >> >> >> @@ -773,6 +773,8 @@ check_startmsgs() >> >> >> # >> >> >> # ${name}_login_class n Login class to use, else "daemon". >> >> >> # >> >> >> +# ${name}_limits n limits(1) to apply to ${command}. >> >> >> +# >> >> > >> >> > Caution, limits(1) is in /usr/bin, this code can fail if used before >> >> > /usr is mounted. (Ie, our rc.initdiskless) is probably broken by >> >> > this change if a call is made to limits. >> >> > >> >> > >> >> >> >> Sorry for jumping on this so late. This is also an issue in CURRENT, >> >> and has been since at least 2016. >> > >> > I was aware that it was an issue and why I made a comment about it >> > being MFC'ed. Though I had forgot a bug report existed. >> >> I'm kind of surprised we haven't had more complaints about this- the >> original commit for this stuff landed before stable/11 was even >> branched, so it's been broken for all of 11.x's lifetime. > > History has taught me it takes a long time for this type of > breakage to usually surface in a noticable way. Also I think > until you merged this last ${name}_limits thing it actually > didn't cause an issue, except for the few like me running > diskless systems and or seperate /usr. I don't see how this merge could possibly have been the cause of any claimed issues- like I said before, it didn't add any limits invocations, it added an arg to the limits invocation that already existed. You can see this pretty clearly from the diff, we didn't even change any conditions for limits to be invoked. > > This latest issue is a name space collision between base and ports. > > People who see limits issues due to missing /usr files usually > know how to work around it, and they do, they just cp /usr/bin/limits > to /bin, and do not submit a bug report or send an email. > That saddens me. =/