Tejun Heo wrote:
Tejun Heo wrote:
Alan Cox wrote:
Private command type / transfer length filtering in sata_promise,
pata_it821x and pata_pdc2027x are removed as core layer filtering is
more conservative.
Signed-off-by: Tejun Heo <htejun@xxxxxxxxx>
NAK - as before
This is the wrong way to do stuff. Stop penalizing the 99.99% of users
with perfectly sane hardware. In addition I note that the whitelist for
drives would rapidly be thousands of entries long and unmanagable.
Well, I guess we'll have to agree to disagree here. I just don't think
that penalty is worth considering and definitely don't think whitelist
is necessary at all.
Also, what do other people think about the subject? IIRC, Jeff agrees
with Alan, right? Albert, Mark, what do you guys think?
...
I'm split on this one. For fast systems (typical notebook/desktop) it's
almost a non-issue either way.
But a lot of media boxes will be using this code, and they tend to have
very low-power, slow-clockrate CPUs in the 200-800Mhz range, and so the
real-time hit there (from PIO) will have a much more significant impact.
Using DMA as much as possible on those slower platforms is definitely
a plus towards avoiding non-jerky, skipped frames, or start-stoppy DVD recording.
Now the most intensive commands are still DMA under the proposed scheme,
so it's not those that one would be concerned with. But dropping to PIO
even for a few uncommon commands will still peg a real-time hit or two
on a slower media processor.
So.. ????
-
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
