Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753393AbcD2OMD (ORCPT ); Fri, 29 Apr 2016 10:12:03 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:50579 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752670AbcD2OMB (ORCPT ); Fri, 29 Apr 2016 10:12:01 -0400 Date: Fri, 29 Apr 2016 11:11:49 -0300 From: Gustavo Padovan To: Hillf Danton Cc: linux-kernel Subject: Re: [PATCH v2 02/13] staging/android: drop sync_file_install() and sync_file_put() Message-ID: <20160429141149.GD2524@joana> References: <00da01d1a1ed$f8d8e4a0$ea8aade0$@alibaba-inc.com> <00dc01d1a1ef$2692b6e0$73b824a0$@alibaba-inc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <00dc01d1a1ef$2692b6e0$73b824a0$@alibaba-inc.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 788 Lines: 19 2016-04-29 Hillf Danton : > > > > From: Gustavo Padovan > > > > These two functions are just wrappers for one line functions, they > > call fd_install() and fput() respectively, so just get rid of them > > and use fd_install() and fput() directly for more simplicity. > > > Given sync_file is not file, I don't see that simplicity is worth of > the change of 20+ lines. > Can you please specify the disadvantages of the wrappers? Our idea here was to simplify as much as possible this API, also we do not want to hide sync_file internals from its users. This is staging and we are re-thinking the API (and userspace ABI too) before moving it out if staging. I can add some more information to the commit message. Gustavo