This is the mail archive of the
ecos-patches@sourceware.org
mailing list for the eCos project.
Re: [PATCH] Speed-up sprintf() family of functions.
- From: Jonathan Larmour <jifl at eCosCentric dot com>
- To: Sergei Organov <osv at javad dot com>
- Cc: ecos-patches at ecos dot sourceware dot org
- Date: Mon, 01 Jan 2007 03:13:50 +0000
- Subject: Re: [PATCH] Speed-up sprintf() family of functions.
- References: <87fyb3x070.fsf@javad.com> <459529B0.8080407@eCosCentric.com> <87wt4allsq.fsf@javad.com> <4596F7EA.4090605@eCosCentric.com> <87slewl90j.fsf@javad.com>
Sergei Organov wrote:
Oh yes, I know there's a lot of things I would do very differently
given the chance and the benefit of 10 more years of experience. There
are many, many places the code could be improved (or rewritten!), and
that is one.
I think I need to experiment on this then. Hopefully this eCos part is
usually compiled without -fno-builtins?
Correct. The only times -fno-builtin is used normally is when we know GCC
is buggy in some area (and therefore usually only temporarily).
Hmm, that kernel change did seem big enough to need one in my view. I
think someone may have dropped the ball there. We would want an
assignment for that, irrespective of this libc stdio change.
What kernel change are you talking about?
It was the kernel change I referred to earlier in the thread. This one:
2006-04-10 Sergei Organov <osv@javad.com>
Implement FIFO variant of scheduling of DSRs and make it the
default. This is reworked patch originally suggested by Stefan
Sommerfeld <sommerfeld@mikrom.com>.
* cdl/interrupts.cdl (CYGIMP_KERNEL_INTERRUPTS_DSRS_LIST): make it
cdl_component.
* cdl/interrupts.cdl (CYGSEM_KERNEL_INTERRUPTS_DSRS_LIST_FIFO):
new option for CYGIMP_KERNEL_INTERRUPTS_DSRS_LIST.
* include/intr.hxx (class Cyg_Interrupt): new static variable
dsr_list_tail.
* src/intr/intr.cxx (call_pending_DSRs_inner): add
CYGSEM_KERNEL_INTERRUPTS_DSRS_LIST_FIFO variant.
(post_dsr): likewise.
* tests/intr0.cxx: fix comments to match actual option names.
* tests/kintr0.c: likewise.
Happy New Year!
Likewise to you and all!
Jifl
--
eCosCentric http://www.eCosCentric.com/ The eCos and RedBoot experts
Company legal info, address and number: http://www.ecoscentric.com/legal
------["The best things in life aren't things."]------ Opinions==mine