On 8/12/2017 10:51 PM, John Snow wrote:
Looks about right, just remember that this flow won't call block_acct_invalid because you're bypassing the return to ide_dma_cb. I assume you'll get to that in your next series.
Yes; I meant to keep the trim accounting in ide_issue_trim_cb()
For now, this should properly reject bogus TRIM commands. When you send your next series, may I ask for a simple test case if possible?
Sure, I'll look into it
1-3: Reviewed-by: John Snow <js...@redhat.com>