| Message ID | 20210820050705.76853-1-hj.tedd.an@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Delegated to: | Luiz Von Dentz |
| Headers | show |
| Series | [BlueZ] tools: userchan-tester: Add test case for the closing channel | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/checkpatch_v2 | success | None |
| tedd_an/gitlint_v2 | success | None |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=534645 ---Test result--- Test Summary: CheckPatch PASS 0.40 seconds GitLint PASS 0.12 seconds Prep - Setup ELL PASS 48.16 seconds Build - Prep PASS 0.14 seconds Build - Configure PASS 8.45 seconds Build - Make PASS 203.34 seconds Make Check PASS 9.51 seconds Make Distcheck PASS 238.94 seconds Build w/ext ELL - Configure PASS 7.95 seconds Build w/ext ELL - Make PASS 190.48 seconds Details ############################## Test: CheckPatch - PASS Desc: Run checkpatch.pl script with rule in .checkpatch.conf ############################## Test: GitLint - PASS Desc: Run gitlint with rule in .gitlint ############################## Test: Prep - Setup ELL - PASS Desc: Clone, build, and install ELL ############################## Test: Build - Prep - PASS Desc: Prepare environment for build ############################## Test: Build - Configure - PASS Desc: Configure the BlueZ source tree ############################## Test: Build - Make - PASS Desc: Build the BlueZ source tree ############################## Test: Make Check - PASS Desc: Run 'make check' ############################## Test: Make Distcheck - PASS Desc: Run distcheck to check the distribution ############################## Test: Build w/ext ELL - Configure - PASS Desc: Configure BlueZ source with '--enable-external-ell' configuration ############################## Test: Build w/ext ELL - Make - PASS Desc: Build BlueZ source with '--enable-external-ell' configuration --- Regards, Linux Bluetooth
Hi Tedd, On Thu, Aug 19, 2021 at 11:00 PM <bluez.test.bot@gmail.com> wrote: > > This is automated email and please do not reply to this email! > > Dear submitter, > > Thank you for submitting the patches to the linux bluetooth mailing list. > This is a CI test results with your patch series: > PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=534645 > > ---Test result--- > > Test Summary: > CheckPatch PASS 0.40 seconds > GitLint PASS 0.12 seconds > Prep - Setup ELL PASS 48.16 seconds > Build - Prep PASS 0.14 seconds > Build - Configure PASS 8.45 seconds > Build - Make PASS 203.34 seconds > Make Check PASS 9.51 seconds > Make Distcheck PASS 238.94 seconds > Build w/ext ELL - Configure PASS 7.95 seconds > Build w/ext ELL - Make PASS 190.48 seconds > > Details > ############################## > Test: CheckPatch - PASS > Desc: Run checkpatch.pl script with rule in .checkpatch.conf > > ############################## > Test: GitLint - PASS > Desc: Run gitlint with rule in .gitlint > > ############################## > Test: Prep - Setup ELL - PASS > Desc: Clone, build, and install ELL > > ############################## > Test: Build - Prep - PASS > Desc: Prepare environment for build > > ############################## > Test: Build - Configure - PASS > Desc: Configure the BlueZ source tree > > ############################## > Test: Build - Make - PASS > Desc: Build the BlueZ source tree > > ############################## > Test: Make Check - PASS > Desc: Run 'make check' > > ############################## > Test: Make Distcheck - PASS > Desc: Run distcheck to check the distribution > > ############################## > Test: Build w/ext ELL - Configure - PASS > Desc: Configure BlueZ source with '--enable-external-ell' configuration > > ############################## > Test: Build w/ext ELL - Make - PASS > Desc: Build BlueZ source with '--enable-external-ell' configuration > > > > --- > Regards, > Linux Bluetooth Applied, thanks.
diff --git a/tools/userchan-tester.c b/tools/userchan-tester.c index c17644fb8..095ef2689 100644 --- a/tools/userchan-tester.c +++ b/tools/userchan-tester.c @@ -40,6 +40,8 @@ struct test_data { enum hciemu_type hciemu_type; const void *test_data; unsigned int remove_id; + struct bt_hci *hci; + uint32_t current_settings; }; static void mgmt_debug(const char *str, void *user_data) @@ -81,6 +83,8 @@ static void read_info_callback(uint8_t status, uint16_t length, tester_print(" Name: %s", rp->name); tester_print(" Short name: %s", rp->short_name); + data->current_settings = current_settings; + if (strcmp(hciemu_get_address(data->hciemu), addr)) { tester_pre_setup_failed(); return; @@ -291,6 +295,68 @@ static void test_open_failed(const void *test_data) tester_test_failed(); } +static void close_read_info_callback(uint8_t status, uint16_t length, + const void *param, void *user_data) +{ + const struct mgmt_rp_read_info *rp = param; + uint32_t current_settings; + + tester_print("Read Info callback"); + tester_print(" Status: 0x%02x", status); + + if (status || !param) { + tester_test_failed(); + return; + } + + current_settings = btohl(rp->current_settings); + if (current_settings & MGMT_SETTING_POWERED) { + tester_print("Controller is powered"); + tester_test_failed(); + return; + } + + tester_test_passed(); +} + +static void setup_channel_open(const void *test_data) +{ + struct test_data *data = tester_get_data(); + + /* Check power off */ + if (data->current_settings & MGMT_SETTING_POWERED) { + tester_print("Controller is powered"); + tester_setup_failed(); + return; + } + + /* Open Channel */ + data->hci = bt_hci_new_user_channel(data->mgmt_index); + if (!data->hci) { + mgmt_unregister(data->mgmt, data->remove_id); + data->remove_id = 0; + + tester_setup_failed(); + return; + } + + tester_print("User Channel Opened"); + + tester_setup_complete(); +} + +static void test_close_success(const void *test_data) +{ + struct test_data *data = tester_get_data(); + + tester_print("Close User Channel"); + bt_hci_unref(data->hci); + + /* Check if power is off */ + mgmt_send(data->mgmt, MGMT_OP_READ_INFO, data->mgmt_index, 0, NULL, + close_read_info_callback, NULL, NULL); +} + #define test_user(name, data, setup, func) \ do { \ struct test_data *user; \ @@ -316,6 +382,9 @@ int main(int argc, char *argv[]) setup_powered, test_open_failed); test_user("User channel open - Power Toggle Success", INT_TO_PTR(true), toggle_powered, test_open_success); + test_user("User channel close - Success", NULL, + setup_channel_open, test_close_success); + return tester_run(); }
From: Tedd Ho-Jeong An <tedd.an@intel.com> This patch adds a test casse for checking the controller power state after closing the user channel. When the user channel is closed, the controller should be powered down state. --- tools/userchan-tester.c | 69 +++++++++++++++++++++++++++++++++++++++++ 1 file changed, 69 insertions(+)