From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1163770AbdDXAbn (ORCPT ); Sun, 23 Apr 2017 20:31:43 -0400 Received: from ozlabs.org ([103.22.144.67]:56579 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758552AbdDXAbi (ORCPT ); Sun, 23 Apr 2017 20:31:38 -0400 Date: Mon, 24 Apr 2017 10:31:34 +1000 From: Stephen Rothwell To: Al Viro Cc: LKML , Linus Subject: patch series moving compat syscalls from fs/compat.c Message-ID: <20170424103134.48baba07@canb.auug.org.au> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Al, I couldn't easily find this patch series posted anywhere (I didn't try too hard) so I sill just comment here. I first noticed them in the vfs tree in linux-next today. Overall, I like what they day, but when I first created kernel/compat.c (in 2002), Linus did not want the compat code sprinkled all over the other files (due to the uglifiying effect of #ifdefs in the C code, and because the compat layer was only secondary). He may have changed his mind since then, but it is worth asking. The cleanups (including more static functions) make it worth while for me, at least. It also might have been worth asking the original (and subsequent) authors of the code for review and/or comments. One thing I noticed from a quick look: do_handle_open() could now be static. -- Cheers, Stephen Rothwell