<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Mar 3, 2013 at 2:14 PM, Richard Tran Mills <span dir="ltr"><<a href="mailto:rtm@eecs.utk.edu" target="_blank">rtm@eecs.utk.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>It works fine with the patch.  Thanks for your help!<br>
      <br>
      Regarding my question about having the DMShell object default to
      using DMShellGlobalToLocalBegin()/End() if a scatter context has
      been specified with DMShellSetGlobalToLocalVecScatter(), does that
      sound appropriate?  I think the thing to do is to set
      dm->ops->globaltolocalbegin to point to
      DMShellDefaultGlobalToLocalBegin when the DMShell is created, and
      make DMShellDefaultGlobalToLocalBegin check to see if a global to
      local VecScatter has been set and then complain (with a helpful
      error message) if not.</div></blockquote></div><br>That sounds fine to me. At some point, we should deal with the chaos of having DM options use a global SF when available. Aesthetically, I'd rather that "use SF for communication" be decoupled from "here is an SF", possibly to the extent that the SF be in DM_XX instead of _p_DM, but not strongly enough to change it now.</div>
</div>