Skip to contentSkip to conversation

bt: Mark the bluetooth subsystem as deprecated

PATCH In review Thomas Huth 4 participants 4 messages Nov 12, 2018

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 bt: Mark the bluetooth subsystem as deprecated

It has been unmaintained since years, and there were only trivial or tree-wide changes to the related files since many years, so the code is likely very bitrotten and broken. For example the following segfaults as soon as as you press a key:

qemu-system-x86_64 -usb -device usb-bt-dongle -bt hci -bt device:keyboard

Since we are not aware of anybody using bluetooth with the current version of QEMU, let's mark the subsystem as deprecated, with a special request for the users to write to the qemu-devel mailing list in case they still use it (so we could revert the deprecation status in that case).

---
 qemu-deprecated.texi | 7 +++++++
 qemu-options.hx      | 4 ++++
 vl.c                 | 4 ++++
 3 files changed, 15 insertions(+)
qemu-deprecated.texi +7 -0
index 5d2d7a3..cb4291f 100644
@@ -128,6 +128,13 @@ The @option{[hub_id name]} parameter tuple of the 'hostfwd_add' and
 The ``ivshmem'' device type is replaced by either the ``ivshmem-plain''
 or ``ivshmem-doorbell`` device types.
 
+@subsection bluetooth (since 3.1)
+
+The bluetooth subsystem is unmaintained since many years and likely bitrotten
+quite a bit. It will be removed without replacement unless some users speaks
+up at the @email{qemu-devel@@nongnu.org} mailing list with information about
+their usecases.
+
 @section System emulator machines
 
 @subsection pc-0.10 and pc-0.11 (since 3.0)
qemu-options.hx +4 -0
index 38c7a97..ee379b3 100644
@@ -2772,6 +2772,10 @@ logic.  The Transport Layer is decided by the machine type.  Currently
 the machines @code{n800} and @code{n810} have one HCI and all other
 machines have none.
 
+Note: This option and the whole bluetooth subsystem is considered as deprecated.
+If you still use it, please send a mail to @email{qemu-devel@@nongnu.org} where
+you describe your usecase.
+
 @anchor{bt-hcis}
 The following three types are recognized:
 
vl.c +4 -0
index 55bab00..fa25d1a 100644
@@ -3269,6 +3269,10 @@ int main(int argc, char **argv, char **envp)
                 break;
 #endif
             case QEMU_OPTION_bt:
+                warn_report("The bluetooth subsystem is deprecated and will "
+                            "be removed soon. If the bluetooth subsystem is "
+                            "still useful for you, please send a mail to "
+                            "[email protected] with your usecase.");
                 add_device_config(DEV_BT, optarg);
                 break;
             case QEMU_OPTION_audio_help:
Signature · 1 line
1.8.3.1
#1 lore raw
Quoted 13 lines · Thomas Huth

It has been unmaintained since years, and there were only trivial or tree-wide changes to the related files since many years, so the code is likely very bitrotten and broken. For example the following segfaults as soon as as you press a key:

qemu-system-x86_64 -usb -device usb-bt-dongle -bt hci -bt device:keyboard

Since we are not aware of anybody using bluetooth with the current version of QEMU, let's mark the subsystem as deprecated, with a special request for the users to write to the qemu-devel mailing list in case they still use it (so we could revert the deprecation status in that case).

Quoted 6 lines
---
 qemu-deprecated.texi | 7 +++++++
 qemu-options.hx      | 4 ++++
 vl.c                 | 4 ++++
 3 files changed, 15 insertions(+)

libvirt never implemented any configuration option for bluetooth

#2 lore raw
In reply to Thomas Huth
Quoted 5 lines · Thomas Huth

It has been unmaintained since years, and there were only trivial or tree-wide changes to the related files since many years, so the code is likely very bitrotten and broken. For example the following segfaults as soon as as you press a key:

Quoted 7 lines

qemu-system-x86_64 -usb -device usb-bt-dongle -bt hci -bt device:keyboard

Since we are not aware of anybody using bluetooth with the current version of QEMU, let's mark the subsystem as deprecated, with a special request for the users to write to the qemu-devel mailing list in case they still use it (so we could revert the deprecation status in that case).

I think that settles the bluetooth debate for now. We deprecate it, wait for feedback, possibly un-deprecate (parts of) the code. Next year we'll actually remove the unused code.

Patch queued for 3.1.

thanks,
Gerd

#3 lore raw
Quoted 14 lines · Gerd Hoffmann
Quoted 5 lines · Thomas Huth

It has been unmaintained since years, and there were only trivial or tree-wide changes to the related files since many years, so the code is likely very bitrotten and broken. For example the following segfaults as soon as as you press a key:

Quoted 7 lines

qemu-system-x86_64 -usb -device usb-bt-dongle -bt hci -bt device:keyboard

Since we are not aware of anybody using bluetooth with the current version of QEMU, let's mark the subsystem as deprecated, with a special request for the users to write to the qemu-devel mailing list in case they still use it (so we could revert the deprecation status in that case).

Quoted 5 lines

I think that settles the bluetooth debate for now. We deprecate it, wait for feedback, possibly un-deprecate (parts of) the code. Next year we'll actually remove the unused code.

Patch queued for 3.1.

FWIW, libvirt has never exposed any of the bluetooth functionality, so no objections to removing it from our side.

Regards,
Daniel

Signature · 3 lines
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|