<div dir="ltr">Nope, you are right.<div><br></div><div>  Thanks,</div><div><br></div><div>    Matt</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Dec 7, 2019 at 6:19 PM Balay, Satish <<a href="mailto:balay@mcs.anl.gov">balay@mcs.anl.gov</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">The fix for this is in 363424266cb675e6465b4c7dcb06a6ff8acf57d2<br>
<br>
Do you have this commit in your branch - and still seeing issues?<br>
<br>
Satish<br>
<br>
On Sat, 7 Dec 2019, Patrick Sanan wrote:<br>
<br>
> I was actually wondering about this, as in some cases valgrind errors appear and sometimes they don't, but I didn't dig into it too deeply.<br>
> <br>
> Here's my workaround, FWIW, which shows some output for that test on master.<br>
> <br>
> I don't see any output when I just run the tests like this:<br>
> <br>
>     VALGRIND=1 make -f gmakefile.test test globsearch="dm_impls_plex_tests-ex1_fluent_2"<br>
> <br>
> But I do see something if I do this to find any non-empty .err files:<br>
> <br>
>     find $PETSC_ARCH/tests -name *.err ! -size 0<br>
> <br>
> And then I see these valgrind warnings after copy-pasting the path:<br>
> <br>
> <br>
> $ cat arch-master-extra-opt/tests/dm/impls/plex/examples/tests/runex1_fluent_2/runex1_fluent_2.err<br>
> ==4990== Conditional jump or move depends on uninitialised value(s)<br>
> ==4990==    at 0x4C3705A: rawmemchr (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)<br>
> ==4990==    by 0x7687351: _IO_str_init_static_internal (strops.c:41)<br>
> ==4990==    by 0x767878C: vsscanf (iovsscanf.c:40)<br>
> ==4990==    by 0x76721A3: sscanf (sscanf.c:32)<br>
> ==4990==    by 0x588147D: DMPlexCreateFluent_ReadSection (plexfluent.c:105)<br>
> ==4990==    by 0x5882E3A: DMPlexCreateFluent (plexfluent.c:246)<br>
> ==4990==    by 0x588492F: DMPlexCreateFluentFromFile (plexfluent.c:30)<br>
> ==4990==    by 0x577E2B7: DMPlexCreateFromFile (plexcreate.c:3254)<br>
> ==4990==    by 0x10BEDF: CreateMesh (ex1.c:170)<br>
> ==4990==    by 0x10A20B: main (ex1.c:430)<br>
> ==4990==  Uninitialised value was created by a stack allocation<br>
> ==4990==    at 0x5881313: DMPlexCreateFluent_ReadSection (plexfluent.c:96)<br>
> ==4990==<br>
> ==4990== Use of uninitialised value of size 8<br>
> ==4990==    at 0x766276F: _IO_vfscanf (vfscanf.c:633)<br>
> ==4990==    by 0x767879C: vsscanf (iovsscanf.c:41)<br>
> ==4990==    by 0x76721A3: sscanf (sscanf.c:32)<br>
> ==4990==    by 0x588147D: DMPlexCreateFluent_ReadSection (plexfluent.c:105)<br>
> ==4990==    by 0x5882E3A: DMPlexCreateFluent (plexfluent.c:246)<br>
> ==4990==    by 0x588492F: DMPlexCreateFluentFromFile (plexfluent.c:30)<br>
> ==4990==    by 0x577E2B7: DMPlexCreateFromFile (plexcreate.c:3254)<br>
> ==4990==    by 0x10BEDF: CreateMesh (ex1.c:170)<br>
> ==4990==    by 0x10A20B: main (ex1.c:430)<br>
> ==4990==  Uninitialised value was created by a stack allocation<br>
> ==4990==    at 0x5881313: DMPlexCreateFluent_ReadSection (plexfluent.c:96)<br>
> ==4990==<br>
> ==4990== Conditional jump or move depends on uninitialised value(s)<br>
> ==4990==    at 0x766277B: _IO_vfscanf (vfscanf.c:630)<br>
> ==4990==    by 0x767879C: vsscanf (iovsscanf.c:41)<br>
> ==4990==    by 0x76721A3: sscanf (sscanf.c:32)<br>
> ==4990==    by 0x588147D: DMPlexCreateFluent_ReadSection (plexfluent.c:105)<br>
> ==4990==    by 0x5882E3A: DMPlexCreateFluent (plexfluent.c:246)<br>
> ==4990==    by 0x588492F: DMPlexCreateFluentFromFile (plexfluent.c:30)<br>
> ==4990==    by 0x577E2B7: DMPlexCreateFromFile (plexcreate.c:3254)<br>
> ==4990==    by 0x10BEDF: CreateMesh (ex1.c:170)<br>
> ==4990==    by 0x10A20B: main (ex1.c:430)<br>
> ==4990==  Uninitialised value was created by a stack allocation<br>
> ==4990==    at 0x5881313: DMPlexCreateFluent_ReadSection (plexfluent.c:96)<br>
> ==4990==<br>
> > Am 07.12.2019 um 21:45 schrieb Matthew Knepley <<a href="mailto:knepley@gmail.com" target="_blank">knepley@gmail.com</a>>:<br>
> > <br>
> > I am trying to clean up valgrind errors. However this one<br>
> > <br>
> >   dm_impls_plex_tests-ex1_fluent_2<br>
> > <br>
> > is valgrind clean on my machine. Does anyone get it to output something?<br>
> > <br>
> >   Thanks,<br>
> > <br>
> >      Matt<br>
> > <br>
> > -- <br>
> > What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>
> > -- Norbert Wiener<br>
> > <br>
> > <a href="https://www.cse.buffalo.edu/~knepley/" rel="noreferrer" target="_blank">https://www.cse.buffalo.edu/~knepley/</a> <<a href="http://www.cse.buffalo.edu/~knepley/" rel="noreferrer" target="_blank">http://www.cse.buffalo.edu/~knepley/</a>><br>
> <br>
> <br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>What most experimenters take for granted before they begin their experiments is infinitely more interesting than any results to which their experiments lead.<br>-- Norbert Wiener</div><div><br></div><div><a href="http://www.cse.buffalo.edu/~knepley/" target="_blank">https://www.cse.buffalo.edu/~knepley/</a><br></div></div></div></div></div></div></div>