linux-kernel - RE: [PATCH]iwlan dma mapping read and write changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <6F5C1D715B2DA5498A628E6B9C124F04F639FADE@hasmsx504.ger.corp.intel.com>
Date:	Thu, 12 Feb 2009 15:42:05 +0200
From:	"Winkler, Tomas" <tomas.winkler@...el.com>
To:	David Woodhouse <dwmw2@...radead.org>
CC:	Johannes Berg <johannes@...solutions.net>,
	"Yu, Fenghua" <fenghua.yu@...el.com>, Ingo Molnar <mingo@...e.hu>,
	Stephen Rothwell <sfr@...b.auug.org.au>,
	"Grumbach, Emmanuel" <emmanuel.grumbach@...el.com>,
	"Zhu, Yi" <yi.zhu@...el.com>, Chris Wright <chrisw@...s-sol.org>,
	LKML <linux-kernel@...r.kernel.org>,
	"iommu@...ts.linux-foundation.org" <iommu@...ts.linux-foundation.org>
Subject: RE: [PATCH]iwlan dma mapping read and write changes



>-----Original Message-----
>From: David Woodhouse [mailto:dwmw2@...radead.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@...ts.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@...solutions.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@...ts.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.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/