Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/

> I added likely() and unlikely() to all tests, here are the results from 3 
> runs on my box : 

Any chance you could summarize what these results are, for those
of us too lazy to parse it all out?  The time spent by one author
to summarize in English what the numbers state can save the time of
a hundred readers each individually having to parse the numbers.

Just looking at the third run, it seems to me that "if (likely(p))
kfree(p);" beats a naked "kfree(p);" everytime, whether p is half
NULL's, or very few NULL's, or almost all NULL's.

If I'm reading this right, and if these results are valid, then we are
going about this optimization all wrong, at least if your CPU is an
AMD Athlon (T-bird). Weird.  Instead of stripping the "if (p)" test, we
should be changing it to "if (likely(p))", regardless of whether it
is very likely, or unlikely, or in between.  That is not what I would
call intuitive.

-- 
                  I won't rest till it's the best ...
                  Programmer, Linux Scalability
                  Paul Jackson <[email protected]> 1.650.933.1373, 1.925.600.0401
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
    • From: Jan Engelhardt <[email protected]>

• References:
  • [PATCH] no need to check for NULL before calling kfree() - fs/ext2/
    • From: Jesper Juhl <[email protected]>
  • Re: [PATCH] no need to check for NULL before calling kfree() - fs/ext2/
    • From: linux-os <[email protected]>
  • Re: [PATCH] no need to check for NULL before calling kfree() - fs/ext2/
    • From: Arjan van de Ven <[email protected]>
  • Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
    • From: linux-os <[email protected]>
  • Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
    • From: Jesper Juhl <[email protected]>
  • Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
    • From: Lee Revell <[email protected]>

• Prev by Date: Re: [RFC] Changes to the driver model class code.
• Next by Date: Re: [PATCH 6/7] - MPT FUSION - SPLITTING SCSI HOST DRIVERS
• Previous by thread: Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
• Next by thread: Re: [PATCH] no need to check for NULL before calling kfree() -fs/ext2/
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]