Skip to contentSkip to conversation

media change events for polled devices broken with current sdev_evt_send()

DISCUSSION Kay Sievers 2 participants 2 messages Mar 6, 2008 – Mar 17, 2008 SCSI

lore
Share & tools
lore
#0 lore raw

Hi Jeff,
a341cd0f (SCSI: add asynchronous event notification API)
introduces an event filter like:
+ if (!test_bit(evt->evt_type, sdev->supported_events)) {
+ kfree(evt);
+ return;
+ }

which breaks:
285e9670 (sr,sd: send media state change modification events)

Why are events sent through sdev_evt_send() filtered? In which case does code send an event which we don't want to deliver?

Side note: introduced with a341cd0f is the sysfs file:
"evt_media_change", which has a store() method, but has the permission 0444. It should either have no store() method assigned, by introducing something like DECLARE_EVT_RO(), or the file permission should be changed to 0644.

Thanks,
Kay

#1 lore raw

Ping! Jeff, please answer the question. We need the media change events for non AN devices.

Thanks,
Kay

Quoted 23 lines · Kay Sievers

Hi Jeff,
a341cd0f (SCSI: add asynchronous event notification API)
introduces an event filter like:
+ if (!test_bit(evt->evt_type, sdev->supported_events)) {
+ kfree(evt);
+ return;
+ }

which breaks:
285e9670 (sr,sd: send media state change modification events)

Why are events sent through sdev_evt_send() filtered? In which case does code send an event which we don't want to deliver?

Side note: introduced with a341cd0f is the sysfs file:
"evt_media_change", which has a store() method, but has the permission 0444. It should either have no store() method assigned, by introducing something like DECLARE_EVT_RO(), or the file permission should be changed to 0644.

Thanks,
Kay