svn commit: r337554 - in head/sys/cam: . ata nvme scsi

[Conrad Meyer]

I had two functional changes in mind for after this part (which I hoped
would be non controversial) that you would be included on the review.
Markj pointed me at the related D15592.  I’m not sure that approach was the
correct one and hope to continue discussion there.  Once that’s reached
consensus, I may need to revisit my subsequent change.  If it ends up that
ultimately this particular patch isn’t used, backing it out is trivial.
Until then, it’s pretty harmless.

Thanks,
Conrad

On Thu, Aug 9, 2018 at 2:56 PM Scott Long <scottl at samsco.org> wrote:

>
>
> > On Aug 9, 2018, at 3:53 PM, Conrad Meyer <cem at FreeBSD.org> 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 <sferris AT isilon.com>, jhibbits@
> >  Reviewed by: bdrewery@, markj@, and non-committer FreeBSD contributor
> Anton Rang
> >  Sponsored by:        Dell EMC Isilon
>
> What’s the purpose of this flag?  I see it gets set in plenty of places,
> but never checked.
>
> Scott
>
>