mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] Including valgrind in the configuration doesn't trigger an un-optimized build of glibc.
Date: Thu, 29 Mar 2018 11:20:53 +0200	[thread overview]
Message-ID: <20180329092053.fcnigz4wqs3p2dsh@pengutronix.de> (raw)
In-Reply-To: <2D79272AFFD75A4392ED2D3EF16D3B53AE5C30@mdcxch20.na.ops.local>

On Wed, Mar 28, 2018 at 10:43:14AM -0500, Crim, Jason wrote:
> Thanks for your information.  My setup includes glibc 2.20 rather than the
> 2.16 referenced in your patch file, so I don't think it'll work directly (the
> source of 2.20 and 2.16 look pretty different - actually, I'm not even seeing
> the files referenced in the 2.16 source I pulled from the glibc archive
> today), but perhaps it'll help me find the answers I need.
> 
> My results from readelf on libc.so.6 look a bit different:
> >		 14499: 00058f21     0 FUNC    GLOBAL DEFAULT   11 memcpy
> >		 12773: 00056f11   740 FUNC    GLOBAL DEFAULT   11 strcmp
> >		 14536: 00057701   220 FUNC    GLOBAL DEFAULT   11 strlen
> 
> there are no "FUNC	LOCAL	DEFAULT" for the base functions - though memcpy does have a size of 0 here.
> 
> My ld-2.20.so (which is the file valgrind referenced) yeilds similar results to yours:
> >		   916: 00012921     0 FUNC    LOCAL  DEFAULT   10 memcpy
> >		  1033: 00011b31   740 FUNC    LOCAL  DEFAULT   10 strcmp
> >		  1083: 00012041   220 FUNC    LOCAL  DEFAULT   10 strlen
> 		  
> with the exception of strcmp and strlen.  Which makes sense given the
> valgrind output in my last message.  Replacing ld-2.20.so moved the error
> from strcmp to memcpy.
> 
> I'm actually seeing similar entries ('0 FUNC    LOCAL  DEFAULT') for memcpy,
> memchr, index, memset, and strchr in ld-2.20.so.

If you use the cortexa8 toolchain, the you can avoid the problem mentioned
by Andrej by switching to the v7a toolchain. It optimizes a bit differently
but should work for you.

Michael

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2018-03-29  9:20 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-28 10:01 Andrej.Gantvorg
2018-03-28 15:43 ` Crim, Jason
2018-03-29  9:20   ` Michael Olbrich [this message]
2018-04-05 19:41     ` Crim, Jason
2018-04-09  8:41       ` Michael Olbrich
2018-04-09 12:13         ` Crim, Jason
2018-04-09 13:49           ` Michael Olbrich
  -- strict thread matches above, loose matches on Subject: below --
2018-03-22 17:38 Crim, Jason
2018-03-26  6:30 ` Michael Olbrich
2018-03-26 12:03   ` Crim, Jason
2018-03-27  7:33     ` Michael Olbrich
2018-03-27 11:54       ` Crim, Jason

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180329092053.fcnigz4wqs3p2dsh@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox