Joe Eykholt wrote:
Subtract 1, so that scsi_host still starts with 0.
...
-static int scsi_host_next_hn; /* host_no for next new host */
+static atomic_t scsi_host_next_hn; /* host_no for next new host */
static void scsi_host_cls_release(struct device *dev)
@@ -333,7 +333,7 @@ struct Scsi_Host *scsi_host_alloc(struct scsi_host_template *sht, int privsize)
mutex_init(&shost->scan_mutex);
- shost->host_no = scsi_host_next_hn++; /* XXX(hch): still racy */
+ shost->host_no = atomic_inc_return(&scsi_host_next_hn) - 1;
Can also be written
static atomic_t scsi_host_next_hn = ATOMIC_INIT(-1);
...
shost->host_no = atomic_inc_return(&scsi_host_next_hn);
--
Stefan Richter
-=====-==--= -==- =----
http://arcgraph.de/sr/
--
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
