This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Speed-up sprintf() family of functions.


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]