[openib-general][PATCH][kdapl]: adding DAT_MEM_TYPE_IA support
James Lentini
jlentini at netapp.com
Tue Aug 2 11:31:20 PDT 2005
On Tue, 2 Aug 2005, Guy German wrote:
> Hi,
>
> James Lentini <mailto:jlentini at netapp.com> wrote:
>> On Fri, 29 Jul 2005, Guy German wrote:
>>
>>>> + array = (u64 *)phys_addr.for_array; /* need to add for_u64_array
>>>> to union */ What does this comment mean?
>>>
>>> I think the right way to do it is :
>>> array = phys_addr.for_u64_array
>>> (Givven the union consists of a new type u64* called "for_u64_array")
>>
>> I believe the original idea was to have IA memory use the
>> DAT_REGION_DESCRIPTION's for_pointer value.
>
> for_pointer is void*, and: *((void *)array)!=*((u64 *)array) in 32
> bit machines
I'd rather update the for_pointer type to be correct (and change the
name if necessary) than add a new memmber, for_u64_array, to the
union.
More information about the general
mailing list