Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757329Ab2JWSVB (ORCPT ); Tue, 23 Oct 2012 14:21:01 -0400 Received: from mail-pa0-f46.google.com ([209.85.220.46]:58260 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756776Ab2JWSU7 (ORCPT ); Tue, 23 Oct 2012 14:20:59 -0400 Date: Tue, 23 Oct 2012 11:20:55 -0700 From: Greg KH To: Jaegeuk Kim Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, viro@zeniv.linux.org.uk, arnd@arndb.de, tytso@mit.edu, chur.lee@samsung.com, cm224.lee@samsung.com, jooyoung.hwang@samsung.com Subject: [PATCH 0/3] f2fs: move proc files to debugfs Message-ID: <20121023182055.GA6096@kroah.com> References: <001001cdb0c5$2ac96520$805c2f60$%kim@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <001001cdb0c5$2ac96520$805c2f60$%kim@samsung.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1304 Lines: 32 Here are 3 patches, moving the proc file usage on f2fs to debugfs. The first one fixes a bug in the gc.h file preventing it from being able to be included by any other files. The second patch moves all current proc file accesses to a single file, removing all #ifdefs from the .c files. This should have been done in the first place. The last file converts the files to use debugfs instead of proc. Note, these patches have been compile tested only, I haven't tested them out, as I haven't had the chance to yet. I'll go do that this afternoon after I catch up on some other pending kernel work. One question, it seems that the proc files show all information for all super blocks in the system, no matter which subdirectory you are reading from in the proc f2fs tree. Is that really what you want? Shouldn't we only be showing the stats of the superblock we are saying we will report? I'll test that later today, and if it really is wrong, will fix the debugfs code up to handle this properly. Do these patches look reasonable? thanks, greg k-h -- 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/