From mboxrd@z Thu Jan  1 00:00:00 1970
From: Keir Fraser <keir@xen.org>
Subject: Re: [PATCH] Extra check in grant table code for mapping
 of shared frame
Date: Mon, 17 Sep 2012 12:17:27 +0100
Message-ID: <CC7CC557.4BEFD%keir@xen.org>
References: <7A2C5C61-AFEF-4258-9EEB-CC4A0727920E@gridcentric.ca>
Mime-Version: 1.0
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <7A2C5C61-AFEF-4258-9EEB-CC4A0727920E@gridcentric.ca>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Andres Lagar-Cavilla <andreslc@gridcentric.ca>, Andres Lagar-Cavilla <andres@lagarcavilla.org>
Cc: tim@xen.org, JBeulich@suse.com, xen-devel@lists.xen.org
List-Id: xen-devel@lists.xenproject.org

Probably needs Tim to comment on it. Assuming he's any wiser about this code
than the rest of us. ;)

 -- Keir

On 17/09/2012 12:00, "Andres Lagar-Cavilla" <andreslc@gridcentric.ca> wrote:

> ping=8A
> Thanks,
> Andres
> On Sep 13, 2012, at 11:27 AM, Andres Lagar-Cavilla wrote:
> =

>> xen/common/grant_table.c |  9 ++++++---
>> 1 files changed, 6 insertions(+), 3 deletions(-)
>> =

>> =

>> Signed-off-by: Andres Lagar-Cavilla <andres@lagarcavilla.org>
>> =

>> diff -r 5ce5b53ea68f -r 40b91bed1275 xen/common/grant_table.c
>> --- a/xen/common/grant_table.c
>> +++ b/xen/common/grant_table.c
>> @@ -649,9 +649,12 @@ __gnttab_map_grant_ref(
>>     }
>>     else if ( owner =3D=3D rd || owner =3D=3D dom_cow )
>>     {
>> -        if ( gnttab_host_mapping_get_page_type(op, ld, rd) &&
>> -             !get_page_type(pg, PGT_writable_page) )
>> -            goto could_not_pin;
>> +        if ( gnttab_host_mapping_get_page_type(op, ld, rd) )
>> +        {
>> +            if ( (owner =3D=3D dom_cow) ||
>> +                 !get_page_type(pg, PGT_writable_page) )
>> +                goto could_not_pin;
>> +        }
>> =

>>         nr_gets++;
>>         if ( op->flags & GNTMAP_host_map )
> =