svn commit: r270149 - head/sys/geom

[Sergey Kandaurov]

On 19 August 2014 00:05, Alexander Kabaev <kabaev at gmail.com> wrote:
> On Mon, 18 Aug 2014 23:51:32 +0400
> Sergey Kandaurov <pluknet at freebsd.org> wrote:
>
>> On 18 August 2014 23:27, Scott Long <scottl at freebsd.org> wrote:
>> > Author: scottl
>> > Date: Mon Aug 18 19:27:47 2014
>> > New Revision: 270149
>> > URL: http://svnweb.freebsd.org/changeset/base/270149
>> >
>> > Log:
>> >   Deal explicitly with possible failures of make_dev_alias_p() in
>> > GEOM.
>> >
>> >   Submitted by:   Mariusz Zaborski <oshogbo at FreeBSD.org>
>> >   MFC after:      3 days
>> >
>> > Modified:
>> >   head/sys/geom/geom_dev.c
>> >
>> > Modified: head/sys/geom/geom_dev.c
>> > ==============================================================================
>> > --- head/sys/geom/geom_dev.c    Mon Aug 18 18:07:28 2014
>> > (r270148) +++ head/sys/geom/geom_dev.c    Mon Aug 18 19:27:47
>> > 2014        (r270149)
>> [...]
>> > @@ -263,6 +264,7 @@ g_dev_taste(struct g_class *mp, struct g
>> >         if (adev != NULL) {
>> >                 adev->si_iosize_max = MAXPHYS;
>> >                 adev->si_drv2 = cp;
>> > +               adev->si_flags |= SI_UNMAPPED;
>> >         }
>> >
>> >         g_dev_attrchanged(cp, "GEOM::physpath");
>> >
>>
>> This part looks unrelated.
>>
>
> Commit says that it deals with consequences of make_dev_alias_p
> failing. Preventing trap happening on said failure seems to be
> pretty much related to the stated goal.

I missed setting SI_UNMAPPED was already there, nevermind.

-- 
wbr,
pluknet