Re: [PATCH 06/13] sg_io: whitelist a few more commands for multimediadevices

From: Paolo Bonzini
Date: Fri Jan 25 2013 - 12:16:06 EST


Il 25/01/2013 18:04, Tejun Heo ha scritto:
>> >
>> > I think it's the other way round. What's the justification for leaving
>> > them out, if they are in the standard? Since we're touching the
>> > commands for other standards, it's better to be complete for MMC as well.
> Maybe my experience with ATA left me bitter with the standards, but
> opening gate to everything described in standard sounds like a pretty
> bad idea to me. If there are users and devices which make use of them
> in sane way, sure. If not, what's the point of risking it?

Well, you can find broken devices for pretty much every command in the
list. Anyhow, the other two commands are obsolete so I'm okay with
leaving them out, if only for the sake of avoiding useless email threads.

Paolo

>> > At least one of them (MECHANISM STATUS) is implemented in both QEMU and
>> > Bochs, so someone is using it. If that someone were run virtualized,
>> > with the host /dev/sr0 passed directly to the guest rather than
>> > emulated, it would break.
> Sure, then, enable MECHANISM_STATUS.

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