From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752651AbbJMIBT (ORCPT ); Tue, 13 Oct 2015 04:01:19 -0400 Received: from mail.skyhub.de ([78.46.96.112]:44975 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752190AbbJMIBR (ORCPT ); Tue, 13 Oct 2015 04:01:17 -0400 Date: Tue, 13 Oct 2015 09:41:21 +0200 From: Borislav Petkov To: Tan Xiaojun Cc: dougthompson@xmission.com, mchehab@osg.samsung.com, linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] EDAC: use debugfs_remove_recursive instead of debugfs_remove Message-ID: <20151013074121.GA17498@pd.tnic> References: <1444702971-94112-1-git-send-email-tanxiaojun@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1444702971-94112-1-git-send-email-tanxiaojun@huawei.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Oct 13, 2015 at 10:22:51AM +0800, Tan Xiaojun wrote: > debugfs_remove is used to remove a file or an empty directory from > the debugfs filesystem, but mci->debugfs is not empty. How exactly do you trigger this? And by "this" I'm not even sure what the problem is as your commit message doesn't say. So please explain also what the problem is. Also, use this branch instead: http://git.kernel.org/cgit/linux/kernel/git/bp/bp.git/log/?h=for-next We did change EDAC debugfs handling recently so your patch doesn't apply anymore. Thanks. -- Regards/Gruss, Boris. ECO tip #101: Trim your mails when you reply.