Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1659751 > unrolled thread

Re: [PATCH v2 0/3] xen/blkback: several fixes of resource management

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v2 0/3] xen/blkback: several fixes of resource management Steven Haigh <netwiz@crc.id.au> - 2017-06-07 14:40 +0200
    Re: [Xen-devel] [PATCH v2 0/3] xen/blkback: several fixes of  resource management Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - 2017-06-07 16:00 +0200
      Re: [Xen-devel] [PATCH v2 0/3] xen/blkback: several fixes of resource management Steven Haigh <netwiz@crc.id.au> - 2017-06-07 16:10 +0200

#1659751 — Re: [PATCH v2 0/3] xen/blkback: several fixes of resource management

[Multipart message — attachments visible in raw view] — view raw

On Friday, 19 May 2017 1:28:46 AM AEST Juergen Gross wrote:
> Destroying a Xen guest domain while it was doing I/Os via xen-blkback
> leaked several resources, including references of the guest's memory
> pages.
> 
> This patch series addresses those leaks by correcting usage of
> reference counts and the sequence when to free which resource.
> 
> The series applies on top of commit 2d4456c73a487abe ("block:
> xen-blkback: add null check to avoid null pointer dereference") in
> Jens Axboe's tree kernel/git/axboe/linux-block.git
> 
> V2: changed flag to type bool in patch 1 (Dietmar Hahn)
> 
> Juergen Gross (3):
>   xen/blkback: fix disconnect while I/Os in flight
>   xen/blkback: don't free be structure too early
>   xen/blkback: don't use xen_blkif_get() in xen-blkback kthread
> 
>  drivers/block/xen-blkback/blkback.c |  3 ---
>  drivers/block/xen-blkback/common.h  |  1 +
>  drivers/block/xen-blkback/xenbus.c  | 15 ++++++++-------
>  3 files changed, 9 insertions(+), 10 deletions(-)

Just wanted to give this a bit of a prod.

Is there any plans in having this hit the kernel.org kernels?

My testing was purely on kernel 4.9 branch - but it doesn't look like this has 
shown up there yet?

-- 
Steven Haigh

📧 netwiz@crc.id.au	💻 http://www.crc.id.au
📞 +61 (3) 9001 6090	📱 0412 935 897

[toc] | [next] | [standalone]

#1659823 — Re: [Xen-devel] [PATCH v2 0/3] xen/blkback: several fixes of resource management

On Wed, Jun 07, 2017 at 10:36:58PM +1000, Steven Haigh wrote:
> On Friday, 19 May 2017 1:28:46 AM AEST Juergen Gross wrote:
> > Destroying a Xen guest domain while it was doing I/Os via xen-blkback
> > leaked several resources, including references of the guest's memory
> > pages.
> > 
> > This patch series addresses those leaks by correcting usage of
> > reference counts and the sequence when to free which resource.
> > 
> > The series applies on top of commit 2d4456c73a487abe ("block:
> > xen-blkback: add null check to avoid null pointer dereference") in
> > Jens Axboe's tree kernel/git/axboe/linux-block.git
> > 
> > V2: changed flag to type bool in patch 1 (Dietmar Hahn)
> > 
> > Juergen Gross (3):
> >   xen/blkback: fix disconnect while I/Os in flight
> >   xen/blkback: don't free be structure too early
> >   xen/blkback: don't use xen_blkif_get() in xen-blkback kthread
> > 
> >  drivers/block/xen-blkback/blkback.c |  3 ---
> >  drivers/block/xen-blkback/common.h  |  1 +
> >  drivers/block/xen-blkback/xenbus.c  | 15 ++++++++-------
> >  3 files changed, 9 insertions(+), 10 deletions(-)
> 
> Just wanted to give this a bit of a prod.

<ugh> Ouch!

> 
> Is there any plans in having this hit the kernel.org kernels?

Yes.
> 
> My testing was purely on kernel 4.9 branch - but it doesn't look like this has 
> shown up there yet?

Correct. I am thinking to send these to Jens around June 20th or so.

[toc] | [prev] | [next] | [standalone]

#1659830 — Re: [Xen-devel] [PATCH v2 0/3] xen/blkback: several fixes of resource management

[Multipart message — attachments visible in raw view] — view raw

On Wednesday, 7 June 2017 11:52:34 PM AEST Konrad Rzeszutek Wilk wrote:
> On Wed, Jun 07, 2017 at 10:36:58PM +1000, Steven Haigh wrote:
> > On Friday, 19 May 2017 1:28:46 AM AEST Juergen Gross wrote:
> > > Destroying a Xen guest domain while it was doing I/Os via xen-blkback
> > > leaked several resources, including references of the guest's memory
> > > pages.
> > > 
> > > This patch series addresses those leaks by correcting usage of
> > > reference counts and the sequence when to free which resource.
> > > 
> > > The series applies on top of commit 2d4456c73a487abe ("block:
> > > xen-blkback: add null check to avoid null pointer dereference") in
> > > Jens Axboe's tree kernel/git/axboe/linux-block.git
> > > 
> > > V2: changed flag to type bool in patch 1 (Dietmar Hahn)
> > > 
> > > Juergen Gross (3):
> > >   xen/blkback: fix disconnect while I/Os in flight
> > >   xen/blkback: don't free be structure too early
> > >   xen/blkback: don't use xen_blkif_get() in xen-blkback kthread
> > >  
> > >  drivers/block/xen-blkback/blkback.c |  3 ---
> > >  drivers/block/xen-blkback/common.h  |  1 +
> > >  drivers/block/xen-blkback/xenbus.c  | 15 ++++++++-------
> > >  3 files changed, 9 insertions(+), 10 deletions(-)
> > 
> > Just wanted to give this a bit of a prod.
> 
> <ugh> Ouch!
> 
> > Is there any plans in having this hit the kernel.org kernels?
> 
> Yes.
> 
> > My testing was purely on kernel 4.9 branch - but it doesn't look like this
> > has shown up there yet?
> 
> Correct. I am thinking to send these to Jens around June 20th or so.

Ok, all understood. Thanks for the clarifications.

At the moment, I'm just including them in my kernel builds - then expecting 
them to fail at one point in the future when the patch fails due to 
upstreaming. I'll just keep doing this.

-- 
Steven Haigh

📧 netwiz@crc.id.au	💻 http://www.crc.id.au
📞 +61 (3) 9001 6090	📱 0412 935 897

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web