From owner-svn-src-all@freebsd.org Thu Aug 9 23:40:57 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 162CE1076FAB for ; Thu, 9 Aug 2018 23:40:57 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: from mail-io0-x22c.google.com (mail-io0-x22c.google.com [IPv6:2607:f8b0:4001:c06::22c]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 9C3F78D7F4 for ; Thu, 9 Aug 2018 23:40:56 +0000 (UTC) (envelope-from wlosh@bsdimp.com) Received: by mail-io0-x22c.google.com with SMTP id l7-v6so6258294ioj.1 for ; Thu, 09 Aug 2018 16:40:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bsdimp-com.20150623.gappssmtp.com; s=20150623; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=OzeLmNzbnQEmcYPW2qgN4jsi4hBfUJhRO9az51lZBLc=; b=ByN1XM8DVcMh6L8ScnK3ke5l/Swa/G0NENvQvn7YsRI1maJ4XpeW/mxDzKAm9IJOMN VEMxhVzPTsb2Yff8JQY2s2rO04Ke8WX3JpoSGVmYLPta/yEkSU1AyFcu6XetpCvGQ1z8 JtaSm6+08aSeGHZ4ZhYCC+8zVUlHfC7OF0M+FZ+tykTiHItMACdoB7uiphR9HpjJbGfC 7GKt0JmO77yFB/PDlhHHGtMvi3JIitmixPcQYS+GZD+at6sgMHPwN3z/SDETcI21YJ/J 1R6Jx/xl25B12xUUIKg8//f2Vs82QMeAT4cvgRMFgzDoPr/xcCWdFeVXHTaXoStu8adW sEqg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=OzeLmNzbnQEmcYPW2qgN4jsi4hBfUJhRO9az51lZBLc=; b=cu3Y9DCCQG+MoSKiGqNxFeIw0TJSVL3nFkng5LqzYZrwxW+eF+22VcGTdb4D8xcUVm YdFvjHxASLc4IPqwC+8IxALGY5TBEyXtAr7fSr3mHrLfepVjBHy9ihBd1lO1u2KlgQ/d Mu/sx23v9IBXQ9S/75ooxvkdTHozkpthZiC5Ch4CwUqb3LAzAOHs4y1zMm9SLpFOVgvP 0GdQnyWTafRdZj+LOZFAWz9xt98gQd51zi1mc4sCCHd8hYm2Wtf5dCZOxhR5tJOJF97K 6qkBvP/e8iqMTOlyCjjCh4cwwdrt3jc6Idi5D6ZQMm3qZ7GWIGbCD50hmXptELxghPI9 MdXA== X-Gm-Message-State: AOUpUlECEZYZcAl1jDRfhleAM3O90yJ4zRQmJ4afu7M6dyUVYIxb59dW lCEuElr1+Z2chTx2BR7FMNhTDYtxDYN7XbL1XGo7NA== X-Google-Smtp-Source: AA+uWPysgW0z64rJSEUoeopjcNdsZToG1m9uOtbcZcYsc5q+vPAO364yPGv7VunTFU2M9jZbaqXnQ/ty0ytP3vYMSiY= X-Received: by 2002:a6b:3902:: with SMTP id g2-v6mr3526579ioa.168.1533858055125; Thu, 09 Aug 2018 16:40:55 -0700 (PDT) MIME-Version: 1.0 Sender: wlosh@bsdimp.com Received: by 2002:a4f:381a:0:0:0:0:0 with HTTP; Thu, 9 Aug 2018 16:40:54 -0700 (PDT) X-Originating-IP: [2603:300b:6:5100:1052:acc7:f9de:2b6d] In-Reply-To: References: <201808092153.w79LrWEZ097986@repo.freebsd.org> From: Warner Losh Date: Thu, 9 Aug 2018 17:40:54 -0600 X-Google-Sender-Auth: 3zCQiINs_CFf6Pyp-pnx-xNo8lA Message-ID: Subject: Re: svn commit: r337554 - in head/sys/cam: . ata nvme scsi To: Scott Long Cc: Conrad Meyer , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.27 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.27 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: Thu, 09 Aug 2018 23:40:57 -0000 On Thu, Aug 9, 2018 at 3:56 PM, Scott Long wrote: > > > > On Aug 9, 2018, at 3:53 PM, Conrad Meyer wrote: > > > > Author: cem > > Date: Thu Aug 9 21:53:32 2018 > > New Revision: 337554 > > URL: https://svnweb.freebsd.org/changeset/base/337554 > > > > Log: > > cam(4): Add an xpt-neutral flag indicating a valid panic CCB > > > > No functional change. > > > > Note that this change is careful to set the CCB header xflags after > > foo_fill_bar() routines, which generally zero existing flags. An > earlier > > version of this patch mistakenly set the flag before the fill routines= . > > > > Submitted by: Scott Ferris , jhibbits@ > > Reviewed by: bdrewery@, markj@, and non-committer FreeBSD contributor > Anton Rang > > Sponsored by: Dell EMC Isilon > > What=E2=80=99s the purpose of this flag? I see it gets set in plenty of = places, > but never checked. > It kinda looks like something I recall removing when I did the runccb fixes so they worked when we were dumping. We used to need this, but we don't anymore. I have the same question: Why? Maybe these CAM changes should go through phabricator? Warner