From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([65.50.211.133]:60449 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750954AbdGMHPj (ORCPT ); Thu, 13 Jul 2017 03:15:39 -0400 Date: Thu, 13 Jul 2017 00:15:33 -0700 From: Christoph Hellwig To: Jay Lan Cc: linux-fsdevel@vger.kernel.org Subject: Re: [PATCH] export symbol mount_lock Message-ID: <20170713071533.GA25560@infradead.org> References: <855c5c7d-586a-3b35-76f8-ddca4fae1410@nasa.gov> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <855c5c7d-586a-3b35-76f8-ddca4fae1410@nasa.gov> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Jul 12, 2017 at 02:40:11PM -0700, Jay Lan wrote: > Hi all, > > I was writing a livepatch patch, but compilation failed in creating the .ko > because mount_lock is "undefined." > > The mount_lock is defined globally in fs/namespace.c > __cacheline_aligned_in_smp DEFINE_SEQLOCK(mount_lock); > > and extern in fs/mount.h. > extern seqlock_t mount_lock; > > If there is a reason that mount_lock should not be EXPORT_SYMBOL_GPL, please > advise; otherwise, I propose the patch to export this symbol. Because there is no %$^$^ reason to use it from a module ever?