diff mbox series

[BlueZ,v3,1/2] test-runner: revert udevd and audio support

Message ID fcd0cc37474487489e2567f3de7d90f1b62d9613.1683994090.git.pav@iki.fi (mailing list archive)
State Accepted
Commit a0a10f8c9d78d9add578d9ff79c7ad7e5e681f1a
Headers show
Series [BlueZ,v3,1/2] test-runner: revert udevd and audio support | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/CheckPatch warning WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled? #49: This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88 WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled? #50: This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. /github/workspace/src/src/13240258.patch has style problems, please review. NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS.
tedd_an/GitLint success Gitlint PASS
tedd_an/BuildEll success Build ELL PASS
tedd_an/BluezMake success Bluez Make PASS
tedd_an/MakeCheck success Bluez Make Check PASS
tedd_an/MakeDistcheck success Make Distcheck PASS
tedd_an/CheckValgrind success Check Valgrind PASS
tedd_an/CheckSmatch success CheckSparse PASS
tedd_an/bluezmakeextell success Make External ELL PASS
tedd_an/IncrementalBuild success Incremental Build PASS
tedd_an/ScanBuild warning ScanBuild: tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value printf("Running command %s\n", cmdname ? cmdname : argv[0]); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated.

Commit Message

Pauli Virtanen May 13, 2023, 4:08 p.m. UTC
Tests running inside the VM don't need access to sound cards running on
the host.

This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
---
 tools/test-runner.c | 108 ++------------------------------------------
 1 file changed, 3 insertions(+), 105 deletions(-)

Comments

bluez.test.bot@gmail.com May 13, 2023, 5:20 p.m. UTC | #1
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=747273

---Test result---

Test Summary:
CheckPatch                    FAIL      1.14 seconds
GitLint                       PASS      0.52 seconds
BuildEll                      PASS      26.21 seconds
BluezMake                     PASS      745.32 seconds
MakeCheck                     PASS      11.04 seconds
MakeDistcheck                 PASS      151.18 seconds
CheckValgrind                 PASS      243.86 seconds
CheckSmatch                   PASS      326.99 seconds
bluezmakeextell               PASS      98.83 seconds
IncrementalBuild              PASS      1262.88 seconds
ScanBuild                     WARNING   964.73 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[BlueZ,v3,1/2] test-runner: revert udevd and audio support
WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
#49: 
This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88

WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
#50: 
This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f

/github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/github/workspace/src/src/13240258.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.


##############################
Test: ScanBuild - WARNING
Desc: Run Scan Build
Output:
tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
        printf("Running command %s\n", cmdname ? cmdname : argv[0]);
        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.



---
Regards,
Linux Bluetooth
Luiz Augusto von Dentz May 15, 2023, 8:01 p.m. UTC | #2
Hi Pauli,

On Sat, May 13, 2023 at 10:41 AM <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=747273
>
> ---Test result---
>
> Test Summary:
> CheckPatch                    FAIL      1.14 seconds
> GitLint                       PASS      0.52 seconds
> BuildEll                      PASS      26.21 seconds
> BluezMake                     PASS      745.32 seconds
> MakeCheck                     PASS      11.04 seconds
> MakeDistcheck                 PASS      151.18 seconds
> CheckValgrind                 PASS      243.86 seconds
> CheckSmatch                   PASS      326.99 seconds
> bluezmakeextell               PASS      98.83 seconds
> IncrementalBuild              PASS      1262.88 seconds
> ScanBuild                     WARNING   964.73 seconds
>
> Details
> ##############################
> Test: CheckPatch - FAIL
> Desc: Run checkpatch.pl script
> Output:
> [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> #49:
> This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
>
> WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> #50:
> This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
>
> /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
>       mechanically convert to the typical style using --fix or --fix-inplace.
>
> /github/workspace/src/src/13240258.patch has style problems, please review.
>
> NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
>
> NOTE: If any of the errors are false positives, please report
>       them to the maintainer, see CHECKPATCH in MAINTAINERS.
>
>
> ##############################
> Test: ScanBuild - WARNING
> Desc: Run Scan Build
> Output:
> tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
>         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
>         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 1 warning generated.
>
>
>
> ---
> Regards,
> Linux Bluetooth

I tried running on fedora but it looks like something is missing in my setup:

[E][00003.767959] spa.dbus     | [          dbus.c:  329
impl_connection_get()] Failed to connect to session bus: Unable to
autolaun1
[E][00003.769967] mod.portal   | [ module-portal.c:  326
pipewire__module_init()] Failed to connect to session bus:
Input/output error
Pauli Virtanen May 15, 2023, 8:30 p.m. UTC | #3
Hi Luiz,

ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
> 
> On Sat, May 13, 2023 at 10:41 AM <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=747273
> > 
> > ---Test result---
> > 
> > Test Summary:
> > CheckPatch                    FAIL      1.14 seconds
> > GitLint                       PASS      0.52 seconds
> > BuildEll                      PASS      26.21 seconds
> > BluezMake                     PASS      745.32 seconds
> > MakeCheck                     PASS      11.04 seconds
> > MakeDistcheck                 PASS      151.18 seconds
> > CheckValgrind                 PASS      243.86 seconds
> > CheckSmatch                   PASS      326.99 seconds
> > bluezmakeextell               PASS      98.83 seconds
> > IncrementalBuild              PASS      1262.88 seconds
> > ScanBuild                     WARNING   964.73 seconds
> > 
> > Details
> > ##############################
> > Test: CheckPatch - FAIL
> > Desc: Run checkpatch.pl script
> > Output:
> > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > #49:
> > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > 
> > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > #50:
> > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > 
> > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > 
> > NOTE: For some of the reported defects, checkpatch may be able to
> >       mechanically convert to the typical style using --fix or --fix-inplace.
> > 
> > /github/workspace/src/src/13240258.patch has style problems, please review.
> > 
> > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > 
> > NOTE: If any of the errors are false positives, please report
> >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > 
> > 
> > ##############################
> > Test: ScanBuild - WARNING
> > Desc: Run Scan Build
> > Output:
> > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > 1 warning generated.
> > 
> > 
> > 
> > ---
> > Regards,
> > Linux Bluetooth
> 
> I tried running on fedora but it looks like something is missing in my setup:
> 
> [E][00003.767959] spa.dbus     | [          dbus.c:  329
> impl_connection_get()] Failed to connect to session bus: Unable to
> autolaun1
> [E][00003.769967] mod.portal   | [ module-portal.c:  326
> pipewire__module_init()] Failed to connect to session bus:
> Input/output error

I have those two "errors" too, they should be harmless as the xdg-
desktop-portal module is optional. With `test-runner -d -l -A` you
should get the Bluetooth endpoints to appear.

Removing the mod.portal from the configuration can't be done with drop-
in config files currently, and I didn't want to include a full config
file here as it has some tens of lines of mandatory boilerplate.
Luiz Augusto von Dentz May 17, 2023, 7:34 p.m. UTC | #4
Hi Pauli,

On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
>
> Hi Luiz,
>
> ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > Hi Pauli,
> >
> > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > >
> > > ---Test result---
> > >
> > > Test Summary:
> > > CheckPatch                    FAIL      1.14 seconds
> > > GitLint                       PASS      0.52 seconds
> > > BuildEll                      PASS      26.21 seconds
> > > BluezMake                     PASS      745.32 seconds
> > > MakeCheck                     PASS      11.04 seconds
> > > MakeDistcheck                 PASS      151.18 seconds
> > > CheckValgrind                 PASS      243.86 seconds
> > > CheckSmatch                   PASS      326.99 seconds
> > > bluezmakeextell               PASS      98.83 seconds
> > > IncrementalBuild              PASS      1262.88 seconds
> > > ScanBuild                     WARNING   964.73 seconds
> > >
> > > Details
> > > ##############################
> > > Test: CheckPatch - FAIL
> > > Desc: Run checkpatch.pl script
> > > Output:
> > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > #49:
> > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > >
> > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > #50:
> > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > >
> > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > >
> > > NOTE: For some of the reported defects, checkpatch may be able to
> > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > >
> > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > >
> > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > >
> > > NOTE: If any of the errors are false positives, please report
> > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > >
> > >
> > > ##############################
> > > Test: ScanBuild - WARNING
> > > Desc: Run Scan Build
> > > Output:
> > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > 1 warning generated.
> > >
> > >
> > >
> > > ---
> > > Regards,
> > > Linux Bluetooth
> >
> > I tried running on fedora but it looks like something is missing in my setup:
> >
> > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > impl_connection_get()] Failed to connect to session bus: Unable to
> > autolaun1
> > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > pipewire__module_init()] Failed to connect to session bus:
> > Input/output error
>
> I have those two "errors" too, they should be harmless as the xdg-
> desktop-portal module is optional. With `test-runner -d -l -A` you
> should get the Bluetooth endpoints to appear.
>
> Removing the mod.portal from the configuration can't be done with drop-
> in config files currently, and I didn't want to include a full config
> file here as it has some tens of lines of mandatory boilerplate.

Ok, looks like it configuring the endpoints:

https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7

That said I probably need to compile pipewire from source in order to
have the LE Audio endpoints registered, btw do you have any
instructions on how to build it, do I need to build wireplumber as
well?

>
> --
> Pauli Virtanen
patchwork-bot+bluetooth@kernel.org May 17, 2023, 7:50 p.m. UTC | #5
Hello:

This series was applied to bluetooth/bluez.git (master)
by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>:

On Sat, 13 May 2023 16:08:36 +0000 you wrote:
> Tests running inside the VM don't need access to sound cards running on
> the host.
> 
> This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> ---
>  tools/test-runner.c | 108 ++------------------------------------------
>  1 file changed, 3 insertions(+), 105 deletions(-)

Here is the summary with links:
  - [BlueZ,v3,1/2] test-runner: revert udevd and audio support
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=a0a10f8c9d78
  - [BlueZ,v3,2/2] tools/test-runner: add option to start Pipewire inside the VM
    https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=86b8d4192dbd

You are awesome, thank you!
Pauli Virtanen May 17, 2023, 8:30 p.m. UTC | #6
Hi Luiz,

ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
> 
> On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > 
> > Hi Luiz,
> > 
> > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > Hi Pauli,
> > > 
> > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > 
> > > > ---Test result---
> > > > 
> > > > Test Summary:
> > > > CheckPatch                    FAIL      1.14 seconds
> > > > GitLint                       PASS      0.52 seconds
> > > > BuildEll                      PASS      26.21 seconds
> > > > BluezMake                     PASS      745.32 seconds
> > > > MakeCheck                     PASS      11.04 seconds
> > > > MakeDistcheck                 PASS      151.18 seconds
> > > > CheckValgrind                 PASS      243.86 seconds
> > > > CheckSmatch                   PASS      326.99 seconds
> > > > bluezmakeextell               PASS      98.83 seconds
> > > > IncrementalBuild              PASS      1262.88 seconds
> > > > ScanBuild                     WARNING   964.73 seconds
> > > > 
> > > > Details
> > > > ##############################
> > > > Test: CheckPatch - FAIL
> > > > Desc: Run checkpatch.pl script
> > > > Output:
> > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > #49:
> > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > 
> > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > #50:
> > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > 
> > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > 
> > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > 
> > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > 
> > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > 
> > > > NOTE: If any of the errors are false positives, please report
> > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > 
> > > > 
> > > > ##############################
> > > > Test: ScanBuild - WARNING
> > > > Desc: Run Scan Build
> > > > Output:
> > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > 1 warning generated.
> > > > 
> > > > 
> > > > 
> > > > ---
> > > > Regards,
> > > > Linux Bluetooth
> > > 
> > > I tried running on fedora but it looks like something is missing in my setup:
> > > 
> > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > autolaun1
> > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > pipewire__module_init()] Failed to connect to session bus:
> > > Input/output error
> > 
> > I have those two "errors" too, they should be harmless as the xdg-
> > desktop-portal module is optional. With `test-runner -d -l -A` you
> > should get the Bluetooth endpoints to appear.
> > 
> > Removing the mod.portal from the configuration can't be done with drop-
> > in config files currently, and I didn't want to include a full config
> > file here as it has some tens of lines of mandatory boilerplate.
> 
> Ok, looks like it configuring the endpoints:
> 
> https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> 
> That said I probably need to compile pipewire from source in order to
> have the LE Audio endpoints registered, btw do you have any
> instructions on how to build it, do I need to build wireplumber as
> well?

Should be along these lines to install under $PWD/install and enable
BAP parts:

git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
cd pipewire
LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
   meson setup builddir --prefix=$PWD/install \
       -Dsystemd-user-service=disabled \
       -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
       -Dbluez5-codec-lc3=enabled
meson install -C builddir

Meson will clone and install Wireplumber too. Turning experimental on
in src/main.conf and it should give

$ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
...
bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
...
Luiz Augusto von Dentz May 17, 2023, 8:53 p.m. UTC | #7
Hi Pauli,

On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
>
> Hi Luiz,
>
> ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > Hi Pauli,
> >
> > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > >
> > > Hi Luiz,
> > >
> > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > Hi Pauli,
> > > >
> > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > >
> > > > > ---Test result---
> > > > >
> > > > > Test Summary:
> > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > GitLint                       PASS      0.52 seconds
> > > > > BuildEll                      PASS      26.21 seconds
> > > > > BluezMake                     PASS      745.32 seconds
> > > > > MakeCheck                     PASS      11.04 seconds
> > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > ScanBuild                     WARNING   964.73 seconds
> > > > >
> > > > > Details
> > > > > ##############################
> > > > > Test: CheckPatch - FAIL
> > > > > Desc: Run checkpatch.pl script
> > > > > Output:
> > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > #49:
> > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > >
> > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > #50:
> > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > >
> > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > >
> > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > >
> > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > >
> > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > >
> > > > > NOTE: If any of the errors are false positives, please report
> > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > >
> > > > >
> > > > > ##############################
> > > > > Test: ScanBuild - WARNING
> > > > > Desc: Run Scan Build
> > > > > Output:
> > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > 1 warning generated.
> > > > >
> > > > >
> > > > >
> > > > > ---
> > > > > Regards,
> > > > > Linux Bluetooth
> > > >
> > > > I tried running on fedora but it looks like something is missing in my setup:
> > > >
> > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > autolaun1
> > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > pipewire__module_init()] Failed to connect to session bus:
> > > > Input/output error
> > >
> > > I have those two "errors" too, they should be harmless as the xdg-
> > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > should get the Bluetooth endpoints to appear.
> > >
> > > Removing the mod.portal from the configuration can't be done with drop-
> > > in config files currently, and I didn't want to include a full config
> > > file here as it has some tens of lines of mandatory boilerplate.
> >
> > Ok, looks like it configuring the endpoints:
> >
> > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> >
> > That said I probably need to compile pipewire from source in order to
> > have the LE Audio endpoints registered, btw do you have any
> > instructions on how to build it, do I need to build wireplumber as
> > well?
>
> Should be along these lines to install under $PWD/install and enable
> BAP parts:
>
> git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> cd pipewire
> LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
>    meson setup builddir --prefix=$PWD/install \
>        -Dsystemd-user-service=disabled \
>        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
>        -Dbluez5-codec-lc3=enabled
> meson install -C builddir
>
> Meson will clone and install Wireplumber too. Turning experimental on
> in src/main.conf and it should give
>
> $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> ...
> bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> ...

Thanks, something is not quite right for me since I don't see any
objects on MediaEndpointLE:

https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d

Ive used the following command line though to use a real controller:

sudo tools/test-runner -u -d
-A/home/vudentz/git/pipewire/install/bin/pipewire -k
~/git/linux/arch/x86/boot/bzImage -- /bin/bash

Starting with -l instead of -u didn't seem to fix the problem either,
starting bluetoothctl -e does work though:

bluetoothd[35]: profiles/audio/media.c:endpoint_init_pac() PAC
:1.2:/local/endpoint/ep2 registered
bluetoothd[35]: Endpoint registered: sender=:1.2 path=/local/endpoint/ep2

> --
> Pauli Virtanen
Pauli Virtanen May 17, 2023, 9:16 p.m. UTC | #8
Hi Luiz,

ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
> 
> On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > 
> > Hi Luiz,
> > 
> > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > Hi Pauli,
> > > 
> > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > 
> > > > Hi Luiz,
> > > > 
> > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > Hi Pauli,
> > > > > 
> > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > 
> > > > > > ---Test result---
> > > > > > 
> > > > > > Test Summary:
> > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > GitLint                       PASS      0.52 seconds
> > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > 
> > > > > > Details
> > > > > > ##############################
> > > > > > Test: CheckPatch - FAIL
> > > > > > Desc: Run checkpatch.pl script
> > > > > > Output:
> > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > #49:
> > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > 
> > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > #50:
> > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > 
> > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > 
> > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > 
> > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > 
> > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > 
> > > > > > NOTE: If any of the errors are false positives, please report
> > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > 
> > > > > > 
> > > > > > ##############################
> > > > > > Test: ScanBuild - WARNING
> > > > > > Desc: Run Scan Build
> > > > > > Output:
> > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > 1 warning generated.
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > ---
> > > > > > Regards,
> > > > > > Linux Bluetooth
> > > > > 
> > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > 
> > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > autolaun1
> > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > Input/output error
> > > > 
> > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > should get the Bluetooth endpoints to appear.
> > > > 
> > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > in config files currently, and I didn't want to include a full config
> > > > file here as it has some tens of lines of mandatory boilerplate.
> > > 
> > > Ok, looks like it configuring the endpoints:
> > > 
> > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > 
> > > That said I probably need to compile pipewire from source in order to
> > > have the LE Audio endpoints registered, btw do you have any
> > > instructions on how to build it, do I need to build wireplumber as
> > > well?
> > 
> > Should be along these lines to install under $PWD/install and enable
> > BAP parts:
> > 
> > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > cd pipewire
> > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> >    meson setup builddir --prefix=$PWD/install \
> >        -Dsystemd-user-service=disabled \
> >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> >        -Dbluez5-codec-lc3=enabled
> > meson install -C builddir
> > 
> > Meson will clone and install Wireplumber too. Turning experimental on
> > in src/main.conf and it should give
> > 
> > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > ...
> > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > ...
> 
> Thanks, something is not quite right for me since I don't see any
> objects on MediaEndpointLE:
> 
> https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d

It gets "No object received" when trying to register the BAP
application, so probably the LC3 codec is missing somehow.

meson setup should have reported 
"Bluetooth audio codecs ... LC3 : YES".

`ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
the install/ prefix and not to system lib. If not, the rpath in
compilation accordingly should be adjusted accordingly, or set
LD_LIBRARY_PATH there.

Also the codec plugin 
install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.


> Ive used the following command line though to use a real controller:
> 
> sudo tools/test-runner -u -d
> -A/home/vudentz/git/pipewire/install/bin/pipewire -k
> ~/git/linux/arch/x86/boot/bzImage -- /bin/bash
> 
> Starting with -l instead of -u didn't seem to fix the problem either,
> starting bluetoothctl -e does work though:
> 
> bluetoothd[35]: profiles/audio/media.c:endpoint_init_pac() PAC
> :1.2:/local/endpoint/ep2 registered
> bluetoothd[35]: Endpoint registered: sender=:1.2 path=/local/endpoint/ep2
Luiz Augusto von Dentz May 17, 2023, 9:25 p.m. UTC | #9
Hi Pauli,

On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
>
> Hi Luiz,
>
> ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > Hi Pauli,
> >
> > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > >
> > > Hi Luiz,
> > >
> > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > Hi Pauli,
> > > >
> > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > >
> > > > > Hi Luiz,
> > > > >
> > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > Hi Pauli,
> > > > > >
> > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > >
> > > > > > > ---Test result---
> > > > > > >
> > > > > > > Test Summary:
> > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > >
> > > > > > > Details
> > > > > > > ##############################
> > > > > > > Test: CheckPatch - FAIL
> > > > > > > Desc: Run checkpatch.pl script
> > > > > > > Output:
> > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > #49:
> > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > >
> > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > #50:
> > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > >
> > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > >
> > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > >
> > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > >
> > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > >
> > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > >
> > > > > > >
> > > > > > > ##############################
> > > > > > > Test: ScanBuild - WARNING
> > > > > > > Desc: Run Scan Build
> > > > > > > Output:
> > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > 1 warning generated.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > ---
> > > > > > > Regards,
> > > > > > > Linux Bluetooth
> > > > > >
> > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > >
> > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > autolaun1
> > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > Input/output error
> > > > >
> > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > should get the Bluetooth endpoints to appear.
> > > > >
> > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > in config files currently, and I didn't want to include a full config
> > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > >
> > > > Ok, looks like it configuring the endpoints:
> > > >
> > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > >
> > > > That said I probably need to compile pipewire from source in order to
> > > > have the LE Audio endpoints registered, btw do you have any
> > > > instructions on how to build it, do I need to build wireplumber as
> > > > well?
> > >
> > > Should be along these lines to install under $PWD/install and enable
> > > BAP parts:
> > >
> > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > cd pipewire
> > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > >    meson setup builddir --prefix=$PWD/install \
> > >        -Dsystemd-user-service=disabled \
> > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > >        -Dbluez5-codec-lc3=enabled
> > > meson install -C builddir
> > >
> > > Meson will clone and install Wireplumber too. Turning experimental on
> > > in src/main.conf and it should give
> > >
> > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > ...
> > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > ...
> >
> > Thanks, something is not quite right for me since I don't see any
> > objects on MediaEndpointLE:
> >
> > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
>
> It gets "No object received" when trying to register the BAP
> application, so probably the LC3 codec is missing somehow.
>
> meson setup should have reported
> "Bluetooth audio codecs ... LC3 : YES".

Yep, that shows as enabled.

> `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> the install/ prefix and not to system lib. If not, the rpath in
> compilation accordingly should be adjusted accordingly, or set
> LD_LIBRARY_PATH there.

libpipewire-0.3.so.0 =>
/home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
(0x00007f3b8e947000)

> Also the codec plugin
> install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.

-rwxr-xr-x. 1 **** **** 166592 May 17 13:39
install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so

Perhaps we need to enable more debugs for pipewire to sort this out?
Pauli Virtanen May 17, 2023, 9:42 p.m. UTC | #10
ke, 2023-05-17 kello 14:25 -0700, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
> 
> On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
> > 
> > Hi Luiz,
> > 
> > ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > > Hi Pauli,
> > > 
> > > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > 
> > > > Hi Luiz,
> > > > 
> > > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > Hi Pauli,
> > > > > 
> > > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > 
> > > > > > Hi Luiz,
> > > > > > 
> > > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > Hi Pauli,
> > > > > > > 
> > > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > > > 
> > > > > > > > ---Test result---
> > > > > > > > 
> > > > > > > > Test Summary:
> > > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > > > 
> > > > > > > > Details
> > > > > > > > ##############################
> > > > > > > > Test: CheckPatch - FAIL
> > > > > > > > Desc: Run checkpatch.pl script
> > > > > > > > Output:
> > > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > > #49:
> > > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > > > 
> > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > > #50:
> > > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > > > 
> > > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > > > 
> > > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > > > 
> > > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > > > 
> > > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > > > 
> > > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > > > 
> > > > > > > > 
> > > > > > > > ##############################
> > > > > > > > Test: ScanBuild - WARNING
> > > > > > > > Desc: Run Scan Build
> > > > > > > > Output:
> > > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > > 1 warning generated.
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > ---
> > > > > > > > Regards,
> > > > > > > > Linux Bluetooth
> > > > > > > 
> > > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > > > 
> > > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > > autolaun1
> > > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > > Input/output error
> > > > > > 
> > > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > > should get the Bluetooth endpoints to appear.
> > > > > > 
> > > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > > in config files currently, and I didn't want to include a full config
> > > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > > > 
> > > > > Ok, looks like it configuring the endpoints:
> > > > > 
> > > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > > > 
> > > > > That said I probably need to compile pipewire from source in order to
> > > > > have the LE Audio endpoints registered, btw do you have any
> > > > > instructions on how to build it, do I need to build wireplumber as
> > > > > well?
> > > > 
> > > > Should be along these lines to install under $PWD/install and enable
> > > > BAP parts:
> > > > 
> > > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > > cd pipewire
> > > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > > >    meson setup builddir --prefix=$PWD/install \
> > > >        -Dsystemd-user-service=disabled \
> > > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > > >        -Dbluez5-codec-lc3=enabled
> > > > meson install -C builddir
> > > > 
> > > > Meson will clone and install Wireplumber too. Turning experimental on
> > > > in src/main.conf and it should give
> > > > 
> > > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > > ...
> > > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > > ...
> > > 
> > > Thanks, something is not quite right for me since I don't see any
> > > objects on MediaEndpointLE:
> > > 
> > > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
> > 
> > It gets "No object received" when trying to register the BAP
> > application, so probably the LC3 codec is missing somehow.
> > 
> > meson setup should have reported
> > "Bluetooth audio codecs ... LC3 : YES".
> 
> Yep, that shows as enabled.
> 
> > `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> > the install/ prefix and not to system lib. If not, the rpath in
> > compilation accordingly should be adjusted accordingly, or set
> > LD_LIBRARY_PATH there.
> 
> libpipewire-0.3.so.0 =>
> /home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
> (0x00007f3b8e947000)
> 
> > Also the codec plugin
> > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.
> 
> -rwxr-xr-x. 1 **** **** 166592 May 17 13:39
> install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so

Does `ldd libspa-codec-bluez5-lc3.so` show it finds liblc3.so?

> Perhaps we need to enable more debugs for pipewire to sort this out?

./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage -- /bin/bash
... inside vm: ...
pkill wireplumber
export XDG_CONFIG_HOME=/run/conf
export XDG_STATE_HOME=/run
export XDG_RUNTIME_DIR=/run
export WIREPLUMBER_DEBUG=4
/home/pauli/tmp/pipewire/install/bin/wireplumber

should spit out what it tried to do.
Luiz Augusto von Dentz May 17, 2023, 9:50 p.m. UTC | #11
Hi Pauli,

On Wed, May 17, 2023 at 2:43 PM Pauli Virtanen <pav@iki.fi> wrote:
>
> ke, 2023-05-17 kello 14:25 -0700, Luiz Augusto von Dentz kirjoitti:
> > Hi Pauli,
> >
> > On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
> > >
> > > Hi Luiz,
> > >
> > > ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > Hi Pauli,
> > > >
> > > > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > >
> > > > > Hi Luiz,
> > > > >
> > > > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > Hi Pauli,
> > > > > >
> > > > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > >
> > > > > > > Hi Luiz,
> > > > > > >
> > > > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > > Hi Pauli,
> > > > > > > >
> > > > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > > > >
> > > > > > > > > ---Test result---
> > > > > > > > >
> > > > > > > > > Test Summary:
> > > > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > > > >
> > > > > > > > > Details
> > > > > > > > > ##############################
> > > > > > > > > Test: CheckPatch - FAIL
> > > > > > > > > Desc: Run checkpatch.pl script
> > > > > > > > > Output:
> > > > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > > > #49:
> > > > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > > > >
> > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > > > #50:
> > > > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > > > >
> > > > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > > > >
> > > > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > > > >
> > > > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > > > >
> > > > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > > > >
> > > > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > ##############################
> > > > > > > > > Test: ScanBuild - WARNING
> > > > > > > > > Desc: Run Scan Build
> > > > > > > > > Output:
> > > > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > > > 1 warning generated.
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > ---
> > > > > > > > > Regards,
> > > > > > > > > Linux Bluetooth
> > > > > > > >
> > > > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > > > >
> > > > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > > > autolaun1
> > > > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > > > Input/output error
> > > > > > >
> > > > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > > > should get the Bluetooth endpoints to appear.
> > > > > > >
> > > > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > > > in config files currently, and I didn't want to include a full config
> > > > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > > > >
> > > > > > Ok, looks like it configuring the endpoints:
> > > > > >
> > > > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > > > >
> > > > > > That said I probably need to compile pipewire from source in order to
> > > > > > have the LE Audio endpoints registered, btw do you have any
> > > > > > instructions on how to build it, do I need to build wireplumber as
> > > > > > well?
> > > > >
> > > > > Should be along these lines to install under $PWD/install and enable
> > > > > BAP parts:
> > > > >
> > > > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > > > cd pipewire
> > > > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > > > >    meson setup builddir --prefix=$PWD/install \
> > > > >        -Dsystemd-user-service=disabled \
> > > > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > > > >        -Dbluez5-codec-lc3=enabled
> > > > > meson install -C builddir
> > > > >
> > > > > Meson will clone and install Wireplumber too. Turning experimental on
> > > > > in src/main.conf and it should give
> > > > >
> > > > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > > > ...
> > > > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > > > ...
> > > >
> > > > Thanks, something is not quite right for me since I don't see any
> > > > objects on MediaEndpointLE:
> > > >
> > > > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
> > >
> > > It gets "No object received" when trying to register the BAP
> > > application, so probably the LC3 codec is missing somehow.
> > >
> > > meson setup should have reported
> > > "Bluetooth audio codecs ... LC3 : YES".
> >
> > Yep, that shows as enabled.
> >
> > > `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> > > the install/ prefix and not to system lib. If not, the rpath in
> > > compilation accordingly should be adjusted accordingly, or set
> > > LD_LIBRARY_PATH there.
> >
> > libpipewire-0.3.so.0 =>
> > /home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
> > (0x00007f3b8e947000)
> >
> > > Also the codec plugin
> > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.
> >
> > -rwxr-xr-x. 1 **** **** 166592 May 17 13:39
> > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so
>
> Does `ldd libspa-codec-bluez5-lc3.so` show it finds liblc3.so?

I guess we found the problem:

    liblc3.so.1 => not found

> > Perhaps we need to enable more debugs for pipewire to sort this out?
>
> ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage -- /bin/bash
> ... inside vm: ...
> pkill wireplumber
> export XDG_CONFIG_HOME=/run/conf
> export XDG_STATE_HOME=/run
> export XDG_RUNTIME_DIR=/run
> export WIREPLUMBER_DEBUG=4
> /home/pauli/tmp/pipewire/install/bin/wireplumber
>
> should spit out what it tried to do.

Yep, here is the full trace:

https://gist.github.com/Vudentz/36f9acdc62b623c85e978883ea29c2d3

But I think the problem is with the linking of liblc3, are you running
with the latest from git? Perhaps I need to install it also in the
same path?
Luiz Augusto von Dentz May 17, 2023, 9:58 p.m. UTC | #12
Hi Pauli,

On Wed, May 17, 2023 at 2:50 PM Luiz Augusto von Dentz
<luiz.dentz@gmail.com> wrote:
>
> Hi Pauli,
>
> On Wed, May 17, 2023 at 2:43 PM Pauli Virtanen <pav@iki.fi> wrote:
> >
> > ke, 2023-05-17 kello 14:25 -0700, Luiz Augusto von Dentz kirjoitti:
> > > Hi Pauli,
> > >
> > > On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > >
> > > > Hi Luiz,
> > > >
> > > > ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > Hi Pauli,
> > > > >
> > > > > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > >
> > > > > > Hi Luiz,
> > > > > >
> > > > > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > Hi Pauli,
> > > > > > >
> > > > > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > > >
> > > > > > > > Hi Luiz,
> > > > > > > >
> > > > > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > > > Hi Pauli,
> > > > > > > > >
> > > > > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > > > > >
> > > > > > > > > > ---Test result---
> > > > > > > > > >
> > > > > > > > > > Test Summary:
> > > > > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > > > > >
> > > > > > > > > > Details
> > > > > > > > > > ##############################
> > > > > > > > > > Test: CheckPatch - FAIL
> > > > > > > > > > Desc: Run checkpatch.pl script
> > > > > > > > > > Output:
> > > > > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > > > > #49:
> > > > > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > > > > >
> > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > > > > #50:
> > > > > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > > > > >
> > > > > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > > > > >
> > > > > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > > > > >
> > > > > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > > > > >
> > > > > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > > > > >
> > > > > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > ##############################
> > > > > > > > > > Test: ScanBuild - WARNING
> > > > > > > > > > Desc: Run Scan Build
> > > > > > > > > > Output:
> > > > > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > > > > 1 warning generated.
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > ---
> > > > > > > > > > Regards,
> > > > > > > > > > Linux Bluetooth
> > > > > > > > >
> > > > > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > > > > >
> > > > > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > > > > autolaun1
> > > > > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > > > > Input/output error
> > > > > > > >
> > > > > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > > > > should get the Bluetooth endpoints to appear.
> > > > > > > >
> > > > > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > > > > in config files currently, and I didn't want to include a full config
> > > > > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > > > > >
> > > > > > > Ok, looks like it configuring the endpoints:
> > > > > > >
> > > > > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > > > > >
> > > > > > > That said I probably need to compile pipewire from source in order to
> > > > > > > have the LE Audio endpoints registered, btw do you have any
> > > > > > > instructions on how to build it, do I need to build wireplumber as
> > > > > > > well?
> > > > > >
> > > > > > Should be along these lines to install under $PWD/install and enable
> > > > > > BAP parts:
> > > > > >
> > > > > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > > > > cd pipewire
> > > > > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > > > > >    meson setup builddir --prefix=$PWD/install \
> > > > > >        -Dsystemd-user-service=disabled \
> > > > > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > > > > >        -Dbluez5-codec-lc3=enabled
> > > > > > meson install -C builddir
> > > > > >
> > > > > > Meson will clone and install Wireplumber too. Turning experimental on
> > > > > > in src/main.conf and it should give
> > > > > >
> > > > > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > > > > ...
> > > > > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > > > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > > > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > > > > ...
> > > > >
> > > > > Thanks, something is not quite right for me since I don't see any
> > > > > objects on MediaEndpointLE:
> > > > >
> > > > > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
> > > >
> > > > It gets "No object received" when trying to register the BAP
> > > > application, so probably the LC3 codec is missing somehow.
> > > >
> > > > meson setup should have reported
> > > > "Bluetooth audio codecs ... LC3 : YES".
> > >
> > > Yep, that shows as enabled.
> > >
> > > > `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> > > > the install/ prefix and not to system lib. If not, the rpath in
> > > > compilation accordingly should be adjusted accordingly, or set
> > > > LD_LIBRARY_PATH there.
> > >
> > > libpipewire-0.3.so.0 =>
> > > /home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
> > > (0x00007f3b8e947000)
> > >
> > > > Also the codec plugin
> > > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.
> > >
> > > -rwxr-xr-x. 1 **** **** 166592 May 17 13:39
> > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so
> >
> > Does `ldd libspa-codec-bluez5-lc3.so` show it finds liblc3.so?
>
> I guess we found the problem:
>
>     liblc3.so.1 => not found

sudo ln -s liblc3.so liblc3.so.1 fixed it for me, not sure if we
should inform liblc3 folks that this is not done properly when
installing or what.
Pauli Virtanen May 17, 2023, 10:05 p.m. UTC | #13
Hi,

ke, 2023-05-17 kello 14:50 -0700, Luiz Augusto von Dentz kirjoitti:
> Hi Pauli,
> 
> On Wed, May 17, 2023 at 2:43 PM Pauli Virtanen <pav@iki.fi> wrote:
> > 
> > ke, 2023-05-17 kello 14:25 -0700, Luiz Augusto von Dentz kirjoitti:
> > > Hi Pauli,
> > > 
> > > On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > 
> > > > Hi Luiz,
> > > > 
> > > > ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > Hi Pauli,
> > > > > 
> > > > > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > 
> > > > > > Hi Luiz,
> > > > > > 
> > > > > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > Hi Pauli,
> > > > > > > 
> > > > > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > > > 
> > > > > > > > Hi Luiz,
> > > > > > > > 
> > > > > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > > > Hi Pauli,
> > > > > > > > > 
> > > > > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > > > > > 
> > > > > > > > > > ---Test result---
> > > > > > > > > > 
> > > > > > > > > > Test Summary:
> > > > > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > > > > > 
> > > > > > > > > > Details
> > > > > > > > > > ##############################
> > > > > > > > > > Test: CheckPatch - FAIL
> > > > > > > > > > Desc: Run checkpatch.pl script
> > > > > > > > > > Output:
> > > > > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > > > > #49:
> > > > > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > > > > > 
> > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > > > > #50:
> > > > > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > > > > > 
> > > > > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > > > > > 
> > > > > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > > > > > 
> > > > > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > > > > > 
> > > > > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > > > > > 
> > > > > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > ##############################
> > > > > > > > > > Test: ScanBuild - WARNING
> > > > > > > > > > Desc: Run Scan Build
> > > > > > > > > > Output:
> > > > > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > > > > 1 warning generated.
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > ---
> > > > > > > > > > Regards,
> > > > > > > > > > Linux Bluetooth
> > > > > > > > > 
> > > > > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > > > > > 
> > > > > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > > > > autolaun1
> > > > > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > > > > Input/output error
> > > > > > > > 
> > > > > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > > > > should get the Bluetooth endpoints to appear.
> > > > > > > > 
> > > > > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > > > > in config files currently, and I didn't want to include a full config
> > > > > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > > > > > 
> > > > > > > Ok, looks like it configuring the endpoints:
> > > > > > > 
> > > > > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > > > > > 
> > > > > > > That said I probably need to compile pipewire from source in order to
> > > > > > > have the LE Audio endpoints registered, btw do you have any
> > > > > > > instructions on how to build it, do I need to build wireplumber as
> > > > > > > well?
> > > > > > 
> > > > > > Should be along these lines to install under $PWD/install and enable
> > > > > > BAP parts:
> > > > > > 
> > > > > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > > > > cd pipewire
> > > > > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > > > > >    meson setup builddir --prefix=$PWD/install \
> > > > > >        -Dsystemd-user-service=disabled \
> > > > > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > > > > >        -Dbluez5-codec-lc3=enabled
> > > > > > meson install -C builddir
> > > > > > 
> > > > > > Meson will clone and install Wireplumber too. Turning experimental on
> > > > > > in src/main.conf and it should give
> > > > > > 
> > > > > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > > > > ...
> > > > > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > > > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > > > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > > > > ...
> > > > > 
> > > > > Thanks, something is not quite right for me since I don't see any
> > > > > objects on MediaEndpointLE:
> > > > > 
> > > > > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
> > > > 
> > > > It gets "No object received" when trying to register the BAP
> > > > application, so probably the LC3 codec is missing somehow.
> > > > 
> > > > meson setup should have reported
> > > > "Bluetooth audio codecs ... LC3 : YES".
> > > 
> > > Yep, that shows as enabled.
> > > 
> > > > `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> > > > the install/ prefix and not to system lib. If not, the rpath in
> > > > compilation accordingly should be adjusted accordingly, or set
> > > > LD_LIBRARY_PATH there.
> > > 
> > > libpipewire-0.3.so.0 =>
> > > /home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
> > > (0x00007f3b8e947000)
> > > 
> > > > Also the codec plugin
> > > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.
> > > 
> > > -rwxr-xr-x. 1 **** **** 166592 May 17 13:39
> > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so
> > 
> > Does `ldd libspa-codec-bluez5-lc3.so` show it finds liblc3.so?
> 
> I guess we found the problem:
> 
>     liblc3.so.1 => not found
> 
> > > Perhaps we need to enable more debugs for pipewire to sort this out?
> > 
> > ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage -- /bin/bash
> > ... inside vm: ...
> > pkill wireplumber
> > export XDG_CONFIG_HOME=/run/conf
> > export XDG_STATE_HOME=/run
> > export XDG_RUNTIME_DIR=/run
> > export WIREPLUMBER_DEBUG=4
> > /home/pauli/tmp/pipewire/install/bin/wireplumber
> > 
> > should spit out what it tried to do.
> 
> Yep, here is the full trace:
> 
> https://gist.github.com/Vudentz/36f9acdc62b623c85e978883ea29c2d3
> 
> But I think the problem is with the linking of liblc3, are you running
> with the latest from git? Perhaps I need to install it also in the
> same path?

Copying liblc3.so* under install/lib64/ probably works since rpath is
set there. Or maybe /usr/local if not.

I've been running on some older liblc3 git version, but current liblc3
git 1.0.3 also works.
Luiz Augusto von Dentz May 18, 2023, 7:20 p.m. UTC | #14
Hi Pauli,

On Wed, May 17, 2023 at 3:08 PM Pauli Virtanen <pav@iki.fi> wrote:
>
> Hi,
>
> ke, 2023-05-17 kello 14:50 -0700, Luiz Augusto von Dentz kirjoitti:
> > Hi Pauli,
> >
> > On Wed, May 17, 2023 at 2:43 PM Pauli Virtanen <pav@iki.fi> wrote:
> > >
> > > ke, 2023-05-17 kello 14:25 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > Hi Pauli,
> > > >
> > > > On Wed, May 17, 2023 at 2:16 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > >
> > > > > Hi Luiz,
> > > > >
> > > > > ke, 2023-05-17 kello 13:53 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > Hi Pauli,
> > > > > >
> > > > > > On Wed, May 17, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > >
> > > > > > > Hi Luiz,
> > > > > > >
> > > > > > > ke, 2023-05-17 kello 12:34 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > > Hi Pauli,
> > > > > > > >
> > > > > > > > On Mon, May 15, 2023 at 1:30 PM Pauli Virtanen <pav@iki.fi> wrote:
> > > > > > > > >
> > > > > > > > > Hi Luiz,
> > > > > > > > >
> > > > > > > > > ma, 2023-05-15 kello 13:01 -0700, Luiz Augusto von Dentz kirjoitti:
> > > > > > > > > > Hi Pauli,
> > > > > > > > > >
> > > > > > > > > > On Sat, May 13, 2023 at 10:41 AM <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=747273
> > > > > > > > > > >
> > > > > > > > > > > ---Test result---
> > > > > > > > > > >
> > > > > > > > > > > Test Summary:
> > > > > > > > > > > CheckPatch                    FAIL      1.14 seconds
> > > > > > > > > > > GitLint                       PASS      0.52 seconds
> > > > > > > > > > > BuildEll                      PASS      26.21 seconds
> > > > > > > > > > > BluezMake                     PASS      745.32 seconds
> > > > > > > > > > > MakeCheck                     PASS      11.04 seconds
> > > > > > > > > > > MakeDistcheck                 PASS      151.18 seconds
> > > > > > > > > > > CheckValgrind                 PASS      243.86 seconds
> > > > > > > > > > > CheckSmatch                   PASS      326.99 seconds
> > > > > > > > > > > bluezmakeextell               PASS      98.83 seconds
> > > > > > > > > > > IncrementalBuild              PASS      1262.88 seconds
> > > > > > > > > > > ScanBuild                     WARNING   964.73 seconds
> > > > > > > > > > >
> > > > > > > > > > > Details
> > > > > > > > > > > ##############################
> > > > > > > > > > > Test: CheckPatch - FAIL
> > > > > > > > > > > Desc: Run checkpatch.pl script
> > > > > > > > > > > Output:
> > > > > > > > > > > [BlueZ,v3,1/2] test-runner: revert udevd and audio support
> > > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id '91a48af52efb0751fab396b2b9026c9186b10b88', maybe rebased or not pulled?
> > > > > > > > > > > #49:
> > > > > > > > > > > This reverts commit 91a48af52efb0751fab396b2b9026c9186b10b88
> > > > > > > > > > >
> > > > > > > > > > > WARNING:UNKNOWN_COMMIT_ID: Unknown commit id 'e20e7e0b05c7edb74255c9b092916ac5bb99c97f', maybe rebased or not pulled?
> > > > > > > > > > > #50:
> > > > > > > > > > > This reverts commit e20e7e0b05c7edb74255c9b092916ac5bb99c97f
> > > > > > > > > > >
> > > > > > > > > > > /github/workspace/src/src/13240258.patch total: 0 errors, 2 warnings, 193 lines checked
> > > > > > > > > > >
> > > > > > > > > > > NOTE: For some of the reported defects, checkpatch may be able to
> > > > > > > > > > >       mechanically convert to the typical style using --fix or --fix-inplace.
> > > > > > > > > > >
> > > > > > > > > > > /github/workspace/src/src/13240258.patch has style problems, please review.
> > > > > > > > > > >
> > > > > > > > > > > NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
> > > > > > > > > > >
> > > > > > > > > > > NOTE: If any of the errors are false positives, please report
> > > > > > > > > > >       them to the maintainer, see CHECKPATCH in MAINTAINERS.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > ##############################
> > > > > > > > > > > Test: ScanBuild - WARNING
> > > > > > > > > > > Desc: Run Scan Build
> > > > > > > > > > > Output:
> > > > > > > > > > > tools/test-runner.c:924:2: warning: 2nd function call argument is an uninitialized value
> > > > > > > > > > >         printf("Running command %s\n", cmdname ? cmdname : argv[0]);
> > > > > > > > > > >         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > > > > > > > > > > 1 warning generated.
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > ---
> > > > > > > > > > > Regards,
> > > > > > > > > > > Linux Bluetooth
> > > > > > > > > >
> > > > > > > > > > I tried running on fedora but it looks like something is missing in my setup:
> > > > > > > > > >
> > > > > > > > > > [E][00003.767959] spa.dbus     | [          dbus.c:  329
> > > > > > > > > > impl_connection_get()] Failed to connect to session bus: Unable to
> > > > > > > > > > autolaun1
> > > > > > > > > > [E][00003.769967] mod.portal   | [ module-portal.c:  326
> > > > > > > > > > pipewire__module_init()] Failed to connect to session bus:
> > > > > > > > > > Input/output error
> > > > > > > > >
> > > > > > > > > I have those two "errors" too, they should be harmless as the xdg-
> > > > > > > > > desktop-portal module is optional. With `test-runner -d -l -A` you
> > > > > > > > > should get the Bluetooth endpoints to appear.
> > > > > > > > >
> > > > > > > > > Removing the mod.portal from the configuration can't be done with drop-
> > > > > > > > > in config files currently, and I didn't want to include a full config
> > > > > > > > > file here as it has some tens of lines of mandatory boilerplate.
> > > > > > > >
> > > > > > > > Ok, looks like it configuring the endpoints:
> > > > > > > >
> > > > > > > > https://gist.github.com/Vudentz/0d8851e719affd9f2dc1f4081ce32fb7
> > > > > > > >
> > > > > > > > That said I probably need to compile pipewire from source in order to
> > > > > > > > have the LE Audio endpoints registered, btw do you have any
> > > > > > > > instructions on how to build it, do I need to build wireplumber as
> > > > > > > > well?
> > > > > > >
> > > > > > > Should be along these lines to install under $PWD/install and enable
> > > > > > > BAP parts:
> > > > > > >
> > > > > > > git clone https://gitlab.freedesktop.org/pipewire/pipewire.git
> > > > > > > cd pipewire
> > > > > > > LDFLAGS="-Wl,-rpath=$PWD/install/lib64" \
> > > > > > >    meson setup builddir --prefix=$PWD/install \
> > > > > > >        -Dsystemd-user-service=disabled \
> > > > > > >        -Dudevrulesdir=$PWD/install/lib/udev/rules.d \
> > > > > > >        -Dbluez5-codec-lc3=enabled
> > > > > > > meson install -C builddir
> > > > > > >
> > > > > > > Meson will clone and install Wireplumber too. Turning experimental on
> > > > > > > in src/main.conf and it should give
> > > > > > >
> > > > > > > $ ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage sleep 5
> > > > > > > ...
> > > > > > > bluetoothd[29]: profiles/audio/media.c:endpoint_init_pac() PAC :1.1:/MediaEndpointLE/BAPSource/lc3 registered
> > > > > > > bluetoothd[29]: Endpoint registered: sender=:1.1 path=/MediaEndpointLE/BAPSource/lc3
> > > > > > > bluetoothd[29]: profiles/audio/media.c:client_ready_cb() Media application registered: :1.1:/MediaEndpointLE
> > > > > > > ...
> > > > > >
> > > > > > Thanks, something is not quite right for me since I don't see any
> > > > > > objects on MediaEndpointLE:
> > > > > >
> > > > > > https://gist.github.com/Vudentz/242e4e2bfb8907acfdfdf18b458eec1d
> > > > >
> > > > > It gets "No object received" when trying to register the BAP
> > > > > application, so probably the LC3 codec is missing somehow.
> > > > >
> > > > > meson setup should have reported
> > > > > "Bluetooth audio codecs ... LC3 : YES".
> > > >
> > > > Yep, that shows as enabled.
> > > >
> > > > > `ldd install/bin/pipewire` should link to libpipewire-0.3.so.0 under
> > > > > the install/ prefix and not to system lib. If not, the rpath in
> > > > > compilation accordingly should be adjusted accordingly, or set
> > > > > LD_LIBRARY_PATH there.
> > > >
> > > > libpipewire-0.3.so.0 =>
> > > > /home/vudentz/git/pipewire/install/lib64/libpipewire-0.3.so.0
> > > > (0x00007f3b8e947000)
> > > >
> > > > > Also the codec plugin
> > > > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so should exist.
> > > >
> > > > -rwxr-xr-x. 1 **** **** 166592 May 17 13:39
> > > > install/lib64/spa-0.2/bluez5/libspa-codec-bluez5-lc3.so
> > >
> > > Does `ldd libspa-codec-bluez5-lc3.so` show it finds liblc3.so?
> >
> > I guess we found the problem:
> >
> >     liblc3.so.1 => not found
> >
> > > > Perhaps we need to enable more debugs for pipewire to sort this out?
> > >
> > > ./tools/test-runner -d -l --audio=/home/pauli/tmp/pipewire/install/bin/pipewire -k../linux/arch/x86_64/boot/bzImage -- /bin/bash
> > > ... inside vm: ...
> > > pkill wireplumber
> > > export XDG_CONFIG_HOME=/run/conf
> > > export XDG_STATE_HOME=/run
> > > export XDG_RUNTIME_DIR=/run
> > > export WIREPLUMBER_DEBUG=4
> > > /home/pauli/tmp/pipewire/install/bin/wireplumber
> > >
> > > should spit out what it tried to do.
> >
> > Yep, here is the full trace:
> >
> > https://gist.github.com/Vudentz/36f9acdc62b623c85e978883ea29c2d3
> >
> > But I think the problem is with the linking of liblc3, are you running
> > with the latest from git? Perhaps I need to install it also in the
> > same path?
>
> Copying liblc3.so* under install/lib64/ probably works since rpath is
> set there. Or maybe /usr/local if not.
>
> I've been running on some older liblc3 git version, but current liblc3
> git 1.0.3 also works.

Btw, do you know gjhenrique? He wrote a very detailed post about the
overall architecture of pipewire with respect to Bluetooth:

https://gjhenrique.com/pipewire/

I'm actually thinking on incorporating something he has done for
capturing TX latency, etc, which is probably going to be useful for LE
as well:

https://github.com/gjhenrique/bluetooth_exporter

> --
> Pauli Virtanen
diff mbox series

Patch

diff --git a/tools/test-runner.c b/tools/test-runner.c
index 6660ea8de..cd65c4cf0 100644
--- a/tools/test-runner.c
+++ b/tools/test-runner.c
@@ -54,7 +54,6 @@  static bool start_monitor = false;
 static int num_devs = 0;
 static const char *qemu_binary = NULL;
 static const char *kernel_image = NULL;
-static bool audio_support;
 
 static const char *qemu_table[] = {
 	"qemu-system-x86_64",
@@ -252,16 +251,14 @@  static void start_qemu(void)
 				"acpi=off pci=noacpi noapic quiet ro init=%s "
 				"TESTHOME=%s TESTDBUS=%u TESTDAEMON=%u "
 				"TESTDBUSSESSION=%u XDG_RUNTIME_DIR=/run/user/0 "
-				"TESTAUDIO=%u "
 				"TESTMONITOR=%u TESTEMULATOR=%u TESTDEVS=%d "
 				"TESTAUTO=%u TESTARGS=\'%s\'",
 				initcmd, cwd, start_dbus, start_daemon,
-				start_dbus_session, audio_support,
+				start_dbus_session,
 				start_monitor, start_emulator, num_devs,
 				run_auto, testargs);
 
 	argv = alloca(sizeof(qemu_argv) +
-				(audio_support ? 4 : 0) +
 				(sizeof(char *) * (4 + (num_devs * 4))));
 	memcpy(argv, qemu_argv, sizeof(qemu_argv));
 
@@ -274,24 +271,6 @@  static void start_qemu(void)
 	}
 	argv[0] = (char *) qemu_binary;
 
-	if (audio_support) {
-		char *xdg_runtime_dir, *audiodev;
-
-		xdg_runtime_dir = getenv("XDG_RUNTIME_DIR");
-		if (!xdg_runtime_dir) {
-			fprintf(stderr, "XDG_RUNTIME_DIR not set\n");
-			exit(1);
-		}
-		audiodev = alloca(40 + strlen(xdg_runtime_dir));
-		sprintf(audiodev, "id=audio,driver=pa,server=%s/pulse/native",
-				xdg_runtime_dir);
-
-		argv[pos++] = "-audiodev";
-		argv[pos++] = audiodev;
-		argv[pos++] = "-device";
-		argv[pos++] = "AC97,audiodev=audio";
-	}
-
 	argv[pos++] = "-kernel";
 	argv[pos++] = (char *) kernel_image;
 	argv[pos++] = "-append";
@@ -744,70 +723,13 @@  static pid_t start_btvirt(const char *home)
 	return pid;
 }
 
-static void trigger_udev(void)
-{
-	char *argv[3], *envp[1];
-	pid_t pid;
-
-	argv[0] = "/bin/udevadm";
-	argv[1] = "trigger";
-	argv[2] = NULL;
-
-	envp[0] = NULL;
-
-	printf("Triggering udev events\n");
-
-	pid = fork();
-	if (pid < 0) {
-		perror("Failed to fork new process");
-		return;
-	}
-
-	if (pid == 0) {
-		execve(argv[0], argv, envp);
-		exit(EXIT_SUCCESS);
-	}
-
-	printf("udev trigger process %d created\n", pid);
-}
-
-static pid_t start_udevd(void)
-{
-	char *argv[2], *envp[1];
-	pid_t pid;
-
-	argv[0] = "/lib/systemd/systemd-udevd";
-	argv[1] = NULL;
-
-	envp[0] = NULL;
-
-	printf("Starting udevd daemon\n");
-
-	pid = fork();
-	if (pid < 0) {
-		perror("Failed to fork new process");
-		return -1;
-	}
-
-	if (pid == 0) {
-		execve(argv[0], argv, envp);
-		exit(EXIT_SUCCESS);
-	}
-
-	printf("udevd daemon process %d created\n", pid);
-
-	trigger_udev();
-
-	return pid;
-}
-
 static void run_command(char *cmdname, char *home)
 {
 	char *argv[9], *envp[3];
 	int pos = 0, idx = 0;
 	int serial_fd;
 	pid_t pid, dbus_pid, daemon_pid, monitor_pid, emulator_pid,
-	      dbus_session_pid, udevd_pid;
+	      dbus_session_pid;
 
 	if (!home) {
 		perror("Invalid parameter: TESTHOME");
@@ -828,11 +750,6 @@  static void run_command(char *cmdname, char *home)
 	} else
 		serial_fd = -1;
 
-	if (audio_support)
-		udevd_pid = start_udevd();
-	else
-		udevd_pid = -1;
-
 	if (start_dbus) {
 		create_dbus_system_conf();
 		dbus_pid = start_dbus_daemon(false);
@@ -961,11 +878,6 @@  start_next:
 			monitor_pid = -1;
 		}
 
-		if (corpse == udevd_pid) {
-			printf("udevd terminated\n");
-			udevd_pid = -1;
-		}
-
 		if (corpse == pid)
 			break;
 	}
@@ -990,9 +902,6 @@  start_next:
 	if (monitor_pid > 0)
 		kill(monitor_pid, SIGTERM);
 
-	if (udevd_pid > 0)
-		kill(udevd_pid, SIGTERM);
-
 	if (serial_fd >= 0)
 		close(serial_fd);
 }
@@ -1073,12 +982,6 @@  static void run_tests(void)
 		start_emulator = true;
 	}
 
-	ptr = strstr(cmdline, "TESTAUDIO=1");
-	if (ptr) {
-		printf("Audio support requested\n");
-		audio_support = true;
-	}
-
 	ptr = strstr(cmdline, "TESTHOME=");
 	if (ptr) {
 		home = ptr + 4;
@@ -1105,7 +1008,6 @@  static void usage(void)
 		"\t-u, --unix [path]      Provide serial device\n"
 		"\t-q, --qemu <path>      QEMU binary\n"
 		"\t-k, --kernel <image>   Kernel image (bzImage)\n"
-		"\t-A, --audio            Add audio support\n"
 		"\t-h, --help             Show help options\n");
 }
 
@@ -1120,7 +1022,6 @@  static const struct option main_options[] = {
 	{ "monitor", no_argument,       NULL, 'm' },
 	{ "qemu",    required_argument, NULL, 'q' },
 	{ "kernel",  required_argument, NULL, 'k' },
-	{ "audio",   no_argument,       NULL, 'A' },
 	{ "version", no_argument,       NULL, 'v' },
 	{ "help",    no_argument,       NULL, 'h' },
 	{ }
@@ -1140,7 +1041,7 @@  int main(int argc, char *argv[])
 	for (;;) {
 		int opt;
 
-		opt = getopt_long(argc, argv, "aubdslmq:k:Avh", main_options,
+		opt = getopt_long(argc, argv, "aubdslmq:k:vh", main_options,
 								NULL);
 		if (opt < 0)
 			break;
@@ -1174,9 +1075,6 @@  int main(int argc, char *argv[])
 		case 'k':
 			kernel_image = optarg;
 			break;
-		case 'A':
-			audio_support = true;
-			break;
 		case 'v':
 			printf("%s\n", VERSION);
 			return EXIT_SUCCESS;