Can I add your Signed-off-by: on that? I'll send this upstream with the other patches so it'll hopefully make 3.1... Thanks! sage On Wed, 28 Sep 2011, David Flynn wrote: > The finish_read callback introduced in 63c90314546c1cec1f220f6ab24ea > fails to release the page list allocated in start_read. > --- > fs/ceph/addr.c | 1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/fs/ceph/addr.c b/fs/ceph/addr.c > index e06a322..4144caf 100644 > --- a/fs/ceph/addr.c > +++ b/fs/ceph/addr.c > @@ -261,6 +261,7 @@ static void finish_read(struct ceph_osd_request *req, struct ceph_msg *msg) > unlock_page(page); > page_cache_release(page); > } > + kfree(req->r_pages); > } > > /* > -- > 1.7.4.1 > -- > To unsubscribe from this list: send the line "unsubscribe ceph-devel" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
