RE: [PATCH] check the power management bit in MAC Header on Action Frame.

[기상원 (SW Ki) <oprsystem@xxxxxxxxxxxxx>]

Dear Johannes,

I have found one problem in the mac80211 rx routine during making the WiFi-Direct Certification. 
The Step 4 of 6.1.9 for "GOUT handling P2P Presensce Request" must have following result.
  "SN(Sniffer): P2P Presensce Response needs to comback within 100ms. If no response is received within this thie, FAIL test."

Unfortunately, The GOUT has Presensce Response packet transmission after 200ms. Then Our DUT do not pass this test. .

So, I have observed packets through sniffer for this test step.
I saw the NULL Packet that include the power management bit set to '1' before sending the Presensce Request that include PM bit set to '0'.
(The time interval between two packets is about 2ms.)


diff -urN orig/mac80211/rx.c new/mac80211/rx.c
--- orig/mac80211/rx.c  2013-09-13 12:17:59.000000000 +0900
+++ new/mac80211/rx.c   2013-09-13 12:17:02.000000000 +0900
@@ -1325,7 +1325,7 @@
                         * doze->wake transition for the probe request,
                         * and that is clearly undesirable.
                         */
-                       if (ieee80211_is_data(hdr->frame_control) &&
+                       if ((ieee80211_is_data(hdr->frame_control) || ieee80211_is_action(hdr->frame_control)) &&
                            !ieee80211_has_pm(hdr->frame_control))
                                ap_sta_ps_end(sta);
                } else {



I can see Presensce Response within 30ms after applying the below patch.
(That is success result for Step 4 of 6.1.9)

Could you review my patch code.
Best Regards, Sangwon, ki
��.n��������+%������w��{.n�����{���zW����ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f