[petsc-users] memory scalable AO

[Satish Balay]

Thanks! pushed to maint - and merged to master.

Satish

On Thu, 18 Feb 2016, Randall Mackie wrote:

> Hi Satish,
> 
> This patch works now.
> 
> Thanks very much to all of you for your patience and help.
> 
> Randy
> 
> 
> > On Feb 17, 2016, at 9:58 PM, Satish Balay <balay at mcs.anl.gov> wrote:
> > 
> > Look like zdaindexf.c is not listed in makefile - so it never gets compiled.
> > 
> > Also it has code for dmdagetglobalindices_() dmdarestoreglobalindices_(). But then
> > src/docs/website/documentation/changes/35.html has:
> > 
> >>>>> 
> > <li>DMDAGetGlobalIndices(DM,PetscInt*,const PetscInt*[]) and DMDARestoreGlobalIndices(DM,PetscInt*,const PetscInt*[]) are removed, use DMGetLocalToGlobalMapping() to get this information</li>
> > <<<<
> > 
> > So perhaps the attached modified patch should be used? [instead of the previous fix-fortran-dmdasetaotype.patch]
> > 
> > patch -Np1 -R < fix-fortran-dmdasetaotype.patch
> > patch -Np1 < dmdasetaotype-fixed.patch
> > 
> > Might be easier to use 'maint' branch from git..
> > 
> > Satish
> > 
> > On Wed, 17 Feb 2016, Barry Smith wrote:
> > 
> >> 
> >>  Randy,
> >> 
> >>  It is not suppose to be this hard!  When you ran the make gnumake after you applied the patch did it recompile the file zdaindexf.c (in fact that file and daindexf.c should be the only files that changed and hence the only files that got recompiled).  
> >> 
> >>  Barry
> >> 
> >> 
> >> 
> >>> On Feb 17, 2016, at 11:41 PM, Randall Mackie <rlmackie862 at gmail.com> wrote:
> >>> 
> >>> Satish, 
> >>> 
> >>> It is unclear from this email exchange if I am still suppose to do make allfortranstubs after updating these two files, or if these are the files that are suppose to result from that make operation.
> >>> 
> >>> Although these files looked the same as what I tried before with Barry’s suggestions, I dropped these in, recompiled petsc, but again, when I try to compile the test code I sent, it still complains about an unresolved symbol to dmdasetaotype_.
> >>> 
> >>> If I still need to do the make allfortranstubs, I’ll do that via a git clone tomorrow.
> >>> 
> >>> Thanks, Randy
> >>> 
> >>> 
> >>>> On Feb 17, 2016, at 8:54 PM, Satish Balay <balay at mcs.anl.gov> wrote:
> >>>> 
> >>>> Attached are the modified src/dm/impls/da/ftn-auto/daindexf.c and
> >>>> src/dm/impls/da/ftn-custom/zdaindexf.c files.
> >>>> 
> >>>> Satish
> >>>> 
> >>>> On Wed, 17 Feb 2016, Jed Brown wrote:
> >>>> 
> >>>>> Randall Mackie <rlmackie862 at gmail.com> writes:
> >>>>> 
> >>>>>> this leads to the error ‘bin/maint/generatefortranstubs.py’ …No such file.
> >>>>>> 
> >>>>>> there is no maint directory under bin.
> >>>>> 
> >>>>> Ah, I missed that you have a tarball, not a clone of the repository.
> >>>>> Then you'll have to fix the file manually.  I would start from a clean
> >>>>> tarball because we have no idea what state your source tree is in after
> >>>>> applying the match and manually editing files.  Note that this would be
> >>>>> easy and completely reliable if you used a clone instead of tarball.
> >>>>> 
> >>>> <daindexf.c><zdaindexf.c>
> >>> 
> >> 
> > <dmdasetaotype-fixed.patch>
> 
>