From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:51528 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726499AbfABJi5 (ORCPT ); Wed, 2 Jan 2019 04:38:57 -0500 From: David Howells In-Reply-To: <20181226061856.GA2122@avx2> References: <20181226061856.GA2122@avx2> <20181226030642.72757-1-kjlu@umn.edu> To: Alexey Dobriyan Cc: dhowells@redhat.com, Kangjie Lu , pakki001@umn.edu, Andrew Morton , Davidlohr Bueso , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH] fs: proc: check status of register_filesystem MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-ID: <27135.1546421935.1@warthog.procyon.org.uk> Date: Wed, 02 Jan 2019 09:38:55 +0000 Message-ID: <27136.1546421935@warthog.procyon.org.uk> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Alexey Dobriyan wrote: > > - register_filesystem(&proc_fs_type); > > + if (register_filesystem(&proc_fs_type)) > > + pr_err("failed to register the filesystem.\n"); > > No, register_filesystem() should do it. +1 David