Re: [patch 06/10] scsi: expose AN support to user space

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



akpm@xxxxxxxxxxxxxxxxxxxx wrote:
From: Kristen Carlson Accardi <kristen.c.accardi@xxxxxxxxx>

If a scsi_device supports async notification for media change, then let user
space know this capability exists by creating a new sysfs entry
"media_change_notify", which will be 1 if it is supported, and 0 if not
supported.  Create a routine which allows scsi devices to send a uevent when
media change events occur.

Signed-off-by: Kristen Carlson Accardi <kristen.c.accardi@xxxxxxxxx>
Cc: Jeff Garzik <jeff@xxxxxxxxxx>
Cc: Tejun Heo <htejun@xxxxxxxxx>
Cc: James Bottomley <James.Bottomley@xxxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>

ACK

Though I either (a) would like SCSI maintainers to apply this, or (b) would like SCSI maintainers to ACK this before I apply it myself.

Or IOW, if others are happy, I'm happy :)

The follow-up patch "libata: expose AN to user space" requires this one in order to compile, so it has the same fate.


Additional background, for those interested: libata-dev.git#upstream (2.6.24 queue) already contains Kristen's libata-specific portions of the media event notification. Internally, all the "guts" are there, and we currently call a no-op function that's waiting for the SCSI guts:

void ata_scsi_media_change_notify(struct ata_device *dev)
{
#ifdef OTHER_AN_PATCHES_HAVE_BEEN_APPLIED
        if (dev->sdev)
                scsi_device_event_notify(dev->sdev, SDEV_MEDIA_CHANGE);
#endif
}

Regards,

	Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux