linux-kernel - Re: [PATCH] agp: fix scratch page cleanup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <4F212ECC020000780006F26B@nat28.tlf.novell.com>
Date:	Thu, 26 Jan 2012 09:45:32 +0000
From:	"Jan Beulich" <JBeulich@...e.com>
To:	"Dave Airlie" <airlied@...ux.ie>
Cc:	"Michal Kubecek" <mkubecek@...e.cz>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] agp: fix scratch page cleanup

>>> On 25.01.12 at 20:01, Dave Airlie <airlied@...ux.ie> wrote:

>> In error cleanup of agp_backend_initialize() and in agp_backend_cleanup(),
>> agp_destroy_page() is passed virtual address of the scratch page. This
>> leads to a kernel warning if the initialization fails (or upon regular
>> cleanup) as pointer to struct page should be passed instead.
> 
> Jan can you check and ack this if okay? since you wrote this chunk 
> originally?

The change here is fine, but rather fixes an omission of your own
"agp: switch AGP to use page array instead of unsigned long array"
(which went in for 2.6.31, whereas my change went into 2.6.26),
which also wasn't noticed when David Woodhouse touched the
code again for 2.6.32 ("agp: Switch agp_{un,}map_page() to take
struct page * argument").

Jan

>> 
>> Signed-off-by: Michal Kubecek <mkubecek@...e.cz>
>> ---
>>  drivers/char/agp/backend.c |   12 ++++++------
>>  1 files changed, 6 insertions(+), 6 deletions(-)
>> 
>> diff --git a/drivers/char/agp/backend.c b/drivers/char/agp/backend.c
>> index 4b71647..317c28c 100644
>> --- a/drivers/char/agp/backend.c
>> +++ b/drivers/char/agp/backend.c
>> @@ -194,10 +194,10 @@ static int agp_backend_initialize(struct 
> agp_bridge_data *bridge)
>>  
>>  err_out:
>>  	if (bridge->driver->needs_scratch_page) {
>> -		void *va = page_address(bridge->scratch_page_page);
>> +		struct page *page = bridge->scratch_page_page;
>>  
>> -		bridge->driver->agp_destroy_page(va, AGP_PAGE_DESTROY_UNMAP);
>> -		bridge->driver->agp_destroy_page(va, AGP_PAGE_DESTROY_FREE);
>> +		bridge->driver->agp_destroy_page(page, AGP_PAGE_DESTROY_UNMAP);
>> +		bridge->driver->agp_destroy_page(page, AGP_PAGE_DESTROY_FREE);
>>  	}
>>  	if (got_gatt)
>>  		bridge->driver->free_gatt_table(bridge);
>> @@ -221,10 +221,10 @@ static void agp_backend_cleanup(struct agp_bridge_data 
> *bridge)
>>  
>>  	if (bridge->driver->agp_destroy_page &&
>>  	    bridge->driver->needs_scratch_page) {
>> -		void *va = page_address(bridge->scratch_page_page);
>> +		struct page *page = bridge->scratch_page_page;
>>  
>> -		bridge->driver->agp_destroy_page(va, AGP_PAGE_DESTROY_UNMAP);
>> -		bridge->driver->agp_destroy_page(va, AGP_PAGE_DESTROY_FREE);
>> +		bridge->driver->agp_destroy_page(page, AGP_PAGE_DESTROY_UNMAP);
>> +		bridge->driver->agp_destroy_page(page, AGP_PAGE_DESTROY_FREE);
>>  	}
>>  }
>>  
>> 



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/