<p dir="ltr">That's gonna suck. Karl, can you apply his patch to the old code, run uncrustify on it, then send out the diff (which should apply cleanly to head).</p>
<div class="gmail_quote">On Feb 1, 2013 6:32 PM, "Karl Rupp" <<a href="mailto:rupp@mcs.anl.gov">rupp@mcs.anl.gov</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Paul,<br>
<br>
I just uncrustified src/mat/impls/aij/* and pushed it to petsc-dev. Could you please re-generate your patch based on the latest commit?<br>
<br>
Thanks and best regards,<br>
Karli<br>
<br>
<br>
On 02/01/2013 06:11 PM, Paul Mullowney wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
Here's a reworked patch for running BiCG on GPUs (with ILU(0)<br>
preconditioners) on GPUs for the <a href="http://aijcusparse.cu" target="_blank">aijcusparse.cu</a> class. I fixed the<br>
comments from the previous emails on this patch. In particular, I added<br>
<br>
(1) VecConjugate implementation in <a href="http://veccusp.cu" target="_blank">veccusp.cu</a> with the correct method<br>
for getting the device ptr (VecCUSPGetArrayReadWrite()).<br>
(2) Various methods in <a href="http://aijcusparse.cu" target="_blank">aijcusparse.cu</a> for building the transpose<br>
matrices for MatSolveTranspose* methods. The implementation of the<br>
solves is done under the hood in the txpetscgpu library. A protection<br>
was added to ensure the matrix generation routines are only called once.<br>
(3) I fixed the uninitialized compiler warning when building in double<br>
complex. This required a slight fix in VecCUSPGetArrayWrite().<br>
(4) Small Style fixes.<br>
<br>
I wasn't clear to me how to break this up patch into a small<br>
organizational patch and then a large implementation patch. If you have<br>
suggestions on what corresponds to organization and what corresponds to<br>
implementation, I can try to do that in subsequent patches.<br>
<br>
Everything builds and runs fine on my end.<br>
<br>
Thanks,<br>
-Paul<br>
</blockquote>
<br>
</blockquote></div>