[Qemu-devel] Re: megasas: Usage of is_write in megasas_map_sgl()

[Hannes Reinecke]

On 12/03/2010 11:25 AM, Kevin Wolf wrote:
> Am 02.12.2010 23:24, schrieb Nicholas A. Bellinger:
>> The usage of a inverted is_write with cpu_physical_memory_map() also
>> seems to be the case in dma-helpers.c:dma_brdv_cb():
>>
>>      mem = cpu_physical_memory_map(cur_addr, &cur_len, !dbs->is_write);
>>
>> After changing to an inverted is_write in megasas.v3/megasas-upstream-v1
>> code, this still does *not* seem to make a difference wrt to the 64-bit
>> Win7 case, and the same BSOD appears..  In any event, we should verify
>> this with QEMU folks in terms of what the proper usage of is_write for
>> cpu_physical_memory_map().  Gerd and Kevin comments here..?
> 
> I haven't looked at the SCSI code, but the dma-helpers.c one is easy to
> explain: If you read from the disk, you write to memory.
> 
But the main point here is: When should we set the 'is_write' parameter?
Both reading and writing will go into memory, so from that point we
always would need to set this ...

And some documentation about this parameter would be really beneficial.

Cheers,

Hannes