From: Chuck Lever Subject: Re: recent failover-by-IP changes Date: Fri, 2 May 2008 17:35:47 -0400 Message-ID: References: <1B257A25-2B59-448F-B11C-637B8688D883@oracle.com> <20080502212608.GI21918@fieldses.org> Mime-Version: 1.0 (Apple Message framework v919.2) Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Cc: Wendy Cheng , Linux NFS Mailing List To: "J. Bruce Fields" Return-path: Received: from agminet01.oracle.com ([141.146.126.228]:12977 "EHLO agminet01.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760161AbYEBVhf (ORCPT ); Fri, 2 May 2008 17:37:35 -0400 In-Reply-To: <20080502212608.GI21918@fieldses.org> Sender: linux-nfs-owner@vger.kernel.org List-ID: On May 2, 2008, at 5:26 PM, J. Bruce Fields wrote: > On Fri, May 02, 2008 at 04:58:58PM -0400, Chuck Lever wrote: >> Hi Wendy- >> >> Looking at your recent lockd-failover-by-IP changes... I'd like to >> make >> sure I understand this logic before I merge it into my NLM IPv6 patch >> set. >> >> In fs/lockd/svcsubs.c: >>> static int >>> nlmsvc_match_ip(void *datap, struct nlm_host *host) >>> { >>> __be32 *server_addr = datap; >>> >>> return host->h_saddr.sin_addr.s_addr == *server_addr; >> >> h_saddr is the local host's source address, not the server address, >> and >> is used only on multi-interface systems. Is that what you wanted to >> compare, or did you mean ->h_addr? > > This is server-side code--h_saddr, last I checked, isn't even filled > in > on the client side. So the current host *is* the server. So this API is requesting that the local host should drop locks? Okay, that makes sense. It's not well documented in the code, though. -- Chuck Lever chuck[dot]lever[at]oracle[dot]com