<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
On Jan 25, 2012, at 11:53 AM, Satish Balay wrote:<br>
<br>
> looks like this piece of code was last changed at:<br>
><br>
> <a href="http://petsc.cs.iit.edu/petsc/petsc-dev/rev/83c325962928" target="_blank">http://petsc.cs.iit.edu/petsc/petsc-dev/rev/83c325962928</a><br>
><br>
> the new code added was 'PCSetUp_BJacobi_Multiproc' - different than 'PCSetUp_BJacobi_Multiblock'<br>
<br>
</div>   So likely the multiproc code broke the singeblock/multiblock stuff<br>
<br>
   Hong can you check this please?<br></blockquote><div>Fixed and pushed.</div><div>Hong </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
   Barry<br>
</font></span><div class="HOEnZb"><div class="h5">> Satish<br>
><br>
> On Wed, 25 Jan 2012, Barry Smith wrote:<br>
><br>
>><br>
>>   In PCSetUp_BJacobi() it has at the end<br>
>><br>
>><br>
>>     /* ------<br>
>>     Setup code depends on the number of blocks<br>
>>  */<br>
>>  if (jac->n_local == 0) {<br>
>>    ierr = PCSetUp_BJacobi_Singleblock(pc,mat,pmat);CHKERRQ(ierr);<br>
>>  } else {<br>
>>    ierr = PCSetUp_BJacobi_Multiblock(pc,mat,pmat);CHKERRQ(ierr);<br>
>>  }<br>
>><br>
>> does this make sense? No. In the normal case it is not using PCSetUp_BJacobi_Singleblock() at all! Something has gone wrong in the logic of the code when multiblock was added.<br>
>><br>
>>    Barry<br>
>><br>
>><br>
><br>
<br>
</div></div></blockquote></div><br>