Re: [Xen-devel] xen-blkback: bug fixes

2014-02-04 Thread Konrad Rzeszutek Wilk
On Mon, Feb 03, 2014 at 10:31:40PM -0800, Matt Wilson wrote:
> On Tue, Jan 28, 2014 at 03:38:37PM -0400, Konrad Rzeszutek Wilk wrote:
> > On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
> > > blkback bug fixes for memory leaks (patches 1 and 2) and a race 
> > > (patch 3).
> > 
> > They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
> > branch and are testing them now (hadn't pushed it yet).
> > 
> > Matt and Matt,
> > 
> > Could you take a look at the other two patches as well?
> 
> Sure, though somehow you didn't address your message to us, so I
> didn't see it until today.

Duh!
> 
> Matt Rushton did some review and testing on an earlier version that
> came out fine. We'll give the final series a test since there was
> still a bit of rework.


> 
> --msw
> 
> > David, Boris,
> > 
> > Are you OK with pushing those patches out to Jens Axboe if nobody
> > gives an NACK by Friday?
> 
> ___
> Xen-devel mailing list
> xen-de...@lists.xen.org
> http://lists.xen.org/xen-devel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-02-04 Thread Konrad Rzeszutek Wilk
On Mon, Feb 03, 2014 at 10:31:40PM -0800, Matt Wilson wrote:
 On Tue, Jan 28, 2014 at 03:38:37PM -0400, Konrad Rzeszutek Wilk wrote:
  On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
   blkback bug fixes for memory leaks (patches 1 and 2) and a race 
   (patch 3).
  
  They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
  branch and are testing them now (hadn't pushed it yet).
  
  Matt and Matt,
  
  Could you take a look at the other two patches as well?
 
 Sure, though somehow you didn't address your message to us, so I
 didn't see it until today.

Duh!
 
 Matt Rushton did some review and testing on an earlier version that
 came out fine. We'll give the final series a test since there was
 still a bit of rework.

nods
 
 --msw
 
  David, Boris,
  
  Are you OK with pushing those patches out to Jens Axboe if nobody
  gives an NACK by Friday?
 
 ___
 Xen-devel mailing list
 xen-de...@lists.xen.org
 http://lists.xen.org/xen-devel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-02-03 Thread Matt Wilson
On Tue, Jan 28, 2014 at 03:38:37PM -0400, Konrad Rzeszutek Wilk wrote:
> On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
> > blkback bug fixes for memory leaks (patches 1 and 2) and a race 
> > (patch 3).
> 
> They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
> branch and are testing them now (hadn't pushed it yet).
> 
> Matt and Matt,
> 
> Could you take a look at the other two patches as well?

Sure, though somehow you didn't address your message to us, so I
didn't see it until today.

Matt Rushton did some review and testing on an earlier version that
came out fine. We'll give the final series a test since there was
still a bit of rework.

--msw

> David, Boris,
> 
> Are you OK with pushing those patches out to Jens Axboe if nobody
> gives an NACK by Friday?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-02-03 Thread Matt Wilson
On Tue, Jan 28, 2014 at 03:38:37PM -0400, Konrad Rzeszutek Wilk wrote:
 On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
  blkback bug fixes for memory leaks (patches 1 and 2) and a race 
  (patch 3).
 
 They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
 branch and are testing them now (hadn't pushed it yet).
 
 Matt and Matt,
 
 Could you take a look at the other two patches as well?

Sure, though somehow you didn't address your message to us, so I
didn't see it until today.

Matt Rushton did some review and testing on an earlier version that
came out fine. We'll give the final series a test since there was
still a bit of rework.

--msw

 David, Boris,
 
 Are you OK with pushing those patches out to Jens Axboe if nobody
 gives an NACK by Friday?
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-01-28 Thread Boris Ostrovsky

On 01/28/2014 02:38 PM, Konrad Rzeszutek Wilk wrote:

On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:

blkback bug fixes for memory leaks (patches 1 and 2) and a race
(patch 3).

They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
branch and are testing them now (hadn't pushed it yet).

Matt and Matt,

Could you take a look at the other two patches as well?

David, Boris,

Are you OK with pushing those patches out to Jens Axboe if nobody
gives an NACK by Friday?


The patches look reasonable to me so I don't have any objections (but
I am not particularly familiar with this code.)

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


Re: [Xen-devel] xen-blkback: bug fixes

2014-01-28 Thread Konrad Rzeszutek Wilk
On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
> blkback bug fixes for memory leaks (patches 1 and 2) and a race 
> (patch 3).

They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
branch and are testing them now (hadn't pushed it yet).

Matt and Matt,

Could you take a look at the other two patches as well?

David, Boris,

Are you OK with pushing those patches out to Jens Axboe if nobody
gives an NACK by Friday?

> 
> ___
> Xen-devel mailing list
> xen-de...@lists.xen.org
> http://lists.xen.org/xen-devel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-01-28 Thread Konrad Rzeszutek Wilk
On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:
 blkback bug fixes for memory leaks (patches 1 and 2) and a race 
 (patch 3).

They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
branch and are testing them now (hadn't pushed it yet).

Matt and Matt,

Could you take a look at the other two patches as well?

David, Boris,

Are you OK with pushing those patches out to Jens Axboe if nobody
gives an NACK by Friday?

 
 ___
 Xen-devel mailing list
 xen-de...@lists.xen.org
 http://lists.xen.org/xen-devel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [Xen-devel] xen-blkback: bug fixes

2014-01-28 Thread Boris Ostrovsky

On 01/28/2014 02:38 PM, Konrad Rzeszutek Wilk wrote:

On Tue, Jan 28, 2014 at 06:43:32PM +0100, Roger Pau Monne wrote:

blkback bug fixes for memory leaks (patches 1 and 2) and a race
(patch 3).

They all look OK to me. I've stuck them in my 'stable/for-jens-3.14'
branch and are testing them now (hadn't pushed it yet).

Matt and Matt,

Could you take a look at the other two patches as well?

David, Boris,

Are you OK with pushing those patches out to Jens Axboe if nobody
gives an NACK by Friday?


The patches look reasonable to me so I don't have any objections (but
I am not particularly familiar with this code.)

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