This message generated a parse failure. Raw output follows here. Please use 'back' to navigate. From devnull@lkml.org Sat May 11 18:27:00 2024 Envelope-to: j@jasper.es Delivery-date: Mon, 13 May 2013 21:46:42 +0200 Received: from localhost ([127.0.0.1] helo=squeeze.vs19.net) by squeeze.vs19.net with esmtp (Exim 4.72) (envelope-from ) id 1Ubyhl-0003iV-T0 for j@jasper.es; Mon, 13 May 2013 21:46:42 +0200 Original-Recipient: rfc822;jasper@telfort.nl Received: from pop3.telfort.nl [213.75.3.52] by squeeze.vs19.net with POP3 (fetchmail-6.3.18) for (single-drop); Mon, 13 May 2013 21:46:41 +0200 (CEST) Received: from cpxmta-msg05.kpnxchange.com (10.94.114.26) by cpxmbs-msg01.support.local (8.6.060.17) id 516DA660004F0150 for jasper@telfort.nl; Mon, 13 May 2013 21:46:37 +0200 Received: from cpsmtpb-ews01.kpnxchange.com (213.75.39.4) by cpxmta-msg05.kpnxchange.com (8.6.060.14) id 516576770090B83C for jasper@telfort.nl; Mon, 13 May 2013 21:46:37 +0200 Received: from cpsps-ews01.kpnxchange.com ([10.94.84.168]) by cpsmtpb-ews01.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Mon, 13 May 2013 21:46:37 +0200 Received: from vger.kernel.org ([209.132.180.67]) by cpsps-ews01.kpnxchange.com with Microsoft SMTPSVC(7.5.7601.17514); Mon, 13 May 2013 21:46:37 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755015Ab3EMTqc (ORCPT ); Mon, 13 May 2013 15:46:32 -0400 Received: from merlin.infradead.org ([205.233.59.134]:52775 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751299Ab3EMTqb (ORCPT ); Mon, 13 May 2013 15:46:31 -0400 Received: from static-50-53-38-135.bvtn.or.frontiernet.net ([50.53.38.135] helo=dragon.site) by merlin.infradead.org with esmtpsa (Exim 4.80.1 #2 (Red Hat Linux)) id 1UbyhX-000591-S1; Mon, 13 May 2013 19:46:27 +0000 Message-Id: <519142E7.4080009@infradead.org> Date: Mon, 13 May 2013 12:45:43 -0700 From: Randy Dunlap User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130329 Thunderbird/17.0.5 Mime-Version: 1.0 To: Steven Whitehouse , David Teigland Cc: Stephen Rothwell , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, cluster-devel@redhat.com Subject: Re: linux-next: Tree for May 8 (dlm) References: <20130508140122.e4747b58be4333060b7a248a@canb.auug.org.au> <518A93BD.3020300@infradead.org> <20130509094745.049e46fff25f357e7eb50123@canb.auug.org.au> <20130509165028.GA18077@redhat.com> <518BD826.4040403@infradead.org> <1368436712.2882.14.camel@menh In-Reply-To: <51914045.1060900@infradead.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-Id: X-Mailing-List: linux-kernel@vger.kernel.org X-OriginalArrivalTime: 13 May 2013 19:46:37.0641 (UTC) FILETIME=[94CCDB90:01CE5012] X-RcptDomain: telfort.nl [resending since mail server dropped it] On 05/13/13 12:34, Randy Dunlap wrote: > On 05/13/13 12:31, Randy Dunlap wrote: >> On 05/13/13 09:30, Randy Dunlap wrote: >>> On 05/13/13 02:18, Steven Whitehouse wrote: >>>> Hi, >>>> >>>> On Thu, 2013-05-09 at 10:08 -0700, Randy Dunlap wrote: >>>>> On 05/09/13 09:50, David Teigland wrote: >>>>>> On Thu, May 09, 2013 at 09:47:45AM +1000, Stephen Rothwell wrote: >>>>>>> [Just forwarding to David ...] >>>>>>> >>>>>>> On Wed, 08 May 2013 11:04:45 -0700 Randy Dunlap wrote: >>>>>>>> >>>>>>>> on x86_64: >>>>>>>> >>>>>>>> when CONFIG_GFS2_FS_LOCKING_DLM=y and CONFIG_DLM=m: >>>>>>>> >>>>>>>> fs/built-in.o: In function `gfs2_lock': >>>>>>>> file.c:(.text+0xa512c): undefined reference to `dlm_posix_get' >>>>>>>> file.c:(.text+0xa5140): undefined reference to `dlm_posix_unlock' >>>>>>>> file.c:(.text+0xa514a): undefined reference to `dlm_posix_lock' >>>>>> >>>>>> gfs2/file.c calls the dlm directly, so I suppose gfs2 itself needs >>>>>> to depend on the dlm. It's been like this for a long time, so I >>>>>> don't know why it only appeared now. >>>>> >>>>> Agreed to both statements. >>>>> >>>>>>>> fs/built-in.o: In function `gdlm_cancel': >>>>>>>> lock_dlm.c:(.text+0xb3f57): undefined reference to `dlm_unlock' >>>>>>>> fs/built-in.o: In function `gdlm_unmount': >>>>>>>> lock_dlm.c:(.text+0xb40ff): undefined reference to `dlm_release_lockspace' >>>>>>>> fs/built-in.o: In function `sync_unlock.isra.4': >>>>>>>> lock_dlm.c:(.text+0xb420d): undefined reference to `dlm_unlock' >>>>>>>> fs/built-in.o: In function `sync_lock.isra.5': >>>>>>>> lock_dlm.c:(.text+0xb42d9): undefined reference to `dlm_lock' >>>>>>>> fs/built-in.o: In function `gdlm_put_lock': >>>>>>>> lock_dlm.c:(.text+0xb45e7): undefined reference to `dlm_unlock' >>>>>>>> fs/built-in.o: In function `gdlm_mount': >>>>>>>> lock_dlm.c:(.text+0xb4928): undefined reference to `dlm_new_lockspace' >>>>>>>> lock_dlm.c:(.text+0xb4c75): undefined reference to `dlm_release_lockspace' >>>>>>>> fs/built-in.o: In function `gdlm_lock': >>>>>>>> lock_dlm.c:(.text+0xb529f): undefined reference to `dlm_lock' >>>>>> >>>>>> lock_dlm.c is GFS2_FS_LOCKING_DLM which depends on DLM. >>>>>> Is that not correct? >>>>> >>>>> The problem is that GFS2_FS_LOCKING_DLM is a bool. It depends on DLM, >>>>> which is a tristate with a value of 'm', so the bool is true (as long >>>>> as DLM != 'n'). >>>>> >>>>> One option is to make GFS2_FS_LOCKING_DLM depend on "DLM != n", but a >>>>> better fix is to make GFS2_FS depend on DLM, like you said above. >>>>> >>>>> >>>> >>>> Does this look correct? As Dave says this has not changed for some time. >>>> It seems that every time we try to get this right, there is always some >>>> corner case that is missed :( >>> >>> Sorry, I misspoke above. It will have to depend on DLM=y since DLM=m >>> is what is causing these build errors. >> >> and that is too strict. It needs to allow for both dlm and gfs2 built as >> loadable modules. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ >>>> We can't make GFS2_FS depend on DLM as otherwise there would be no >>>> reason to have GFS2_FS_LOCKING_DLM, at least if I've understood the >>>> issue here correctly. So I've come up with the following... does it look >>>> ok? >>>> >>>> >>>> diff --git a/fs/gfs2/Kconfig b/fs/gfs2/Kconfig >>>> index eb08c9e..edbad96 100644 >>>> --- a/fs/gfs2/Kconfig >>>> +++ b/fs/gfs2/Kconfig >>>> @@ -26,7 +26,7 @@ config GFS2_FS >>>> config GFS2_FS_LOCKING_DLM >>>> bool "GFS2 DLM locking" >>>> depends on (GFS2_FS!=n) && NET && INET && (IPV6 || IPV6=n) && \ >>>> - HOTPLUG && DLM && CONFIGFS_FS && SYSFS >>>> + HOTPLUG && (DLM!=n) && CONFIGFS_FS && SYSFS >>> >>> HOTPLUG && DLM=y && CONFIGFS_FS && SYSFS >> >> HOTPLUG && CONFIGFS_FS && SYSFS && (DLM=y || DLM=GFS2_FS) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ >> I think. > > tested and works AFAICT. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ >>> >>>> help >>>> Multiple node locking module for GFS2 >>>> -- ~Randy -- 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/