> + /* Tell compiler that status_blk fields can change. */ > + barrier(); > + if (status_idx != sblk->status_idx) { Is a compiler barrier sufficient here? It seems an out-of-order CPU could still mess things up, so an rmb() or smp_rmb() would be required. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html