Bug 217993 - blktap: Possible grant table leaks
Summary: blktap: Possible grant table leaks
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: kernel-xen
Version: 5.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Xen Maintainance List
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-12-01 05:18 UTC by Herbert Xu
Modified: 2007-11-30 22:07 UTC (History)
0 users

Fixed In Version: 5.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-01-26 20:55:32 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Herbert Xu 2006-12-01 05:18:03 UTC
We need changesets 11835-11837 from xen-unstable.

[BLKTAP]: Kill duplicate fast_flush_area call

The dispatch_rw_block_io may call fast_flush_area twice if
create_lookup_pte_addr fails (there is a flush call at fail_flush
already).  The second call simply causes warnings to be printed
on the console.  This patch removes the duplicate call.

Signed-off-by: Herbert Xu <herbert.org.au>

[BLKTAP]: Kill bogus fast_flush_area

The function dispatch_rw_block_io may call fast_flush_area even before a
pending request has been initialised.  This patch deletes that call.

Signed-off-by: Herbert Xu <herbert.org.au>

[BLKTAP]: Fix potential grant entry leaks on error

As it stands grant entries for the actual data are mapped in bulk.
If one of the earlier entries fail to be mapped, it can cause all
subsequent entries to be leaked if they were successfully mapped.

This patch changes it so that we will continue to reap grant entries
even when an error is detected.

There is still a potential leak if create_lookup_pte_addr fails
in fast_flush_area.  This leak is not affected by this patch.

Signed-off-by: Herbert Xu <herbert.org.au>

Comment 1 Jay Turner 2006-12-01 14:17:44 UTC
QE ack for RHEL5.

Comment 3 Don Zickus 2007-01-03 01:16:27 UTC
in 2.6.18-1.2943.el5

Comment 4 Jay Turner 2007-01-26 20:55:32 UTC
2.6.18-7.el5 included in 20070125.0.


Note You need to log in before you can comment on or make changes to this bug.