lkml.org 
[lkml]   [2009]   [Feb]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRE: [PATCH]iwlan dma mapping read and write changes


>-----Original Message-----
>From: David Woodhouse [mailto:dwmw2@infradead.org]
>Sent: Thursday, February 12, 2009 12:14 AM
>To: Winkler, Tomas
>Cc: Johannes Berg; Yu, Fenghua; Ingo Molnar; Stephen Rothwell; Grumbach,
>Emmanuel; Zhu, Yi; Chris Wright; LKML; iommu@lists.linux-foundation.org
>Subject: RE: [PATCH]iwlan dma mapping read and write changes
>
>On Thu, 2009-02-12 at 00:10 +0200, Winkler, Tomas wrote:
>>
>> >-----Original Message-----
>> >From: Johannes Berg [mailto:johannes@sipsolutions.net]
>> >Sent: Wednesday, February 11, 2009 11:50 PM
>> >To: Winkler, Tomas
>> >Cc: Yu, Fenghua; David Woodhouse; Ingo Molnar; Stephen Rothwell;
>Grumbach,
>> >Emmanuel; Zhu, Yi; Chris Wright; LKML; iommu@lists.linux-foundation.org
>> >Subject: RE: [PATCH]iwlan dma mapping read and write changes
>> >
>> >On Wed, 2009-02-11 at 22:47 +0100, Johannes Berg wrote:
>> >> > >> Indeed the firmware/ucode write back to host memory to update
>retry
>> >> > >> counter for APMDU packets. I'm not sure if this is needed for
>legacy
>> >> > >> traffic, though.
>> >> > >
>> >> > >I don't think they can be seeing AMPDU packets since that is, well,
>> >> > >broken?
>> >> >
>> >> > Currently I'm not aware of any other cases where memory is accessed
>> >> > back but I may have not complete info. Will be back with the answer
>> >> > tomorrow.
>> >>
>> >> I don't think it really matters, since AMPDU packets will be fixed
>soon,
>> >> and for those you know it's necessary (and the report says that under
>> >> some other circumstances it's also necessary). Unless there'd be some
>> >> benefit from using BIDI only when we know it'll be needed, and using
>> >> TODEV in the other cases?
>> >
>> >Or if there's a firmware bug.
>>
>> That's why it's better to clear the picture.
>> Fenghua
>> What exact HW do you get this error?
>
>Lenovo T400.
>


>03:00.0 0280: 8086:4236
>

I can confirm that for 5000 series the patch is required also for regular frames. In this particular machine the NIC is 5300. For 4965 only for AMPDU frames are affected also the implementation is not the same but I'm not sure if it makes any sense to differentiate.

Does anybody know what performance implications of this change are?

Thanks Tomas


---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.



\
 
 \ /
  Last update: 2009-02-12 14:45    [W:0.042 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site