From owner-freebsd-stable@FreeBSD.ORG Fri Dec 8 14:11:05 2006 Return-Path: X-Original-To: freebsd-stable@freebsd.org Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 50C7916A415 for ; Fri, 8 Dec 2006 14:11:05 +0000 (UTC) (envelope-from freebsd-stable-local@be-well.ilk.org) Received: from mail6.sea5.speakeasy.net (mail6.sea5.speakeasy.net [69.17.117.8]) by mx1.FreeBSD.org (Postfix) with ESMTP id EED6643CA3 for ; Fri, 8 Dec 2006 14:10:06 +0000 (GMT) (envelope-from freebsd-stable-local@be-well.ilk.org) Received: (qmail 30898 invoked from network); 8 Dec 2006 14:11:04 -0000 Received: from dsl092-078-145.bos1.dsl.speakeasy.net (HELO be-well.ilk.org) ([66.92.78.145]) (envelope-sender ) by mail6.sea5.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 8 Dec 2006 14:11:04 -0000 Received: by be-well.ilk.org (Postfix, from userid 1147) id 45D3828430; Fri, 8 Dec 2006 09:11:03 -0500 (EST) To: Dmitry Pryanishnikov References: <4577EBA8.4000703@incunabulum.net> <20061208135634.Q7518@atlantis.atlantis.dp.ua> From: Lowell Gilbert Date: Fri, 08 Dec 2006 09:11:03 -0500 In-Reply-To: <20061208135634.Q7518@atlantis.atlantis.dp.ua> (Dmitry Pryanishnikov's message of "Fri, 8 Dec 2006 13:59:53 +0200 (EET)") Message-ID: <44zm9yxytk.fsf@be-well.ilk.org> User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (berkeley-unix) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: freebsd-stable@freebsd.org Subject: Re: Browser plugin problems still exist in 6.2 X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: freebsd-stable@freebsd.org List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Dec 2006 14:11:05 -0000 Dmitry Pryanishnikov writes: > Hello! > > On Thu, 7 Dec 2006, Bruce M Simpson wrote: >> It still seems to be necessary to patch rtld in order to get the >> Flash plugin to work (www/linux-flashplugin7) due to the '_dlsym' >> symbol not being found. >> >> I was able to use a smaller patch to do this, see attached (although >> this may not build with the Intel C compiler). >> +__strong_reference(dlsym, _dlsym); > > I wonder what's the difference between your one-line patch and one which > I'm using: > > +__weak_reference(dlsym, _dlsym); > > Which of them is more correct? The difference between the two is just that a weak reference can be overridden by code that links into the library. In this case, we know that the outside code (the nvidia driver) doesn't define that symbol at all -- therefore, there is no practical difference between the two. When in doubt, I always use a weak symbol.