* [PATCH] unit/test-gatt: Fix hung issue
@ 2018-08-24 5:33 mingli.yu
0 siblings, 0 replies; 5+ messages in thread
From: mingli.yu @ 2018-08-24 5:33 UTC (permalink / raw)
To: linux-bluetooth
From: Mingli Yu <mingli.yu@windriver.com>
The below test hangs infinitely
$ unit/test-gatt -p /robustness/unkown-request -d
/robustness/unkown-request - init
/robustness/unkown-request - setup
/robustness/unkown-request - setup complete
/robustness/unkown-request - run
GATT: < 02 17 00 ...
bt_gatt_server:MTU exchange complete, with MTU: 23
GATT: > 03 00 02 ...
PDU: = 03 00 02 ...
GATT: < bf 00
Actually, the /robustness/unkown-request test does
no action.
---
unit/test-gatt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/unit/test-gatt.c b/unit/test-gatt.c
index c7e28f865..b57373b07 100644
--- a/unit/test-gatt.c
+++ b/unit/test-gatt.c
@@ -4463,7 +4463,7 @@ int main(int argc, char *argv[])
test_server, service_db_1, NULL,
raw_pdu(0x03, 0x00, 0x02),
raw_pdu(0xbf, 0x00),
- raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06));
+ raw_pdu());
define_test_server("/robustness/unkown-command",
test_server, service_db_1, NULL,
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH] unit/test-gatt: Fix hung issue
@ 2018-08-24 5:30 mingli.yu
2018-08-24 5:28 ` Yu, Mingli
0 siblings, 1 reply; 5+ messages in thread
From: mingli.yu @ 2018-08-24 5:30 UTC (permalink / raw)
To: linux-bluetooth
From: Brian Gix <brian.gix@intel.com>
The below test hangs infinitely
$ unit/test-gatt -p /robustness/unkown-request -d
/robustness/unkown-request - init
/robustness/unkown-request - setup
/robustness/unkown-request - setup complete
/robustness/unkown-request - run
GATT: < 02 17 00 ...
bt_gatt_server:MTU exchange complete, with MTU: 23
GATT: > 03 00 02 ...
PDU: = 03 00 02 ...
GATT: < bf 00
Actually, the /robustness/unkown-request test does
no action.
---
unit/test-gatt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/unit/test-gatt.c b/unit/test-gatt.c
index c7e28f865..b57373b07 100644
--- a/unit/test-gatt.c
+++ b/unit/test-gatt.c
@@ -4463,7 +4463,7 @@ int main(int argc, char *argv[])
test_server, service_db_1, NULL,
raw_pdu(0x03, 0x00, 0x02),
raw_pdu(0xbf, 0x00),
- raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06));
+ raw_pdu());
define_test_server("/robustness/unkown-command",
test_server, service_db_1, NULL,
--
2.17.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] unit/test-gatt: Fix hung issue
2018-08-24 5:30 mingli.yu
@ 2018-08-24 5:28 ` Yu, Mingli
2018-08-24 6:02 ` Yu, Mingli
0 siblings, 1 reply; 5+ messages in thread
From: Yu, Mingli @ 2018-08-24 5:28 UTC (permalink / raw)
To: linux-bluetooth
Please ignore this.
Thanks,
On 2018年08月24日 13:30, mingli.yu@windriver.com wrote:
> From: Brian Gix <brian.gix@intel.com>
>
> The below test hangs infinitely
> $ unit/test-gatt -p /robustness/unkown-request -d
> /robustness/unkown-request - init
> /robustness/unkown-request - setup
> /robustness/unkown-request - setup complete
> /robustness/unkown-request - run
> GATT: < 02 17 00 ...
> bt_gatt_server:MTU exchange complete, with MTU: 23
> GATT: > 03 00 02 ...
> PDU: = 03 00 02 ...
> GATT: < bf 00
>
> Actually, the /robustness/unkown-request test does
> no action.
> ---
> unit/test-gatt.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/unit/test-gatt.c b/unit/test-gatt.c
> index c7e28f865..b57373b07 100644
> --- a/unit/test-gatt.c
> +++ b/unit/test-gatt.c
> @@ -4463,7 +4463,7 @@ int main(int argc, char *argv[])
> test_server, service_db_1, NULL,
> raw_pdu(0x03, 0x00, 0x02),
> raw_pdu(0xbf, 0x00),
> - raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06));
> + raw_pdu());
>
> define_test_server("/robustness/unkown-command",
> test_server, service_db_1, NULL,
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] unit/test-gatt: Fix hung issue
2018-08-24 5:28 ` Yu, Mingli
@ 2018-08-24 6:02 ` Yu, Mingli
2018-08-24 7:42 ` Luiz Augusto von Dentz
0 siblings, 1 reply; 5+ messages in thread
From: Yu, Mingli @ 2018-08-24 6:02 UTC (permalink / raw)
To: linux-bluetooth
Correct the commit author.
Thanks,
On 2018年08月24日 13:28, Yu, Mingli wrote:
> Please ignore this.
>
> Thanks,
>
>
> On 2018年08月24日 13:30, mingli.yu@windriver.com wrote:
>> From: Brian Gix <brian.gix@intel.com>
>>
>> The below test hangs infinitely
>> $ unit/test-gatt -p /robustness/unkown-request -d
>> /robustness/unkown-request - init
>> /robustness/unkown-request - setup
>> /robustness/unkown-request - setup complete
>> /robustness/unkown-request - run
>> GATT: < 02 17 00 ...
>> bt_gatt_server:MTU exchange complete, with MTU: 23
>> GATT: > 03 00 02 ...
>> PDU: = 03 00 02 ...
>> GATT: < bf 00
>>
>> Actually, the /robustness/unkown-request test does
>> no action.
>> ---
>> unit/test-gatt.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/unit/test-gatt.c b/unit/test-gatt.c
>> index c7e28f865..b57373b07 100644
>> --- a/unit/test-gatt.c
>> +++ b/unit/test-gatt.c
>> @@ -4463,7 +4463,7 @@ int main(int argc, char *argv[])
>> test_server, service_db_1, NULL,
>> raw_pdu(0x03, 0x00, 0x02),
>> raw_pdu(0xbf, 0x00),
>> - raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06));
>> + raw_pdu());
>>
>> define_test_server("/robustness/unkown-command",
>> test_server, service_db_1, NULL,
>>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] unit/test-gatt: Fix hung issue
2018-08-24 6:02 ` Yu, Mingli
@ 2018-08-24 7:42 ` Luiz Augusto von Dentz
0 siblings, 0 replies; 5+ messages in thread
From: Luiz Augusto von Dentz @ 2018-08-24 7:42 UTC (permalink / raw)
To: Yu, Mingli; +Cc: linux-bluetooth@vger.kernel.org
Hi,
On Fri, Aug 24, 2018 at 9:02 AM, Yu, Mingli <mingli.yu@windriver.com> wrote=
:
> Correct the commit author.
>
> Thanks,
>
>
> On 2018=E5=B9=B408=E6=9C=8824=E6=97=A5 13:28, Yu, Mingli wrote:
>>
>> Please ignore this.
>>
>> Thanks,
>>
>>
>> On 2018=E5=B9=B408=E6=9C=8824=E6=97=A5 13:30, mingli.yu@windriver.com wr=
ote:
>>>
>>> From: Brian Gix <brian.gix@intel.com>
>>>
>>> The below test hangs infinitely
>>> $ unit/test-gatt -p /robustness/unkown-request -d
>>> /robustness/unkown-request - init
>>> /robustness/unkown-request - setup
>>> /robustness/unkown-request - setup complete
>>> /robustness/unkown-request - run
>>> GATT: < 02 17 00 ...
>>> bt_gatt_server:MTU exchange complete, with MTU: 23
>>> GATT: > 03 00 02 ...
>>> PDU: =3D 03 00 02 ...
>>> GATT: < bf 00
/robustness/unkown-request - init
/robustness/unkown-request - setup
/robustness/unkown-request - setup complete
/robustness/unkown-request - run
GATT: < 02 17 00 ...
bt_gatt_server:MTU exchange complete, with MTU: 23
GATT: > 03 00 02 ...
PDU: =3D 03 00 02 ...
GATT: < bf 00 ..
GATT: > 01 bf 00 00 06 .....
PDU: =3D 01 bf 00 00 06 .....
/robustness/unkown-request - test passed
/robustness/unkown-request - teardown
/robustness/unkown-request - teardown complete
/robustness/unkown-request - done
Test Summary
------------
/robustness/unkown-request Passed 0.001 sec=
onds
Total: 1, Passed: 1 (100.0%), Failed: 0,
>>> Actually, the /robustness/unkown-request test does
>>> no action.
Isn't that last PDU the actual error response, without that the
original intend of the test is lost.
>>> ---
>>> unit/test-gatt.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/unit/test-gatt.c b/unit/test-gatt.c
>>> index c7e28f865..b57373b07 100644
>>> --- a/unit/test-gatt.c
>>> +++ b/unit/test-gatt.c
>>> @@ -4463,7 +4463,7 @@ int main(int argc, char *argv[])
>>> test_server, service_db_1, NULL,
>>> raw_pdu(0x03, 0x00, 0x02),
>>> raw_pdu(0xbf, 0x00),
>>> - raw_pdu(0x01, 0xbf, 0x00, 0x00, 0x06));
>>> + raw_pdu());
>>>
>>> define_test_server("/robustness/unkown-command",
>>> test_server, service_db_1, NULL,
>>>
>
--=20
Luiz Augusto von Dentz
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-08-24 7:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-08-24 5:33 [PATCH] unit/test-gatt: Fix hung issue mingli.yu
-- strict thread matches above, loose matches on Subject: below --
2018-08-24 5:30 mingli.yu
2018-08-24 5:28 ` Yu, Mingli
2018-08-24 6:02 ` Yu, Mingli
2018-08-24 7:42 ` Luiz Augusto von Dentz
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).