From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wanlong Gao Subject: Re: [PATCH 4/4] module: Use the binary search for symbols resolution Date: Tue, 19 Apr 2011 09:44:20 +0800 Message-ID: <4DACE8F4.2050807@gmail.com> References: <1302960373-5309-1-git-send-email-abogani@kernel.org> <1302960373-5309-5-git-send-email-abogani@kernel.org> <87liz73tso.fsf@rustcorp.com.au> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type :content-transfer-encoding; bh=M9QCad8MuRodQJY57QJ10Ok+ROMymN2Ba7ejb7+TaY8=; b=OaBQwysQD1x8qTrF4eqXenw5lXSByd/pGkD4fchApGUcNcbZPJVcjcqMUR4nlGjgAU Q551FocoYGUsgJjnvNfgnVsnMQM6q/s/7wpuRFN3/gXt82RQOzHoHaB4FfDXuXSM95a4 wDs5APv/CLJBhZdABZawXSbw46hVMUVlmr4M0= In-Reply-To: <87liz73tso.fsf@rustcorp.com.au> Sender: linux-kernel-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Rusty Russell Cc: Alessio Igor Bogani , Tim Abbott , Anders Kaseorg , Jason Wessel , Tim Bird , LKML , Linux Embedded 2011-4-19 9:37, Rusty Russell wroted: > On Sat, 16 Apr 2011 22:32:08 +0800, Wanlong Gao wrote: >>> + sym = bsearch(fsa->name, syms->start, syms->stop - syms->start, >> As the bsearch func, why not change the syms->stop to syms->end ? >> Hmm..Just a stupid suggestion. > > > The names are derived from the linker symbols for end of sections, which > is __stop_. > > Cheers, > Rusty. As this , why not change the bsearch's "end" to "stop", too ? It will be more readable. Thanks Wanlong