From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53388) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UjpeF-0005Q9-7R for qemu-devel@nongnu.org; Tue, 04 Jun 2013 07:43:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UjpeD-0007Kq-Su for qemu-devel@nongnu.org; Tue, 04 Jun 2013 07:43:31 -0400 Message-ID: <51ADD2DD.3090909@suse.de> Date: Tue, 04 Jun 2013 13:43:25 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <54b43e8b5fe6acff6fe667062d412c0b224f92f6.1370237352.git.peter.crosthwaite@xilinx.com> In-Reply-To: <54b43e8b5fe6acff6fe667062d412c0b224f92f6.1370237352.git.peter.crosthwaite@xilinx.com> Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v1 2/3] memory: as_update_topology_pass: Improve comments List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: peter.crosthwaite@xilinx.com, pbonzini@redhat.com Cc: qemu-trivial@nongnu.org, edgar.iglesias@gmail.com, qemu-devel@nongnu.org Am 03.06.2013 07:32, schrieb peter.crosthwaite@xilinx.com: > From: Peter Crosthwaite >=20 > These comments we're a little difficult to read. First one had While we're picky: "were" - Paolo can you fix up in your queue? Andreas > incorrect parenthesis. The part about attributes changing is > really applicable to the region being 'in both' rather than 'in > new' >=20 > Second comment has an obscure parenthetic about 'Logging may have > changed'. Made clearer, as this if is supposed to handle the case where > the memory region is unchanged (with the notable exception re logging). >=20 > Signed-off-by: Peter Crosthwaite > --- >=20 > memory.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >=20 > diff --git a/memory.c b/memory.c > index 60e033b..7e710c4 100644 > --- a/memory.c > +++ b/memory.c > @@ -719,7 +719,7 @@ static void address_space_update_topology_pass(Addr= essSpace *as, > || int128_lt(frold->addr.start, frnew->addr.start) > || (int128_eq(frold->addr.start, frnew->addr.start) > && !flatrange_equal(frold, frnew)))) { > - /* In old, but (not in new, or in new but attributes chang= ed). */ > + /* In old but not in new, or in both but attributes change= d. */ > =20 > if (!adding) { > MEMORY_LISTENER_UPDATE_REGION(frold, as, Reverse, regi= on_del); > @@ -727,7 +727,7 @@ static void address_space_update_topology_pass(Addr= essSpace *as, > =20 > ++iold; > } else if (frold && frnew && flatrange_equal(frold, frnew)) { > - /* In both (logging may have changed) */ > + /* In both and unchanged (except logging may have changed)= */ > =20 > if (adding) { > MEMORY_LISTENER_UPDATE_REGION(frnew, as, Forward, regi= on_nop); >=20 --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe= rg