Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 25 Nov 2013 14:40:49 +0200
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Marcel Moolenaar <marcel@FreeBSD.org>, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org
Subject:   Re: svn commit: r258448 - head/sys/geom/part
Message-ID:  <52934551.7050608@FreeBSD.org>
In-Reply-To: <201311212202.rALM2xAS078472@svn.freebsd.org>
References:  <201311212202.rALM2xAS078472@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
on 22/11/2013 00:02 Marcel Moolenaar said the following:
> Author: marcel
> Date: Thu Nov 21 22:02:59 2013
> New Revision: 258448
> URL: http://svnweb.freebsd.org/changeset/base/258448
> 
> Log:
>   Have the GPT probe return a lower priority when the MBR is not a PMBR
>   The purpose of the PMBR is to have the disk appear in use to GPT
>   unaware utilities (like fdisk).  However, if the PMBR has been changed
>   by a GPT unaware utlity then we must assume that this was deliberate
>   (as it involved removal of the special slice) and we should not treat
>   the unmodified GPT-specific sectors as being valid.  By lowering the
>   probe priority in that case, the MBR scheme will take precedence and
>   the kernel will end up using the MBR and not the GPT. We will still
>   use the GPT if the kernel does not support the MBR scheme.

Thank you very much for this change!
Finally I can get rid of my local patch from a while ago:
http://thread.gmane.org/gmane.os.freebsd.devel.geom/3776/focus=3802

Your solution is clearly superior while providing the same end result.

-- 
Andriy Gapon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?52934551.7050608>