Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 15 May 2011 11:09:10 -0400
From:      Marcel Moolenaar <marcel@xcllnt.net>
To:        "Andrey V. Elsukov" <ae@FreeBSD.org>
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r221952 - head/sbin/geom/class/part
Message-ID:  <7B2294F7-8383-42F2-8127-759DBCD3540A@xcllnt.net>
In-Reply-To: <201105151145.p4FBjDVR038539@svn.freebsd.org>
References:  <201105151145.p4FBjDVR038539@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

On May 15, 2011, at 7:45 AM, Andrey V. Elsukov wrote:

> Author: ae
> Date: Sun May 15 11:45:13 2011
> New Revision: 221952
> URL: http://svn.freebsd.org/changeset/base/221952
> 
> Log:
>  Simplify the code a bit. For own providers GEOM_PART always provides
>  "start" and "end" config attributes.
> 
>  MFC after:	1 week

Just to make sure: the code deals with backward compatibility.
If you're aware of the issue, but don't care (anymore) about it,
then the change is fine (as I can't quite remember the case in
point, and I don't care enough to speculatively page in the
details).

If you weren't aware, we should probably determine the impact
before you MFC this. Let me know and I'll dig up the details.

FYI,

-- 
Marcel Moolenaar
marcel@xcllnt.net





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?7B2294F7-8383-42F2-8127-759DBCD3540A>