On 09/10/2015 11:11 PM, Fam Zheng wrote:
> On Thu, 09/10 19:20, John Snow wrote:
>>
>> + /* Nondata commands permit the byte_count_limit to be 0.
>> + * If this is a data-transferring PIO command and BCL is 0,
>> + * we abort at the /ATA/ level, not the ATAPI level.
>> + * See ATA8 ACS3 section 7.17.6.49 and 7.21.5 */
>> + if (!(atapi_cmd_table[s->io_buffer[0]].flags & NONDATA)) {
>> + /* TODO: Check IDENTIFY data word 125 for default BCL (currently 0)
>> */
>> + uint16_t byte_count_limit = s->lcyl | (s->hcyl << 8);
>> + if !(byte_count_limit || s->atapi_dma) {
>
> !( ?
>
> Fam
>
Ugh. Thinko'd the outer (), and didn't apparently rebuild. :(
