Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751518AbaLNXCf (ORCPT ); Sun, 14 Dec 2014 18:02:35 -0500 Received: from mta-out1.inet.fi ([62.71.2.195]:34186 "EHLO jenni1.inet.fi" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751119AbaLNXC2 (ORCPT ); Sun, 14 Dec 2014 18:02:28 -0500 Date: Mon, 15 Dec 2014 01:02:08 +0200 From: "Kirill A. Shutemov" To: Andrew Morton Cc: Benjamin LaHaise , Linus Torvalds , linux-aio@kvack.org, Linux Kernel , linux-mm , Pavel Emelyanov , Dmitry Monakhov Subject: Re: [GIT PULL] aio: changes for 3.19 Message-ID: <20141214230208.GA9217@node.dhcp.inet.fi> References: <20141214202224.GH2672@kvack.org> <20141214215221.GI2672@kvack.org> <20141214141336.a0267e95.akpm@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20141214141336.a0267e95.akpm@linux-foundation.org> User-Agent: Mutt/1.5.23.1 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Dec 14, 2014 at 02:13:36PM -0800, Andrew Morton wrote: > On Sun, 14 Dec 2014 16:52:21 -0500 Benjamin LaHaise wrote: > > > On Sun, Dec 14, 2014 at 01:47:32PM -0800, Linus Torvalds wrote: > > > On Sun, Dec 14, 2014 at 12:22 PM, Benjamin LaHaise wrote: > > > > > > > > Pavel Emelyanov (1): > > > > aio: Make it possible to remap aio ring > > > > > > So quite frankly, I think this should have had more acks from VM > > > people. The patch looks ok to me, but it took me by surprise, and I > > > don't see much any discussion about it on linux-mm either.. > > > > Sadly, nobody responded. Maybe akpm can chime in on this change (included > > below for ease of review and akpm added to the To:)? > > > > -ben > > -- > > "Thought is the essence of where you are now." > > > > commit e4a0d3e720e7e508749c1439b5ba3aff56c92976 > > Author: Pavel Emelyanov > > Date: Thu Sep 18 19:56:17 2014 +0400 > > > > aio: Make it possible to remap aio ring > > The patch appears to be a bugfix which coincidentally helps CRIU? > > If it weren't for the bugfix part, I'd be asking "why not pass the > desired virtual address into io_setup()?". But it seems the problem is bigger than what the patch fixes. To me we are too permisive on what vma can be remapped. How can we know that it's okay to move vma around for random driver which provide .mmap? Or I miss something obvious? -- Kirill A. Shutemov -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/