From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Mon, 13 Mar 2017 23:29:10 +0100 (CET) Received: from bombadil.infradead.org ([65.50.211.133]:48622 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S23993996AbdCMW3DjxSVA (ORCPT ); Mon, 13 Mar 2017 23:29:03 +0100 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20170209; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=HsQELeL4M84a51YFBWk+0Tng76cwXcu89/zl474iRQs=; b=b8NV8GpZi5HS2Gw1ra0xgc5Cv ScDvFazNl3rYLaC8ZkkMy/BlPtWGhwd7opHD5WtQZ2QfjNazFWsKw8dPN9WjLKfhRSoXzqJDYDgZA o0snAUHWLHZa6rs5x/1C+MrrEw6mIUCbzMkwsqjMH88nK5CuRlyfSQHotjqunScxWSINyNc0MvdAu 1+BCeekI97Rmb6k7Sjb1am+TQovBNQJ+7zWmWYv7W06bZOX3y37Fvj5gESGi0gmXm1zG+RWJCc/Uq e2kqhAnaiW25kUKVEsQ6ndcvjigXOr1bdoKZOj/Mnh0tqjZL/gZEbEqM0wRjuXnqzNzAC+9SN6rlh PZLtKqWUg==; Received: from willy by bombadil.infradead.org with local (Exim 4.87 #1 (Red Hat Linux)) id 1cnYRa-0002rp-85; Mon, 13 Mar 2017 22:27:58 +0000 Date: Mon, 13 Mar 2017 15:27:58 -0700 From: Matthew Wilcox To: Till Smejkal Cc: Richard Henderson , Ivan Kokshaysky , Matt Turner , Vineet Gupta , Russell King , Catalin Marinas , Will Deacon , Steven Miao , Richard Kuo , Tony Luck , Fenghua Yu , James Hogan , Ralf Baechle , "James E.J. Bottomley" , Helge Deller , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Martin Schwidefsky , Heiko Carstens , Yoshinori Sato , Rich Felker , "David S. Miller" , Chris Metcalf , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Andy Lutomirski , Chris Zankel , Max Filippov , Arnd Bergmann , Greg Kroah-Hartman , Laurent Pinchart , Mauro Carvalho Chehab , Pawel Osciak , Marek Szyprowski , Kyungmin Park , David Woodhouse , Brian Norris , Boris Brezillon , Marek Vasut , Richard Weinberger , Cyrille Pitchen , Felipe Balbi , Alexander Viro , Benjamin LaHaise , Nadia Yvette Chambers , Jeff Layton , "J. Bruce Fields" , Peter Zijlstra , Hugh Dickins , Arnaldo Carvalho de Melo , Alexander Shishkin , Jaroslav Kysela , Takashi Iwai , linux-kernel@vger.kernel.org, linux-alpha@vger.kernel.org, linux-snps-arc@lists.infradead.org, linux-arm-kernel@lists.infradead.org, adi-buildroot-devel@lists.sourceforge.net, linux-hexagon@vger.kernel.org, linux-ia64@vger.kernel.org, linux-metag@vger.kernel.org, linux-mips@linux-mips.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-sh@vger.kernel.org, sparclinux@vger.kernel.org, linux-xtensa@linux-xtensa.org, linux-media@vger.kernel.org, linux-mtd@lists.infradead.org, linux-usb@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-aio@kvack.org, linux-mm@kvack.org, linux-api@vger.kernel.org, linux-arch@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [RFC PATCH 11/13] mm/vas: Introduce VAS segments - shareable address space regions Message-ID: <20170313222758.GB4033@bombadil.infradead.org> References: <20170313221415.9375-1-till.smejkal@gmail.com> <20170313221415.9375-12-till.smejkal@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170313221415.9375-12-till.smejkal@gmail.com> User-Agent: Mutt/1.7.1 (2016-10-04) Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 57182 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: willy@infradead.org Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On Mon, Mar 13, 2017 at 03:14:13PM -0700, Till Smejkal wrote: > +/** > + * Create a new VAS segment. > + * > + * @param[in] name: The name of the new VAS segment. > + * @param[in] start: The address where the VAS segment begins. > + * @param[in] end: The address where the VAS segment ends. > + * @param[in] mode: The access rights for the VAS segment. > + * > + * @returns: The VAS segment ID on success, -ERRNO otherwise. > + **/ Please follow the kernel-doc conventions, as described in Documentation/doc-guide/kernel-doc.rst. Also, function documentation goes with the implementation, not the declaration. > +/** > + * Get ID of the VAS segment belonging to a given name. > + * > + * @param[in] name: The name of the VAS segment for which the ID > + * should be returned. > + * > + * @returns: The VAS segment ID on success, -ERRNO > + * otherwise. > + **/ > +extern int vas_seg_find(const char *name); So ... segments have names, and IDs ... and access permissions ... Why isn't this a special purpose filesystem?