Re: [Xen-devel] [PATCH] dmar: device scope mem leak fix

2015-05-28 Thread Elena Ufimtseva
On Thu, May 28, 2015 at 08:57:16AM +0100, Jan Beulich wrote: > >>> On 27.05.15 at 21:56, wrote: > > On Tue, May 26, 2015 at 10:46:30AM +0100, Jan Beulich wrote: > >> >>> On 23.05.15 at 03:27, wrote: > >> > @@ -658,6 +661,7 @@ acpi_parse_one_rmrr(struct acpi_dmar_header *header) > >> >

Re: [Xen-devel] [PATCH] dmar: device scope mem leak fix

2015-05-28 Thread Jan Beulich
>>> On 27.05.15 at 21:56, wrote: > On Tue, May 26, 2015 at 10:46:30AM +0100, Jan Beulich wrote: >> >>> On 23.05.15 at 03:27, wrote: >> > @@ -658,6 +661,7 @@ acpi_parse_one_rmrr(struct acpi_dmar_header *header) >> > " Ignore the RMRR (%"PRIx64", %"PRIx64") due to " >> >

Re: [Xen-devel] [PATCH] dmar: device scope mem leak fix

2015-05-27 Thread Elena Ufimtseva
On Tue, May 26, 2015 at 10:46:30AM +0100, Jan Beulich wrote: > >>> On 23.05.15 at 03:27, wrote: > > @@ -318,13 +321,13 @@ static int __init acpi_parse_dev_scope( > > if ( (cnt = scope_device_count(start, end)) < 0 ) > > return cnt; > > > > -scope->devices_cnt = cnt; > > if

Re: [Xen-devel] [PATCH] dmar: device scope mem leak fix

2015-05-26 Thread Jan Beulich
>>> On 23.05.15 at 03:27, wrote: > @@ -318,13 +321,13 @@ static int __init acpi_parse_dev_scope( > if ( (cnt = scope_device_count(start, end)) < 0 ) > return cnt; > > -scope->devices_cnt = cnt; > if ( cnt > 0 ) > { > scope->devices = xzalloc_array(u16, cnt);

[Xen-devel] [PATCH] dmar: device scope mem leak fix

2015-05-22 Thread elena . ufimtseva
From: Elena Ufimtseva Release memory allocated for scope.devices when disabling dmar units. Also set device count after memory allocation when device scope parsing. Signed-off-by: Elena Ufimtseva --- xen/drivers/passthrough/vtd/dmar.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-)