On Fri, 2008-03-21 at 18:35 -0400, Jeff Garzik wrote: > James Bottomley wrote: > > Anyway, realistically, since no CD or DVD on the market today seems to > > support the AHCI AN method, this argument is really moot ... > > No, it's not. This information is published to userspace EVEN WHEN AN > SUPPORT IS ABSENT. As any sane interface would do -- report a zero value. Right, it still does this. Currently hal is taking 0 in the media_change file to mean I don't support AN. > Thus, the interface is useful even in the absence of AN. Yes, that's why we're having the argument ... it's the use of the event in the absence of AN that people care about. > Anyway, to recap... > > before your "fix": > > userspace interface always reflected list of events sent > via my new API It didn't show a list to userspace ... it's a single file with a 0 or 1 value. 0 means doesn't support AN, 1 means does. > after your "fix": > > events may or may not be reflected in userspace interface, > who knows? It's still a single file with a 0 or 1 value. 0 means doesn't support AN, 1 means does. > Is that distinction so difficult to see? Well, if they have the same userspace effect, and display the same information to userspace, it's a bit hard to see how a user would distinguish them, yes. > This interface, like it or not, is in 2.6.24, which means its published > and "out there." > > This is a clear regression from 2.6.24. > > supported_events' value was accurate in 2.6.24. Now it is not. The current published API is the media_events file. HAL is using that to indicate support for AN. This is why we can't simply change it to 1 wholesale because we'll confuse HAL (HAL still has to send polling events if AN isn't supported). So, the best fix for 2.6.25 at the current -rc6 is to keep the meaning of the media_change file the same (0 for no AN, 1 for AN) and let HAL take the polled events via udev, which basically means it's preserving the behaviour and isn't a regression. For 2.6.26 we can add a new media_events_polled (or some other name) file, fix the sysfs ro attribute and make them true writeable filters so some raving user can turn off polled events if they want and everyone will be happy. James -- 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
