broker: fix inaccurate comment

garlick · garlick · commit 9dec0eb8e5d4 · 2025-05-21T08:19:25.000-07:00
Problem: a comment says the broker's taskmap comes from the
enclosing instance, but for size=1, it's hard wired, and for
other sizes, it is placed in PMI by the launcher, which might
not be Flux.

Drop the misleading claim.
diff --git a/src/broker/boot_pmi.c b/src/broker/boot_pmi.c
@@ -133,7 +133,7 @@ static int fetch_taskmap (struct upmi *upmi,
     return 0;
 }
 
-/* Set broker.mapping attribute from enclosing instance taskmap.
+/* Set broker.mapping attribute.
  * It is not an error if the map is NULL.
  */
 static int set_broker_mapping_attr (attr_t *attrs, struct taskmap *map)

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,7 @@ static int fetch_taskmap (struct upmi *upmi,`
`133`	`133`	`return 0;`
`134`	`134`	`}`
`135`	`135`
`136`		`-/* Set broker.mapping attribute from enclosing instance taskmap.`
	`136`	`+/* Set broker.mapping attribute.`
`137`	`137`	`* It is not an error if the map is NULL.`
`138`	`138`	`*/`
`139`	`139`	`static int set_broker_mapping_attr (attr_t attrs, struct taskmap map)`