From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-la0-f45.google.com (mail-la0-f45.google.com [209.85.215.45]) by kanga.kvack.org (Postfix) with ESMTP id 0AFF16B0082 for ; Tue, 3 Feb 2015 17:59:00 -0500 (EST) Received: by mail-la0-f45.google.com with SMTP id gd6so54977422lab.4 for ; Tue, 03 Feb 2015 14:58:59 -0800 (PST) Date: Tue, 3 Feb 2015 14:58:45 -0800 From: Shaohua Li Subject: Re: [PATCH 2/2] aio: make aio .mremap handle size changes Message-ID: <20150203225845.GA749607@devbig257.prn2.facebook.com> References: <798fafb96373cfab0707457a266dd137016cd1e9.1422986358.git.shli@fb.com> <20150203192323.GT2974@kvack.org> <20150203193115.GA296459@devbig257.prn2.facebook.com> <20150203194828.GU2974@kvack.org> <20150203213150.GA543371@devbig257.prn2.facebook.com> <20150203214749.GA14400@kvack.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20150203214749.GA14400@kvack.org> Sender: owner-linux-mm@kvack.org List-ID: To: Benjamin LaHaise Cc: linux-mm@kvack.org, Kernel-team@fb.com, Andrew Morton On Tue, Feb 03, 2015 at 04:47:49PM -0500, Benjamin LaHaise wrote: > On Tue, Feb 03, 2015 at 01:31:50PM -0800, Shaohua Li wrote: > > On Tue, Feb 03, 2015 at 02:48:28PM -0500, Benjamin LaHaise wrote: > > > On Tue, Feb 03, 2015 at 11:31:15AM -0800, Shaohua Li wrote: > > > > On Tue, Feb 03, 2015 at 02:23:23PM -0500, Benjamin LaHaise wrote: > > > > > On Tue, Feb 03, 2015 at 11:18:53AM -0800, Shaohua Li wrote: > > > > > > mremap aio ring buffer to another smaller vma is legal. For example, > > > > > > mremap the ring buffer from the begining, though after the mremap, some > > > > > > ring buffer pages can't be accessed in userspace because vma size is > > > > > > shrinked. The problem is ctx->mmap_size isn't changed if the new ring > > > > > > buffer vma size is changed. Latter io_destroy will zap all vmas within > > > > > > mmap_size, which might zap unrelated vmas. > > > > > > > > > > Nak. Shrinking the aio ring buffer is not a supported operation and will > > > > > cause the application to lose events. Make the size changing mremap fail, > > > > > as this patch will not make the system do the right thing. > > > > > > > > Yes, making the syscall fail (vma ops has .remap) is another option. If > > > > the app uses io_getevents(), looks the app will not lose events, no? On > > > > the other hand, I just want to make sure kernel does the right thing > > > > (not zap unrelated vmas). If app does crazy things, it will break. > > > > > > But reading events out of the ring buffer is a supported mode of operation. > > > Given that constraint, you should make an mremap changing the size of the > > > ring buffer fail. > > > > But if app chooses not to read the ring buffer, the app isn't broken. > > This makes me hesitate to make the syscall fail. > > There is a grey area about what's the correct semantics for mremap. We > > currently don't limit any vma shrink for mremap. > > mremap() on the aio ringbuffer was not previously supported. The support was > added recently was to enable checkpoint/restore to work properly. In that > use-case there is no need resize the ring buffer's mmap window. > > If you really want to fix this, then make the mremap operation do the right > thing and actually change the size of the aio ring. That is going to be > vastly more complex of a change for no real gain since no applications use > that functionality. That's too complex. Don't think I'll spend time on the no-usage functionality. Will leave it be if you don't like the sample workaround. Thanks, Shaohua -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org