diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2010-07-27 07:20:33 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2010-07-29 17:06:25 -0400 |
commit | 071595ebdc66d70219e2d1ce746016f64b2b19e7 (patch) | |
tree | 3083b0de17a6d0633420ffda1dfa04a0b4b880b2 /drivers/firewire/ohci.c | |
parent | 8e2b2b46ea4ca5ef790dddf78b360ed736a62d7c (diff) |
firewire: ohci: use memory barriers to order descriptor updates
When we append to a DMA program, we need to ensure that the order in
which initialization of the new descriptors and update of the
branch_address of the old tail descriptor, as seen by the PCI device,
happen as intended.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/firewire/ohci.c')
-rw-r--r-- | drivers/firewire/ohci.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/firewire/ohci.c b/drivers/firewire/ohci.c index 5f6bb2c53808..3f59d1df6504 100644 --- a/drivers/firewire/ohci.c +++ b/drivers/firewire/ohci.c | |||
@@ -595,6 +595,7 @@ static int ar_context_add_page(struct ar_context *ctx) | |||
595 | ab->descriptor.res_count = cpu_to_le16(PAGE_SIZE - offset); | 595 | ab->descriptor.res_count = cpu_to_le16(PAGE_SIZE - offset); |
596 | ab->descriptor.branch_address = 0; | 596 | ab->descriptor.branch_address = 0; |
597 | 597 | ||
598 | wmb(); /* finish init of new descriptors before branch_address update */ | ||
598 | ctx->last_buffer->descriptor.branch_address = cpu_to_le32(ab_bus | 1); | 599 | ctx->last_buffer->descriptor.branch_address = cpu_to_le32(ab_bus | 1); |
599 | ctx->last_buffer->next = ab; | 600 | ctx->last_buffer->next = ab; |
600 | ctx->last_buffer = ab; | 601 | ctx->last_buffer = ab; |
@@ -982,6 +983,8 @@ static void context_append(struct context *ctx, | |||
982 | d_bus = desc->buffer_bus + (d - desc->buffer) * sizeof(*d); | 983 | d_bus = desc->buffer_bus + (d - desc->buffer) * sizeof(*d); |
983 | 984 | ||
984 | desc->used += (z + extra) * sizeof(*d); | 985 | desc->used += (z + extra) * sizeof(*d); |
986 | |||
987 | wmb(); /* finish init of new descriptors before branch_address update */ | ||
985 | ctx->prev->branch_address = cpu_to_le32(d_bus | z); | 988 | ctx->prev->branch_address = cpu_to_le32(d_bus | z); |
986 | ctx->prev = find_branch_descriptor(d, z); | 989 | ctx->prev = find_branch_descriptor(d, z); |
987 | 990 | ||