Commit 29f8ea8a authored by Stefan Richter's avatar Stefan Richter

ieee1394: use correct barrier types between accesses of nodeid and generation

A compiler barrier (explicit on the read side, implicit on the write
side) is not quite enough for what has to be accomplished here.  Use
hardware memory barriers on systems which need them.

(Of course a full fix of generation handling would require much more
than this.  The ieee1394 core's bus generation counter had to be tied to
the controller's bus generation counter; cf. Kristian's stack.  It's
just that I have other current business with the code around these
barrier()s, so why not do at least this small fix.)
Signed-off-by: default avatarStefan Richter <stefanr@s5r6.in-berlin.de>
parent 612262a5
...@@ -1265,7 +1265,8 @@ static void nodemgr_update_node(struct node_entry *ne, struct csr1212_csr *csr, ...@@ -1265,7 +1265,8 @@ static void nodemgr_update_node(struct node_entry *ne, struct csr1212_csr *csr,
csr1212_destroy_csr(csr); csr1212_destroy_csr(csr);
} }
/* Mark the node current */ /* Finally, mark the node current */
smp_wmb();
ne->generation = generation; ne->generation = generation;
if (ne->in_limbo) { if (ne->in_limbo) {
...@@ -1798,7 +1799,7 @@ void hpsb_node_fill_packet(struct node_entry *ne, struct hpsb_packet *packet) ...@@ -1798,7 +1799,7 @@ void hpsb_node_fill_packet(struct node_entry *ne, struct hpsb_packet *packet)
{ {
packet->host = ne->host; packet->host = ne->host;
packet->generation = ne->generation; packet->generation = ne->generation;
barrier(); smp_rmb();
packet->node_id = ne->nodeid; packet->node_id = ne->nodeid;
} }
...@@ -1807,7 +1808,7 @@ int hpsb_node_write(struct node_entry *ne, u64 addr, ...@@ -1807,7 +1808,7 @@ int hpsb_node_write(struct node_entry *ne, u64 addr,
{ {
unsigned int generation = ne->generation; unsigned int generation = ne->generation;
barrier(); smp_rmb();
return hpsb_write(ne->host, ne->nodeid, generation, return hpsb_write(ne->host, ne->nodeid, generation,
addr, buffer, length); addr, buffer, length);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment