]> https://gitweb.dealii.org/ - dealii.git/commitdiff
Clarify a channel limit. 9408/head
authorDavid Wells <drwells@email.unc.edu>
Tue, 21 Jan 2020 14:33:11 +0000 (09:33 -0500)
committerDavid Wells <drwells@email.unc.edu>
Tue, 21 Jan 2020 14:33:11 +0000 (09:33 -0500)
This is a bit tricky since the BlockVector needs channels 100 to 200, but the
class itself should only use 0 to 100.

include/deal.II/lac/la_parallel_vector.h
include/deal.II/lac/la_parallel_vector.templates.h

index b8a2acfe19d64b3b9fdadfa3fc7d1d19b3b82d2d..18b607f973c78a5556778dee3c2c6f6693d36777 100644 (file)
@@ -524,7 +524,9 @@ namespace LinearAlgebra
        * compress_finish() is invoked, it is mandatory to specify a unique
        * communication channel to each such call, in order to avoid several
        * messages with the same ID that will corrupt this operation. Any
-       * communication channel less than 100 is a valid value.
+       * communication channel less than 100 is a valid value (in particular,
+       * the range $[100, 200)$ is reserved for
+       * LinearAlgebra::distributed::BlockVector).
        */
       void
       compress_start(
@@ -562,7 +564,9 @@ namespace LinearAlgebra
        * update_ghost_values_finish() is invoked, it is mandatory to specify a
        * unique communication channel to each such call, in order to avoid
        * several messages with the same ID that will corrupt this operation.
-       * Any communication channel less than 100 is a valid value.
+       * Any communication channel less than 100 is a valid value (in
+       * particular, the range $[100, 200)$ is reserved for
+       * LinearAlgebra::distributed::BlockVector).
        */
       void
       update_ghost_values_start(
index f5c3560666f7d5548944f822f03787061f2da8bf..e5ad13c72064349913a4b2c2f32235c397240499 100644 (file)
@@ -900,7 +900,7 @@ namespace LinearAlgebra
       const unsigned int                communication_channel,
       ::dealii::VectorOperation::values operation)
     {
-      AssertIndexRange(communication_channel, 100);
+      AssertIndexRange(communication_channel, 200);
       Assert(vector_is_ghosted == false,
              ExcMessage("Cannot call compress() on a ghosted vector"));
 
@@ -1080,7 +1080,7 @@ namespace LinearAlgebra
     Vector<Number, MemorySpaceType>::update_ghost_values_start(
       const unsigned int communication_channel) const
     {
-      AssertIndexRange(communication_channel, 100);
+      AssertIndexRange(communication_channel, 200);
 #ifdef DEAL_II_WITH_MPI
       // nothing to do when we neither have import nor ghost indices.
       if (partitioner->n_ghost_indices() == 0 &&

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.