<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">2. 7. 2018 v 15:20, Stefano Zampini <<a href="mailto:stefano.zampini@gmail.com" class="">stefano.zampini@gmail.com</a>>:</div><br class="Apple-interchange-newline"><div class=""><br class="Apple-interchange-newline"><br style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><div class="gmail_quote" style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;">2018-07-02 11:48 GMT+03:00 Vaclav Hapla<span class="Apple-converted-space"> </span><span dir="ltr" class=""><<a href="mailto:vaclav.hapla@erdw.ethz.ch" target="_blank" class="">vaclav.hapla@erdw.ethz.ch</a>></span>:<br class=""><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word; line-break: after-white-space;" class=""><span class=""><div class="">Barry wrote:</div><div class=""><blockquote type="cite" class=""><span style="font-family: Menlo-Regular; float: none; display: inline !important;" class=""> This could get ugly real fast, for example, for vector operations, there may be dozens of named vectors and each one gets its own logging? You'd have to make sure that only the objects you care about get named, is that possible?</span><br style="font-family: Menlo-Regular;" class=""><br style="font-family: Menlo-Regular;" class=""><span style="font-family: Menlo-Regular; float: none; display: inline !important;" class=""> I don't know if there is a good solution within the PETSc logging infrastructure to get what you want but maybe what you propose is the best possible.</span><br style="font-family: Menlo-Regular;" class=""></blockquote><br class=""></div></span><div class="">As I suggest, this behavior would be only triggered by a specific option.</div><div class=""><br class=""></div><div class="">I think there are actually 4 strings which could be used as an event name suffix in log view:</div><div class="">1) name</div><div class="">2) prefix</div><div class="">3) type</div><div class="">4) custom string (set by something like PetscObjectSetLogViewSuffix)</div><div class="">I think the best would be to let user choose by offering -log_view_by_{name,prefix,type<wbr class="">,suffix}.</div><div class=""><br class=""></div><div class="">For example, with -log_view_by_prefix, you could readily distinguish PCTelescope outer and inner apply, because you would see a separate "PCApply (telescope_)" event.</div><div class="">With -log_view_by_type, you would see PCApply (telescope).</div><div class=""><br class=""></div><div class="">I think this would be useful because the current class-wide events like MatMult or PCApply aggregate very different operations from which some are for free and some form hotspots.</div><span class=""><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Stefano wrote:</div><div class=""><blockquote type="cite" class=""><div style="font-family: Menlo-Regular;" class="">The issue with this sort of “dynamic” logging is that now PETSc requires PetscLogEvent created during the registration of the class, so that all the ranks in PETSC_COMM_WORLD have the same events registered.</div><div style="font-family: Menlo-Regular;" class="">What you propose is not generally supported for this specific reason.</div><div style="font-family: Menlo-Regular;" class=""><br class=""></div><div style="font-family: Menlo-Regular;" class="">Your “log_name” may work if users register their own classes (with their own LogEvents created properly), and currently we don’t have support (maybe I’m wrong) to add an “InitializePackage” method for the users’ registered classes.</div></blockquote></div><div class=""><br class=""></div></span><div class="">I don't agree. What I suggest is basically an ability to allow automatically created object-wise events, so it _can't_ be managed during the class registration. In presence of respective option, the event would be created during PetscLogEventBegin by taking the class-wide event's name, concatenating the suffix and registering a new event. The event id would be stored in the PetscObject structure.</div><span class=""><div class=""><br class=""></div></span></div></blockquote><div class=""><br class=""></div><div class="">As I said before, now PETSc event model assumes the events are created at class registration.</div></div></div></blockquote><div><br class=""></div><div>I don't think there's such assumption. You can register an event where you want. There's nothing special about the class-wise events other that they are registered during *InitializePackage (which can be however called much later than PetscInitialize). They are otherwise by no means tied to that class.</div><div><br class=""></div><div>There are even examples which do that:</div><div><a href="http://www.mcs.anl.gov/petsc/petsc-dev/docs/manualpages/Profiling/PetscLogEventRegister.html" class="">http://www.mcs.anl.gov/petsc/petsc-dev/docs/manualpages/Profiling/PetscLogEventRegister.html</a></div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div class="gmail_quote" style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;"><div class="">I'm not saying your dynamic thing cannot be done; I'm saying it will likely deadlock</div></div></div></blockquote><div><br class=""></div><div>I don't see why this should lead to deadlock? With current class-wise events you can already have many simultaneous instances of the same event at once.</div><div><br class=""></div><div>Thanks</div><div><br class=""></div><div>Vaclav</div><br class=""><blockquote type="cite" class=""><div class=""><div class="gmail_quote" style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;"><div class=""> </div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-style: solid; border-left-color: rgb(204, 204, 204); padding-left: 1ex;"><div style="word-wrap: break-word; line-break: after-white-space;" class=""><span class=""><div class=""></div><div class=""><br class=""></div><div class="">Matt wrote:</div><div class=""><blockquote type="cite" class=""><div style="font-family: Menlo-Regular;" class="">As people have pointed out, this would not work well for Events. However, this is exactly what stages are for.</div><div style="font-family: Menlo-Regular;" class="">Use separate stages for the different types of MatMult. I did this, for example, when looking at performance</div><div style="font-family: Menlo-Regular;" class="">on different MG levels.</div></blockquote><br class=""></div></span><div class="">Yes, performance on different MG levels is a nice use case. I don't understand how you inject stages into MatMults. To me it's exactly the same problem as with events - you have to define MatMult_custom where you take the original mult and wrap into PetscStageLogPush/Pop and then use MatSetOperation to redefine MatMult. Or do you mean something more elegant?</div><div class=""><br class=""></div><div class="">Thanks</div><div class=""><br class=""></div><div class="">Vaclav</div><div class=""><br class=""></div><div class=""><br class=""></div><div class=""><br class=""><blockquote type="cite" class=""><div class="">29. 6. 2018 v 22:42, Smith, Barry F. <<a href="mailto:bsmith@mcs.anl.gov" target="_blank" class="">bsmith@mcs.anl.gov</a>>:</div><div class=""><div class="m_1953725660185562900h5"><br class="m_1953725660185562900m_8898091456720755839Apple-interchange-newline"><div class=""><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><blockquote type="cite" style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class="">On Jun 29, 2018, at 9:33 AM, Vaclav Hapla <<a href="mailto:vaclav.hapla@erdw.ethz.ch" target="_blank" class="">vaclav.hapla@erdw.ethz.ch</a>> wrote:<br class=""><br class=""><br class=""><br class=""><blockquote type="cite" class="">22. 6. 2018 v 17:47, Smith, Barry F. <<a href="mailto:bsmith@mcs.anl.gov" target="_blank" class="">bsmith@mcs.anl.gov</a>>:<br class=""><br class=""><br class=""><br class=""><blockquote type="cite" class="">On Jun 22, 2018, at 5:43 AM, Pierre Jolivet <<a href="mailto:pierre.jolivet@enseeiht.fr" target="_blank" class="">pierre.jolivet@enseeiht.fr</a>> wrote:<br class=""><br class="">Hello,<br class="">I’m solving a system using a MATSHELL and PCGAMG.<br class="">The MPIAIJ Mat I’m giving to GAMG has a specific structure (inherited from the MATSHELL) I’d like to exploit during the solution phase when the smoother on the finest level is doing MatMults.<br class=""><br class="">Is there some way to:<br class="">1) decouple in -log_view the time spent in the MATSHELL MatMult and in the smoothers MatMult<br class=""></blockquote><br class="">You can register a new event and then inside your MATSHELL MatMult() call PetscLogEventBegin/End on your new event.<br class=""><br class=""> Note that the MatMult() like will still contain the time for your MatShell mult so you will need to subtract it off to get the time for your non-shell matmults.<br class=""></blockquote><br class="">In PERMON, we sometimes have quite complicated hierarchy of wrapped matrices and want to measure MatMult{,Transpose,Add,Transpo<wbr class="">seAdd} separately for particular ones. Think e.g. of having additive MATCOMPOSITE wrapping multiplicative MATCOMPOSITE wrapping MATTRANSPOSE wrapping MATAIJ. You want to measure this MATAIJ instance's MatMult separately but you surely don't want to rewrite implementation of MatMult_Transpose or force yourself to use MATSHELL just to hang the events on MatMult*.<br class=""><br class="">We had a special wrapper type just adding some prefix to the events for the given object but this is not nice. What about adding a functionality to PetscLogEventBegin/End that would distinguish based on the first PetscObject's name or option prefix? Of course optionally not to break guys relying on current behavior - e.g. under something like -log_view_by_name. To me it's quite an elegant solution working for any PetscObject and any event.<br class=""></blockquote><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><span style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none; float: none; display: inline !important;" class=""> This could get ugly real fast, for example, for vector operations, there may be dozens of named vectors and each one gets its own logging? You'd have to make sure that only the objects you care about get named, is that possible?</span><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><span style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none; float: none; display: inline !important;" class=""> I don't know if there is a good solution within the PETSc logging infrastructure to get what you want but maybe what you propose is the best possible.</span><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><span style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none; float: none; display: inline !important;" class=""> Barry</span><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><br style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><blockquote type="cite" style="font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; text-decoration: none;" class=""><br class="">I can do that if I get some upvotes.<br class=""><br class="">Vaclav<br class=""><br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class="">2) hardwire a specific MatMult implementation for the smoother on the finest level<br class=""></blockquote><br class="">In the latest release you do MatSetOperation() to override the normal matrix vector product with anything else you want.<span class="m_1953725660185562900m_8898091456720755839Apple-converted-space"> </span><br class=""><br class=""><blockquote type="cite" class=""><br class="">Thanks in advance,<br class="">Pierre<br class=""><br class="">PS : here is what I have right now,<br class="">MatMult 118 1.0 1.0740e+02 1.6 1.04e+13 1.6 1.7e+06 6.1e+05 0.0e+00 47100 90 98 0 47100 90 98 0 81953703<br class="">[…]<br class="">PCSetUp 2 1.0 8.6513e+00 1.0 1.01e+09 1.7 2.6e+05 4.0e+05 1.8e+02 5 0 14 10 66 5 0 14 10 68 94598<br class="">PCApply 14 1.0 8.0373e+01 1.1 9.06e+12 1.6 1.3e+06 6.0e+05 2.1e+01 45 87 72 78 8 45 87 72 78 8 95365211 // I’m guessing a lot of time here is being wasted in doing inefficient MatMults on the finest level but this is only speculation<br class=""><br class="">Same code with -pc_type none -ksp_max_it 13,<br class="">MatMult 14 1.0 1.2936e+01 1.7 1.35e+12 1.6 2.0e+05 6.1e+05 0.0e+00 15100 78 93 0 15100 78 93 0 88202079<br class=""><br class="">The grid itself is rather simple (two levels, extremely aggressive coarsening),<br class="">type is MULTIPLICATIVE, levels=2 cycles=v<br class="">KSP Object: (mg_coarse_) 1024 MPI processes<br class="">linear system matrix = precond matrix:<br class=""> Mat Object: 1024 MPI processes<br class=""> type: mpiaij<br class=""> rows=775, cols=775<br class=""> total: nonzeros=1793, allocated nonzeros=1793<br class=""><br class="">linear system matrix followed by preconditioner matrix:<br class="">Mat Object: 1024 MPI processes<br class="">type: shell<br class="">rows=1369307136, cols=1369307136<br class="">Mat Object: 1024 MPI processes<br class="">type: mpiaij<br class="">rows=1369307136, cols=1369307136<br class="">total: nonzeros=19896719360, allocated nonzeros=19896719360</blockquote></blockquote></blockquote></div></div></div></blockquote></div><br class=""></div></blockquote></div><br style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br clear="all" style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><div style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><br class=""></div><span style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">--<span class="Apple-converted-space"> </span></span><br style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><div class="m_1953725660185562900gmail_signature" data-smartmail="gmail_signature" style="caret-color: rgb(0, 0, 0); font-family: Menlo-Regular; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;">Stefano</div></div></blockquote></div><br class=""></body></html>