[petsc-users] DMLocalToGlobal PETScSF is BROKEN

[Barry Smith]

On Jan 22, 2014, at 4:24 PM, Jed Brown <jed at jedbrown.org> wrote:

> Barry Smith <bsmith at mcs.anl.gov> writes:
> 
>>     Jed and Matt,
>> 
>> 
>>   But the PetscSF stuff has no way of managing multiple begin/begin
>>   in flight and no error checking to warn people against it.
> 
> SF holds a linked list of in-flight operations (indexed by the rootdata
> pointer), see PetscSFBasicGetPackInUse.  

    Ok, well the damn code is so convoluted it is hard to see that it is even trying to do things right for multiple independent calls.

    You’ll have to pry out of Dhamendar his test case that fails.   It must be using PetscSF because it cannot be using VecScatter or it would have ended with the nice error message.

> However, I don't see a specific
> test so it could have broken.  Note that VecScatter does not support
> this at all.

   Nope, it provides a nice error message and stops.

   Barry