]> https://gitweb.dealii.org/ - dealii.git/commitdiff
handle self-assignment in reinit() and copy_from()
authorTimo Heister <timo.heister@gmail.com>
Mon, 12 Jan 2015 16:38:59 +0000 (11:38 -0500)
committerTimo Heister <timo.heister@gmail.com>
Mon, 12 Jan 2015 18:11:12 +0000 (13:11 -0500)
also include a test

source/lac/trilinos_sparse_matrix.cc
tests/gla/mat_06.cc [new file with mode: 0644]
tests/gla/mat_06.mpirun=3.output [new file with mode: 0644]

index 0cdafb5eb203a43766b39e720e71ece08d1569d5..292fdfea62e6f29c8b67a045a632df0dd0d1f7b6 100644 (file)
@@ -378,6 +378,9 @@ namespace TrilinosWrappers
   void
   SparseMatrix::copy_from (const SparseMatrix &rhs)
   {
+    if (this == &rhs)
+      return;
+
     nonlocal_matrix.reset();
     nonlocal_matrix_exporter.reset();
 
@@ -794,6 +797,9 @@ namespace TrilinosWrappers
   void
   SparseMatrix::reinit (const SparseMatrix &sparse_matrix)
   {
+    if (this == &sparse_matrix)
+      return;
+
     column_space_map.reset (new Epetra_Map (sparse_matrix.domain_partitioner()));
     matrix.reset ();
     nonlocal_matrix_exporter.reset();
diff --git a/tests/gla/mat_06.cc b/tests/gla/mat_06.cc
new file mode 100644 (file)
index 0000000..6b2cea3
--- /dev/null
@@ -0,0 +1,110 @@
+// ---------------------------------------------------------------------
+//
+// Copyright (C) 2004 - 2013 by the deal.II authors
+//
+// This file is part of the deal.II library.
+//
+// The deal.II library is free software; you can use it, redistribute
+// it, and/or modify it under the terms of the GNU Lesser General
+// Public License as published by the Free Software Foundation; either
+// version 2.1 of the License, or (at your option) any later version.
+// The full text of the license can be found in the file LICENSE at
+// the top level of the deal.II distribution.
+//
+// ---------------------------------------------------------------------
+
+
+
+// check reinit and copy_from self-assignment for LA::MPI::SparseMatrix
+
+#include "../tests.h"
+#include <deal.II/lac/generic_linear_algebra.h>
+#include <deal.II/base/index_set.h>
+#include <fstream>
+#include <iostream>
+#include <iomanip>
+#include <vector>
+
+#include "gla.h"
+
+template <class LA>
+void test ()
+{
+  unsigned int myid = Utilities::MPI::this_mpi_process (MPI_COMM_WORLD);
+  unsigned int numproc = Utilities::MPI::n_mpi_processes (MPI_COMM_WORLD);
+
+  if (myid==0)
+    deallog << "numproc=" << numproc << std::endl;
+
+  // each processor owns 2 indices and all
+  // are ghosting Element 1 (the second)
+
+  IndexSet local_active(numproc*2);
+  local_active.add_range(myid*2,myid*2+2);
+  IndexSet local_relevant= local_active;
+  local_relevant.add_range(0,1);
+
+  CompressedSimpleSparsityPattern csp (local_relevant);
+
+  for (unsigned int i=0; i<2*numproc; ++i)
+    if (local_relevant.is_element(i))
+      csp.add(i,i);
+
+  csp.add(0,1);
+
+
+  typename LA::MPI::SparseMatrix mat;
+  mat.reinit (local_active, local_active, csp, MPI_COMM_WORLD);
+
+  Assert(mat.n()==numproc*2, ExcInternalError());
+  Assert(mat.m()==numproc*2, ExcInternalError());
+
+  // set local values
+  mat.set(myid*2,myid*2, myid*2.0);
+  mat.set(myid*2+1,myid*2+1, myid*2.0+1.0);
+
+  mat.compress(VectorOperation::insert);
+
+  mat.add(0,1,1.0);
+
+  mat.compress(VectorOperation::add);
+
+  deallog << "l1-norm: " << mat.l1_norm() << std::endl;
+  if (myid==0)
+    deallog << "mat(0,1): " << mat(0,1) << std::endl;
+
+  mat.copy_from(mat);
+  deallog << "l1-norm: " << mat.l1_norm() << std::endl;
+  if (myid==0)
+    deallog << "mat(0,1): " << mat(0,1) << std::endl;
+
+  mat.reinit(mat);
+  deallog << "l1-norm: " << mat.l1_norm() << std::endl;
+  if (myid==0)
+    deallog << "mat(0,1): " << mat(0,1) << std::endl;
+  
+  if (myid==0)
+    deallog << "OK" << std::endl;
+}
+
+
+
+int main (int argc, char **argv)
+{
+  Utilities::MPI::MPI_InitFinalize mpi_initialization(argc, argv, 1);
+  MPILogInitAll log;
+  {
+    deallog.push("PETSc");
+    test<LA_PETSc>();
+    deallog.pop();
+    deallog.push("Trilinos");
+    test<LA_Trilinos>();
+    deallog.pop();
+  }
+
+  // compile, don't run
+  //if (myid==9999)
+  //  test<LA_Dummy>();
+
+
+}
diff --git a/tests/gla/mat_06.mpirun=3.output b/tests/gla/mat_06.mpirun=3.output
new file mode 100644 (file)
index 0000000..d2e2efe
--- /dev/null
@@ -0,0 +1,33 @@
+
+DEAL:0:PETSc::numproc=3
+DEAL:0:PETSc::l1-norm: 5.00000
+DEAL:0:PETSc::mat(0,1): 3.00000
+DEAL:0:PETSc::l1-norm: 5.00000
+DEAL:0:PETSc::mat(0,1): 3.00000
+DEAL:0:PETSc::l1-norm: 5.00000
+DEAL:0:PETSc::mat(0,1): 3.00000
+DEAL:0:PETSc::OK
+DEAL:0:Trilinos::numproc=3
+DEAL:0:Trilinos::l1-norm: 5.00000
+DEAL:0:Trilinos::mat(0,1): 3.00000
+DEAL:0:Trilinos::l1-norm: 5.00000
+DEAL:0:Trilinos::mat(0,1): 3.00000
+DEAL:0:Trilinos::l1-norm: 5.00000
+DEAL:0:Trilinos::mat(0,1): 3.00000
+DEAL:0:Trilinos::OK
+
+DEAL:1:PETSc::l1-norm: 5.00000
+DEAL:1:PETSc::l1-norm: 5.00000
+DEAL:1:PETSc::l1-norm: 5.00000
+DEAL:1:Trilinos::l1-norm: 5.00000
+DEAL:1:Trilinos::l1-norm: 5.00000
+DEAL:1:Trilinos::l1-norm: 5.00000
+
+
+DEAL:2:PETSc::l1-norm: 5.00000
+DEAL:2:PETSc::l1-norm: 5.00000
+DEAL:2:PETSc::l1-norm: 5.00000
+DEAL:2:Trilinos::l1-norm: 5.00000
+DEAL:2:Trilinos::l1-norm: 5.00000
+DEAL:2:Trilinos::l1-norm: 5.00000
+

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.