LinuxLists.cc - [PATCH] android/pts: Update PTS files for L2CAP

2014-08-06 14:11:27

Subject: [PATCH] android/pts: Update PTS files for L2CAP

PICS and PIXITs checked against PTS 5.2. Regression round on PTS
5.2 against Android 4.4.4.
---
android/pics-l2cap.txt | 11 ++++++----
android/pixit-l2cap.txt | 2 +-
android/pts-l2cap.txt | 57 +++++++++++++++++++++++++++----------------------
3 files changed, 40 insertions(+), 30 deletions(-)

diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
index 53e6b9f..8e726e4 100644
--- a/android/pics-l2cap.txt
+++ b/android/pics-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PICS for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
# - not yet implemented/supported
@@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
-TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1, C.16)
+TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
+ (C.1, C.16)
TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
a Unicast connectionless L2CAP
channel (C.16)
@@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
-C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE Excluded.
+C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed,
+ ELSE Excluded.
C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is claimed,
ELSE Excluded.
@@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
+C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported,
+ ELSE Excluded.
C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-------------------------------------------------------------------------------
diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
index 914a995..401e584 100644
--- a/android/pixit-l2cap.txt
+++ b/android/pixit-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PIXIT for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
& - should be set to IUT Bluetooth address
diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
index 29e6d7c..11ea04f 100644
--- a/android/pts-l2cap.txt
+++ b/android/pts-l2cap.txt
@@ -1,8 +1,8 @@
PTS test results for L2CAP

-PTS version: 5.1
-Tested: 27-June-2014
-Android version: 4.4.2
+PTS version: 5.2
+Tested: 14-July-2014
+Android version: 4.4.4
Kernel version: 3.17

Results:
@@ -58,8 +58,10 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
-TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
-TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
+TC_CMC_BV_12_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
+ JIRA #BA-77
+TC_CMC_BV_13_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
+ JIRA #BA-78
TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
@@ -71,10 +73,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
+TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
+TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
@@ -85,31 +86,34 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
-TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
+TC_ERM_BV_07_C INC l2test -X ertm -P 4113
+ JIRA #BA-79
TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
-TC_ERM_BV_13_C PASS
-TC_ERM_BV_14_C PASS
-TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
+TC_ERM_BV_13_C INC JIRA #BA-80
+TC_ERM_BV_14_C INC JIRA #BA-81
+TC_ERM_BV_15_C INC l2test -X ertm -P 4113 -N 4 -Y 4
TC_ERM_BV_16_C N/A
TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
-TC_ERM_BV_18_C PASS
-TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
+TC_ERM_BV_18_C INC JIRA #BA-82
+TC_ERM_NV_19_C INC JIRA #BA-83
TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
-TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
-TC_ERM_BV_22_C PASS
-TC_ERM_BV_23_C PASS
+TC_ERM_BV_21_C INC l2test -w -X ertm -P 4113 -N 3
+ JIRA #BA-84
+TC_ERM_BV_22_C INC JIRA #BA-85
+TC_ERM_BV_23_C INC JIRA #BA-86
TC_ERM_BI_01_C N/A
TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
-TC_ERM_BI_03_C PASS
-TC_ERM_BI_04_C PASS
-TC_ERM_BI_05_C PASS
+TC_ERM_BI_03_C INC l2test -w -X ertm -P 4113 -N 2
+ JIRA #BA-87
+TC_ERM_BI_04_C INC JIRA #BA-88
+TC_ERM_BI_05_C INC JIRA #BA-89
TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
-TC_STM_BV_03_C PASS
+TC_STM_BV_03_C INC JIRA #BA-76
TC_STM_BV_11_C N/A
TC_STM_BV_12_C N/A
TC_STM_BV_13_C N/A
@@ -142,9 +146,12 @@ TC_ECF_BV_05_C N/A
TC_ECF_BV_06_C N/A
TC_ECF_BV_07_C N/A
TC_ECF_BV_08_C N/A
-TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
-TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
-TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>
+ NOTE: add some notes; daemon should be stopped
+ hciattach
+TC_LE_CPU_BV_01_C INC l2test -r -V le_public -J 4
+ JIRA #BA-75
+TC_LE_CPU_BV_02_C INC l2test -n -V le_public -J 4 <braddr>
+TC_LE_CPU_BI_01_C INC l2test -n -V le_public -J 4 <braddr>
TC_LE_CPU_BI_02_C INC PTS issue #12339
l2test -r -V le_public -J 4
-TC_LE_REJ_BV_01_C PASS l2test -n -V le_public -J 4 <braddr>
+TC_LE_REJ_BV_01_C INC l2test -n -V le_public -J 4 <braddr>
--
1.8.5.3

2014-08-19 17:32:53

by Szymon Janc

[permalink] [raw]

Subject: Re: [PATCH] android/pts: Update PTS files for L2CAP

Hi Sebastian,

On Tuesday 19 of August 2014 10:32:34 Sebastian Chlad wrote:
> PICS and PIXITs checked against PTS 5.2. Regression round on PTS
> 5.2 against Android 4.4.4. Interim results.
> ---
> android/pics-l2cap.txt | 11 +++++++----
> android/pixit-l2cap.txt | 2 +-
> android/pts-l2cap.txt | 46 ++++++++++++++++++++++++----------------------
> 3 files changed, 32 insertions(+), 27 deletions(-)
>
> diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
> index 53e6b9f..8e726e4 100644
> --- a/android/pics-l2cap.txt
> +++ b/android/pics-l2cap.txt
> @@ -1,6 +1,6 @@
> L2CAP PICS for the PTS tool.
>
> -PTS version: 5.1
> +PTS version: 5.2
>
> * - different than PTS defaults
> # - not yet implemented/supported
> @@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
> TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
> TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
> TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
> -TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1,
C.16)
> +TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
> + (C.1, C.16)
> TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
> a Unicast connectionless L2CAP
> channel (C.16)
> @@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE
> Excluded. C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
> C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
> C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
> -C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE
> Excluded. +C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is
> claimed, + ELSE Excluded.
> C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
> C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is
> claimed, ELSE Excluded.
> @@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE
> Excluded. C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
> C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
> C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE
> Excluded. -C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is
> supported, ELSE Excluded. +C.8: Mandatory if TSPC_L2CAP_2_44 OR
> TSPC_L2CAP_2_38 is supported, + ELSE Excluded.
> C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE
> Excluded.
> ---------------------------------------------------------------------------
> ---- diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
> index 914a995..401e584 100644
> --- a/android/pixit-l2cap.txt
> +++ b/android/pixit-l2cap.txt
> @@ -1,6 +1,6 @@
> L2CAP PIXIT for the PTS tool.
>
> -PTS version: 5.1
> +PTS version: 5.2
>
> * - different than PTS defaults
> & - should be set to IUT Bluetooth address
> diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
> index 29e6d7c..0746ea7 100644
> --- a/android/pts-l2cap.txt
> +++ b/android/pts-l2cap.txt
> @@ -1,9 +1,9 @@
> PTS test results for L2CAP
>
> -PTS version: 5.1
> -Tested: 27-June-2014
> -Android version: 4.4.2
> -Kernel version: 3.17
> +PTS version: 5.2
> +Tested: 14-August-2014
> +Android version: 4.4.4
> +Kernel version: 3.18
>
> Results:
> PASS test passed
> @@ -58,8 +58,8 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
> TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
> TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
> -TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
> -TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
> +TC_CMC_BV_12_C PASS l2test -z -X ertm <bdaddr>
> +TC_CMC_BV_13_C PASS l2test -z -X ertm <bdaddr>
> TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
> @@ -71,10 +71,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
> TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
> TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
> TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
> +TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
> TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
> +TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
> TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
> TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
> @@ -85,31 +84,31 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
> TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
> TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
> TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
> -TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
> +TC_ERM_BV_07_C PASS l2test -r -H 1000 -K 10000 -X ertm -P 4113
> TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
> TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
> TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
> -TC_ERM_BV_13_C PASS
> -TC_ERM_BV_14_C PASS
> -TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
> +TC_ERM_BV_13_C PASS l2test -x -X ertm -P 4113 -N 2
> +TC_ERM_BV_14_C PASS l2test -x -X ertm -P 4113 -N 4
> +TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4
> TC_ERM_BV_16_C N/A
> TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
> -TC_ERM_BV_18_C PASS
> -TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
> +TC_ERM_BV_18_C PASS l2test -x -X ertm -P 4113 -N 1
> +TC_ERM_NV_19_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
> -TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
> -TC_ERM_BV_22_C PASS
> -TC_ERM_BV_23_C PASS
> +TC_ERM_BV_21_C PASS l2test -x -X ertm -P 4113 -D 2000 -N 2
> +TC_ERM_BV_22_C PASS l2test -r -H 1000 -K 5000 -X ertm -P 4113
> +TC_ERM_BV_23_C PASS l2test -x -X ertm -P 4113 -N 2
> TC_ERM_BI_01_C N/A
> TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
> -TC_ERM_BI_03_C PASS
> -TC_ERM_BI_04_C PASS
> -TC_ERM_BI_05_C PASS
> +TC_ERM_BI_03_C PASS l2test -x -X ertm -P 4113 -N 2
> +TC_ERM_BI_04_C PASS l2test -x -X ertm -P 4113 -N 2
> +TC_ERM_BI_05_C PASS l2test -x -X ertm -P 4113 -N 2
> TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
> TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
> -TC_STM_BV_03_C PASS
> +TC_STM_BV_03_C PASS l2test -x -X streaming -P 4113 -N 2
> TC_STM_BV_11_C N/A
> TC_STM_BV_12_C N/A
> TC_STM_BV_13_C N/A
> @@ -142,6 +141,9 @@ TC_ECF_BV_05_C N/A
> TC_ECF_BV_06_C N/A
> TC_ECF_BV_07_C N/A
> TC_ECF_BV_08_C N/A
> + NOTE: for LE tests daemon should be stopped
> + then:
> + setprop ctl.start hciattach
> TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
> TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
> TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>

Applied. Thanks.

--
BR
Szymon Janc

2014-08-19 08:32:34

by Sebastian Chlad

[permalink] [raw]

Subject: [PATCH] android/pts: Update PTS files for L2CAP

PICS and PIXITs checked against PTS 5.2. Regression round on PTS
5.2 against Android 4.4.4. Interim results.
---
android/pics-l2cap.txt | 11 +++++++----
android/pixit-l2cap.txt | 2 +-
android/pts-l2cap.txt | 46 ++++++++++++++++++++++++----------------------
3 files changed, 32 insertions(+), 27 deletions(-)

diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
index 53e6b9f..8e726e4 100644
--- a/android/pics-l2cap.txt
+++ b/android/pics-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PICS for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
# - not yet implemented/supported
@@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
-TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1, C.16)
+TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
+ (C.1, C.16)
TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
a Unicast connectionless L2CAP
channel (C.16)
@@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
-C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE Excluded.
+C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed,
+ ELSE Excluded.
C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is claimed,
ELSE Excluded.
@@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
+C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported,
+ ELSE Excluded.
C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-------------------------------------------------------------------------------
diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
index 914a995..401e584 100644
--- a/android/pixit-l2cap.txt
+++ b/android/pixit-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PIXIT for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
& - should be set to IUT Bluetooth address
diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
index 29e6d7c..0746ea7 100644
--- a/android/pts-l2cap.txt
+++ b/android/pts-l2cap.txt
@@ -1,9 +1,9 @@
PTS test results for L2CAP

-PTS version: 5.1
-Tested: 27-June-2014
-Android version: 4.4.2
-Kernel version: 3.17
+PTS version: 5.2
+Tested: 14-August-2014
+Android version: 4.4.4
+Kernel version: 3.18

Results:
PASS test passed
@@ -58,8 +58,8 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
-TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
-TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
+TC_CMC_BV_12_C PASS l2test -z -X ertm <bdaddr>
+TC_CMC_BV_13_C PASS l2test -z -X ertm <bdaddr>
TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
@@ -71,10 +71,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
+TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
+TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
@@ -85,31 +84,31 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
-TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
+TC_ERM_BV_07_C PASS l2test -r -H 1000 -K 10000 -X ertm -P 4113
TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
-TC_ERM_BV_13_C PASS
-TC_ERM_BV_14_C PASS
-TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
+TC_ERM_BV_13_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BV_14_C PASS l2test -x -X ertm -P 4113 -N 4
+TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4
TC_ERM_BV_16_C N/A
TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
-TC_ERM_BV_18_C PASS
-TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
+TC_ERM_BV_18_C PASS l2test -x -X ertm -P 4113 -N 1
+TC_ERM_NV_19_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
-TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
-TC_ERM_BV_22_C PASS
-TC_ERM_BV_23_C PASS
+TC_ERM_BV_21_C PASS l2test -x -X ertm -P 4113 -D 2000 -N 2
+TC_ERM_BV_22_C PASS l2test -r -H 1000 -K 5000 -X ertm -P 4113
+TC_ERM_BV_23_C PASS l2test -x -X ertm -P 4113 -N 2
TC_ERM_BI_01_C N/A
TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
-TC_ERM_BI_03_C PASS
-TC_ERM_BI_04_C PASS
-TC_ERM_BI_05_C PASS
+TC_ERM_BI_03_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BI_04_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BI_05_C PASS l2test -x -X ertm -P 4113 -N 2
TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
-TC_STM_BV_03_C PASS
+TC_STM_BV_03_C PASS l2test -x -X streaming -P 4113 -N 2
TC_STM_BV_11_C N/A
TC_STM_BV_12_C N/A
TC_STM_BV_13_C N/A
@@ -142,6 +141,9 @@ TC_ECF_BV_05_C N/A
TC_ECF_BV_06_C N/A
TC_ECF_BV_07_C N/A
TC_ECF_BV_08_C N/A
+ NOTE: for LE tests daemon should be stopped
+ then:
+ setprop ctl.start hciattach
TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>
--
1.8.5.3

2014-08-19 08:32:33

by Sebastian Chlad

[permalink] [raw]

Subject: [PATCH] L2CAP tests on PTS 5.2

With the latest bluetooth-next all l2cap tests are passing except the one
on which there is an errata filled (bluetooth.org)

Sebastian Chlad (1):
android/pts: Update PTS files for L2CAP

android/pics-l2cap.txt | 11 +++++++----
android/pixit-l2cap.txt | 2 +-
android/pts-l2cap.txt | 46 ++++++++++++++++++++++++----------------------
3 files changed, 32 insertions(+), 27 deletions(-)

--
1.8.5.3

2014-08-12 13:38:06

by Sebastian Chlad

[permalink] [raw]

Subject: [PATCH] android/pts: Update PTS files for L2CAP

PICS and PIXITs checked against PTS 5.2. Regression round on PTS
5.2 against Android 4.4.4. Interim results.
---
android/pics-l2cap.txt | 11 +++++++----
android/pixit-l2cap.txt | 2 +-
android/pts-l2cap.txt | 45 ++++++++++++++++++++++++---------------------
3 files changed, 32 insertions(+), 26 deletions(-)

diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
index 53e6b9f..8e726e4 100644
--- a/android/pics-l2cap.txt
+++ b/android/pics-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PICS for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
# - not yet implemented/supported
@@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
-TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1, C.16)
+TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
+ (C.1, C.16)
TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
a Unicast connectionless L2CAP
channel (C.16)
@@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
-C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE Excluded.
+C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed,
+ ELSE Excluded.
C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is claimed,
ELSE Excluded.
@@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
+C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported,
+ ELSE Excluded.
C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
-------------------------------------------------------------------------------
diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
index 914a995..401e584 100644
--- a/android/pixit-l2cap.txt
+++ b/android/pixit-l2cap.txt
@@ -1,6 +1,6 @@
L2CAP PIXIT for the PTS tool.

-PTS version: 5.1
+PTS version: 5.2

* - different than PTS defaults
& - should be set to IUT Bluetooth address
diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
index 29e6d7c..6d1b02e 100644
--- a/android/pts-l2cap.txt
+++ b/android/pts-l2cap.txt
@@ -1,8 +1,8 @@
PTS test results for L2CAP

-PTS version: 5.1
-Tested: 27-June-2014
-Android version: 4.4.2
+PTS version: 5.2
+Tested: 14-July-2014
+Android version: 4.4.4
Kernel version: 3.17

Results:
@@ -58,8 +58,8 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
-TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
-TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
+TC_CMC_BV_12_C PASS l2test -z -X ertm <bdaddr>
+TC_CMC_BV_13_C PASS l2test -z -X ertm <bdaddr>
TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
@@ -71,10 +71,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
+TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
-TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
+TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
@@ -85,31 +84,33 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
-TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
+TC_ERM_BV_07_C INC l2test -X ertm -P 4113
+ JIRA #BA-79
TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
-TC_ERM_BV_13_C PASS
-TC_ERM_BV_14_C PASS
-TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
+TC_ERM_BV_13_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BV_14_C PASS l2test -x -X ertm -P 4113 -N 4
+TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4
TC_ERM_BV_16_C N/A
TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
-TC_ERM_BV_18_C PASS
-TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
+TC_ERM_BV_18_C PASS l2test -x -X ertm -P 4113 -N 1
+TC_ERM_NV_19_C PASS l2test -x -X ertm -P 4113 -N 1
TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
-TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
-TC_ERM_BV_22_C PASS
-TC_ERM_BV_23_C PASS
+TC_ERM_BV_21_C INC l2test -w -X ertm -P 4113 -N 3
+ JIRA #BA-84
+TC_ERM_BV_22_C INC JIRA #BA-85
+TC_ERM_BV_23_C INC JIRA #BA-86
TC_ERM_BI_01_C N/A
TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
-TC_ERM_BI_03_C PASS
-TC_ERM_BI_04_C PASS
-TC_ERM_BI_05_C PASS
+TC_ERM_BI_03_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BI_04_C PASS l2test -x -X ertm -P 4113 -N 2
+TC_ERM_BI_05_C PASS l2test -x -X ertm -P 4113 -N 2
TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
-TC_STM_BV_03_C PASS
+TC_STM_BV_03_C INC JIRA #BA-76
TC_STM_BV_11_C N/A
TC_STM_BV_12_C N/A
TC_STM_BV_13_C N/A
@@ -142,6 +143,8 @@ TC_ECF_BV_05_C N/A
TC_ECF_BV_06_C N/A
TC_ECF_BV_07_C N/A
TC_ECF_BV_08_C N/A
+ NOTE: add some notes; daemon should be stopped
+ hciattach
TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>
--
1.8.5.3

2014-08-12 13:38:05

by Sebastian Chlad

[permalink] [raw]

Subject: [PATCH] Interim test results for L2CAP

Tests run on a kernel with L.Rymanowski's patch:
"Bluetooth: Fix ERTM L2CAP resend packet"

Moreover instructions updated for CMC tests. Test run with -z option.

LE tests are passing. Test round was affected by the test environment.

Sebastian Chlad (1):
android/pts: Update PTS files for L2CAP

android/pics-l2cap.txt | 11 +++++++----
android/pixit-l2cap.txt | 2 +-
android/pts-l2cap.txt | 45 ++++++++++++++++++++++++---------------------
3 files changed, 32 insertions(+), 26 deletions(-)

--
1.8.5.3

2014-08-11 12:41:56

by Sebastian Chlad

[permalink] [raw]

Subject: Re: [PATCH] android/pts: Update PTS files for L2CAP

Hi Lukasz,

indeed. The patch is already invalid. LE part is OK; alse two CMC
tests marked as INC are passing with -z option.

BR,
Sebastian

On 11 August 2014 13:17, Lukasz Rymanowski <[email protected]> wrote:
> Hi Sebastian,
>
> On Wed, Aug 6, 2014 at 4:11 PM, Sebastian Chlad
> <[email protected]> wrote:
>> PICS and PIXITs checked against PTS 5.2. Regression round on PTS
>> 5.2 against Android 4.4.4.
>> ---
>> android/pics-l2cap.txt | 11 ++++++----
>> android/pixit-l2cap.txt | 2 +-
>> android/pts-l2cap.txt | 57 +++++++++++++++++++++++++++----------------------
>> 3 files changed, 40 insertions(+), 30 deletions(-)
>>
>> diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
>> index 53e6b9f..8e726e4 100644
>> --- a/android/pics-l2cap.txt
>> +++ b/android/pics-l2cap.txt
>> @@ -1,6 +1,6 @@
>> L2CAP PICS for the PTS tool.
>>
>> -PTS version: 5.1
>> +PTS version: 5.2
>>
>> * - different than PTS defaults
>> # - not yet implemented/supported
>> @@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
>> TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
>> TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
>> TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
>> -TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1, C.16)
>> +TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
>> + (C.1, C.16)
>> TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
>> a Unicast connectionless L2CAP
>> channel (C.16)
>> @@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
>> C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
>> C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
>> C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
>> -C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE Excluded.
>> +C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed,
>> + ELSE Excluded.
>> C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
>> C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is claimed,
>> ELSE Excluded.
>> @@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
>> C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
>> C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
>> C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
>> -C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
>> +C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported,
>> + ELSE Excluded.
>> C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
>> -------------------------------------------------------------------------------
>> diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
>> index 914a995..401e584 100644
>> --- a/android/pixit-l2cap.txt
>> +++ b/android/pixit-l2cap.txt
>> @@ -1,6 +1,6 @@
>> L2CAP PIXIT for the PTS tool.
>>
>> -PTS version: 5.1
>> +PTS version: 5.2
>>
>> * - different than PTS defaults
>> & - should be set to IUT Bluetooth address
>> diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
>> index 29e6d7c..11ea04f 100644
>> --- a/android/pts-l2cap.txt
>> +++ b/android/pts-l2cap.txt
>> @@ -1,8 +1,8 @@
>> PTS test results for L2CAP
>>
>> -PTS version: 5.1
>> -Tested: 27-June-2014
>> -Android version: 4.4.2
>> +PTS version: 5.2
>> +Tested: 14-July-2014
>> +Android version: 4.4.4
>> Kernel version: 3.17
>>
>> Results:
>> @@ -58,8 +58,10 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
>> TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
>> TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
>> TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
>> -TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
>> -TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
>> +TC_CMC_BV_12_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
>> + JIRA #BA-77
>> +TC_CMC_BV_13_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
>> + JIRA #BA-78
>> TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
>> TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
>> TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
>> @@ -71,10 +73,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
>> TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
>> TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
>> TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
>> -TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
>> -TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
>> +TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
>> TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
>> -TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
>> +TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
>> TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
>> TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
>> TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
>> @@ -85,31 +86,34 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
>> TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
>> TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
>> TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
>> -TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
>> +TC_ERM_BV_07_C INC l2test -X ertm -P 4113
>> + JIRA #BA-79
>> TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
>> TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
>> TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
>> TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
>> TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
>> -TC_ERM_BV_13_C PASS
>> -TC_ERM_BV_14_C PASS
>> -TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
>> +TC_ERM_BV_13_C INC JIRA #BA-80
>> +TC_ERM_BV_14_C INC JIRA #BA-81
>> +TC_ERM_BV_15_C INC l2test -X ertm -P 4113 -N 4 -Y 4
>> TC_ERM_BV_16_C N/A
>> TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
>> -TC_ERM_BV_18_C PASS
>> -TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
>> +TC_ERM_BV_18_C INC JIRA #BA-82
>> +TC_ERM_NV_19_C INC JIRA #BA-83
>> TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
>> -TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
>> -TC_ERM_BV_22_C PASS
>> -TC_ERM_BV_23_C PASS
>> +TC_ERM_BV_21_C INC l2test -w -X ertm -P 4113 -N 3
>> + JIRA #BA-84
>> +TC_ERM_BV_22_C INC JIRA #BA-85
>> +TC_ERM_BV_23_C INC JIRA #BA-86
>> TC_ERM_BI_01_C N/A
>> TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
>> -TC_ERM_BI_03_C PASS
>> -TC_ERM_BI_04_C PASS
>> -TC_ERM_BI_05_C PASS
>> +TC_ERM_BI_03_C INC l2test -w -X ertm -P 4113 -N 2
>> + JIRA #BA-87
>> +TC_ERM_BI_04_C INC JIRA #BA-88
>> +TC_ERM_BI_05_C INC JIRA #BA-89
>> TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
>> TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
>> -TC_STM_BV_03_C PASS
>> +TC_STM_BV_03_C INC JIRA #BA-76
>> TC_STM_BV_11_C N/A
>> TC_STM_BV_12_C N/A
>> TC_STM_BV_13_C N/A
>> @@ -142,9 +146,12 @@ TC_ECF_BV_05_C N/A
>> TC_ECF_BV_06_C N/A
>> TC_ECF_BV_07_C N/A
>> TC_ECF_BV_08_C N/A
>> -TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
>> -TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
>> -TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>
>> + NOTE: add some notes; daemon should be stopped
>> + hciattach
>> +TC_LE_CPU_BV_01_C INC l2test -r -V le_public -J 4
>> + JIRA #BA-75
>> +TC_LE_CPU_BV_02_C INC l2test -n -V le_public -J 4 <braddr>
>> +TC_LE_CPU_BI_01_C INC l2test -n -V le_public -J 4 <braddr>
>> TC_LE_CPU_BI_02_C INC PTS issue #12339
>> l2test -r -V le_public -J 4
>> -TC_LE_REJ_BV_01_C PASS l2test -n -V le_public -J 4 <braddr>
>> +TC_LE_REJ_BV_01_C INC l2test -n -V le_public -J 4 <braddr>
>
> I think this patch is not valid anymore. LE part is still working in
> the same way as for PTS 5.1 as you also confirmed on IRC
>
> BR
> Lukasz
>> --
>> 1.8.5.3
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
>> the body of a message to [email protected]
>> More majordomo info at http://vger.kernel.org/majordomo-info.html

2014-08-11 11:17:44

by Lukasz Rymanowski

[permalink] [raw]

Subject: Re: [PATCH] android/pts: Update PTS files for L2CAP

Hi Sebastian,

On Wed, Aug 6, 2014 at 4:11 PM, Sebastian Chlad
<[email protected]> wrote:
> PICS and PIXITs checked against PTS 5.2. Regression round on PTS
> 5.2 against Android 4.4.4.
> ---
> android/pics-l2cap.txt | 11 ++++++----
> android/pixit-l2cap.txt | 2 +-
> android/pts-l2cap.txt | 57 +++++++++++++++++++++++++++----------------------
> 3 files changed, 40 insertions(+), 30 deletions(-)
>
> diff --git a/android/pics-l2cap.txt b/android/pics-l2cap.txt
> index 53e6b9f..8e726e4 100644
> --- a/android/pics-l2cap.txt
> +++ b/android/pics-l2cap.txt
> @@ -1,6 +1,6 @@
> L2CAP PICS for the PTS tool.
>
> -PTS version: 5.1
> +PTS version: 5.2
>
> * - different than PTS defaults
> # - not yet implemented/supported
> @@ -70,7 +70,8 @@ TSPC_L2CAP_2_31 False AMP Manager Support (C.12)
> TSPC_L2CAP_2_32 False ERTM over AMP (C.12)
> TSPC_L2CAP_2_33 False Streaming Mode Source over AMP Support (C.15)
> TSPC_L2CAP_2_34 False Streaming Mode Sink over AMP Support (C.15)
> -TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception (C.1, C.16)
> +TSPC_L2CAP_2_35 True (*) Unicast Connectionless Data, Reception
> + (C.1, C.16)
> TSPC_L2CAP_2_36 True (*) Ability to transmit an unencrypted packet over
> a Unicast connectionless L2CAP
> channel (C.16)
> @@ -105,7 +106,8 @@ C.10: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
> C.11: Optional IF TSPC_L2CAP_2_12 is claimed, ELSE Excluded.
> C.12: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
> C.13: Mandatory IF SUM_ICS 31/9 (3.0 + HS) is claimed, ELSE Optional.
> -C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed, ELSE Excluded.
> +C.14: Optional IF SUM_ICS 31/8 OR 31/9 OR 31/10 OR 31/11 is claimed,
> + ELSE Excluded.
> C.15: Optional IF TSPC_L2CAP_2_29 is claimed, ELSE Excluded.
> C.16: Optional IF (SUM_ICS 31/8 OR SUM_ICS 31/9 OR 31/10 OR 31/11) is claimed,
> ELSE Excluded.
> @@ -154,6 +156,7 @@ C.4: Mandatory IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
> C.5: Optional IF BR/EDR OR BR/EDR/LE is claimed, ELSE Excluded.
> C.6: Mandatory IF LE OR BR/EDR/LE is claimed, ELSE Excluded.
> C.7: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
> -C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
> +C.8: Mandatory if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported,
> + ELSE Excluded.
> C.9: Optional if TSPC_L2CAP_2_44 OR TSPC_L2CAP_2_38 is supported, ELSE Excluded.
> -------------------------------------------------------------------------------
> diff --git a/android/pixit-l2cap.txt b/android/pixit-l2cap.txt
> index 914a995..401e584 100644
> --- a/android/pixit-l2cap.txt
> +++ b/android/pixit-l2cap.txt
> @@ -1,6 +1,6 @@
> L2CAP PIXIT for the PTS tool.
>
> -PTS version: 5.1
> +PTS version: 5.2
>
> * - different than PTS defaults
> & - should be set to IUT Bluetooth address
> diff --git a/android/pts-l2cap.txt b/android/pts-l2cap.txt
> index 29e6d7c..11ea04f 100644
> --- a/android/pts-l2cap.txt
> +++ b/android/pts-l2cap.txt
> @@ -1,8 +1,8 @@
> PTS test results for L2CAP
>
> -PTS version: 5.1
> -Tested: 27-June-2014
> -Android version: 4.4.2
> +PTS version: 5.2
> +Tested: 14-July-2014
> +Android version: 4.4.4
> Kernel version: 3.17
>
> Results:
> @@ -58,8 +58,10 @@ TC_CMC_BV_08_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BV_09_C PASS l2test -r -X basic -P 4113
> TC_CMC_BV_10_C PASS l2test -n -P 4113 <bdaddr>
> TC_CMC_BV_11_C PASS l2test -n -P 4113 <bdaddr>
> -TC_CMC_BV_12_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
> -TC_CMC_BV_13_C PASS l2test -s -X ertm -P 4113 -U <bdaddr>
> +TC_CMC_BV_12_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
> + JIRA #BA-77
> +TC_CMC_BV_13_C INC l2test -s -X ertm -P 4113 -U <bdaddr>
> + JIRA #BA-78
> TC_CMC_BV_14_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BV_15_C PASS l2test -r -X streaming -P 4113
> TC_CMC_BI_01_C PASS l2test -r -X ertm -P 4113
> @@ -71,10 +73,9 @@ TC_CMC_BI_06_C PASS l2test -r -P 4113
> TC_FOC_BV_01_C PASS l2test -r -X ertm -P 4113 -F 0
> TC_FOC_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
> TC_FOC_BV_03_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_FOC_BV_04_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 N 1
> +TC_OFS_BV_01_C PASS l2test -x -X ertm -P 4113 -F 0 -N 1
> TC_OFS_BV_02_C PASS l2test -r -X ertm -P 4113 -F 0
> -TC_OFS_BV_03_C PASS l2test -r -X streaming -P 4113 -F 0
> +TC_OFS_BV_03_C PASS l2test -x -X streaming -P 4113 -F 0 -N 1
> TC_OFS_BV_04_C PASS l2test -r -X streaming -P 4113 -F 0
> TC_OFS_BV_05_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_OFS_BV_06_C PASS l2test -r -X ertm -P 4113
> @@ -85,31 +86,34 @@ TC_ERM_BV_02_C PASS l2test -r -X ertm -P 4113
> TC_ERM_BV_03_C PASS l2test -r -X ertm -P 4113
> TC_ERM_BV_05_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
> TC_ERM_BV_06_C PASS l2test -x -X ertm -P 4113 -N 2 -Y 2
> -TC_ERM_BV_07_C PASS l2test -X ertm -P 4113
> +TC_ERM_BV_07_C INC l2test -X ertm -P 4113
> + JIRA #BA-79
> TC_ERM_BV_08_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_ERM_BV_09_C PASS l2test -X ertm -P 4113
> TC_ERM_BV_10_C PASS l2test -x -X ertm -P 4113 -N 1
> TC_ERM_BV_11_C PASS l2test -x -X ertm -P 4113 -N 1 -Q 1
> TC_ERM_BV_12_C PASS l2test -x -X ertm -P 4113 -R -N 1 -Q 1
> -TC_ERM_BV_13_C PASS
> -TC_ERM_BV_14_C PASS
> -TC_ERM_BV_15_C PASS l2test -X ertm -P 4113 -N 4 -Y 4
> +TC_ERM_BV_13_C INC JIRA #BA-80
> +TC_ERM_BV_14_C INC JIRA #BA-81
> +TC_ERM_BV_15_C INC l2test -X ertm -P 4113 -N 4 -Y 4
> TC_ERM_BV_16_C N/A
> TC_ERM_BV_17_C PASS l2test -X ertm -P 4113
> -TC_ERM_BV_18_C PASS
> -TC_ERM_BV_19_C PASS l2test -x -X ertm -P 4113 -N 1
> +TC_ERM_BV_18_C INC JIRA #BA-82
> +TC_ERM_NV_19_C INC JIRA #BA-83
> TC_ERM_BV_20_C PASS l2test -x -X ertm -P 4113 -N 1
> -TC_ERM_BV_21_C PASS l2test -w -X ertm -P 4113 -N 3
> -TC_ERM_BV_22_C PASS
> -TC_ERM_BV_23_C PASS
> +TC_ERM_BV_21_C INC l2test -w -X ertm -P 4113 -N 3
> + JIRA #BA-84
> +TC_ERM_BV_22_C INC JIRA #BA-85
> +TC_ERM_BV_23_C INC JIRA #BA-86
> TC_ERM_BI_01_C N/A
> TC_ERM_BI_02_C PASS l2test -X ertm -P 4113
> -TC_ERM_BI_03_C PASS
> -TC_ERM_BI_04_C PASS
> -TC_ERM_BI_05_C PASS
> +TC_ERM_BI_03_C INC l2test -w -X ertm -P 4113 -N 2
> + JIRA #BA-87
> +TC_ERM_BI_04_C INC JIRA #BA-88
> +TC_ERM_BI_05_C INC JIRA #BA-89
> TC_STM_BV_01_C PASS l2test -x -X streaming -P 4113 -N 3 -Y 3
> TC_STM_BV_02_C PASS l2test -r -X streaming -P 4113
> -TC_STM_BV_03_C PASS
> +TC_STM_BV_03_C INC JIRA #BA-76
> TC_STM_BV_11_C N/A
> TC_STM_BV_12_C N/A
> TC_STM_BV_13_C N/A
> @@ -142,9 +146,12 @@ TC_ECF_BV_05_C N/A
> TC_ECF_BV_06_C N/A
> TC_ECF_BV_07_C N/A
> TC_ECF_BV_08_C N/A
> -TC_LE_CPU_BV_01_C PASS l2test -r -V le_public -J 4
> -TC_LE_CPU_BV_02_C PASS l2test -n -V le_public -J 4 <braddr>
> -TC_LE_CPU_BI_01_C PASS l2test -n -V le_public -J 4 <braddr>
> + NOTE: add some notes; daemon should be stopped
> + hciattach
> +TC_LE_CPU_BV_01_C INC l2test -r -V le_public -J 4
> + JIRA #BA-75
> +TC_LE_CPU_BV_02_C INC l2test -n -V le_public -J 4 <braddr>
> +TC_LE_CPU_BI_01_C INC l2test -n -V le_public -J 4 <braddr>
> TC_LE_CPU_BI_02_C INC PTS issue #12339
> l2test -r -V le_public -J 4
> -TC_LE_REJ_BV_01_C PASS l2test -n -V le_public -J 4 <braddr>
> +TC_LE_REJ_BV_01_C INC l2test -n -V le_public -J 4 <braddr>

I think this patch is not valid anymore. LE part is still working in
the same way as for PTS 5.1 as you also confirmed on IRC

BR
Lukasz
> --
> 1.8.5.3
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html