diff --git a/pipewire-alsa/alsa-plugins/ctl_pipewire.c b/pipewire-alsa/alsa-plugins/ctl_pipewire.c index a3465ebb1..d0d0dcee2 100644 --- a/pipewire-alsa/alsa-plugins/ctl_pipewire.c +++ b/pipewire-alsa/alsa-plugins/ctl_pipewire.c @@ -1102,7 +1102,7 @@ static void on_core_error(void *data, uint32_t id, int seq, int res, const char { snd_ctl_pipewire_t *ctl = data; - pw_log_error(NAME" %p: error id:%u seq:%d res:%d (%s): %s", ctl, + pw_log_warn(NAME" %p: error id:%u seq:%d res:%d (%s): %s", ctl, id, seq, res, spa_strerror(res), message); if (id == PW_ID_CORE) { diff --git a/pipewire-alsa/alsa-plugins/pcm_pipewire.c b/pipewire-alsa/alsa-plugins/pcm_pipewire.c index 009241f37..eeb4c78f4 100644 --- a/pipewire-alsa/alsa-plugins/pcm_pipewire.c +++ b/pipewire-alsa/alsa-plugins/pcm_pipewire.c @@ -838,7 +838,7 @@ static void on_core_error(void *data, uint32_t id, int seq, int res, const char { snd_pcm_pipewire_t *pw = data; - pw_log_error(NAME" %p: error id:%u seq:%d res:%d (%s): %s", pw, + pw_log_warn(NAME" %p: error id:%u seq:%d res:%d (%s): %s", pw, id, seq, res, spa_strerror(res), message); if (id == PW_ID_CORE) { diff --git a/spa/plugins/bluez5/bluez5-dbus.c b/spa/plugins/bluez5/bluez5-dbus.c index b63cae285..d207b6815 100644 --- a/spa/plugins/bluez5/bluez5-dbus.c +++ b/spa/plugins/bluez5/bluez5-dbus.c @@ -1471,7 +1471,8 @@ static void interface_added(struct spa_bt_monitor *monitor, if (d == NULL) { d = device_create(monitor, object_path); if (d == NULL) { - spa_log_warn(monitor->log, "can't create device: %m"); + spa_log_warn(monitor->log, "can't create Bluetooth device %s: %m", + object_path); return; } } diff --git a/src/examples/media-session/media-session.c b/src/examples/media-session/media-session.c index 1be746a45..19c67e2fb 100644 --- a/src/examples/media-session/media-session.c +++ b/src/examples/media-session/media-session.c @@ -1988,7 +1988,8 @@ static void core_error(void *data, uint32_t id, int seq, int res, const char *me { struct impl *impl = data; - pw_log_error("error id:%u seq:%d res:%d (%s): %s", + pw_log(res == -ENOENT ? SPA_LOG_LEVEL_INFO : SPA_LOG_LEVEL_WARN, + "error id:%u seq:%d res:%d (%s): %s", id, seq, res, spa_strerror(res), message); if (id == PW_ID_CORE) { diff --git a/src/gst/gstpipewirecore.c b/src/gst/gstpipewirecore.c index d53686af3..b8670a67f 100644 --- a/src/gst/gstpipewirecore.c +++ b/src/gst/gstpipewirecore.c @@ -41,7 +41,7 @@ on_core_error(void *data, uint32_t id, int seq, int res, const char *message) { GstPipeWireCore *core = data; - pw_log_error("error id:%u seq:%d res:%d (%s): %s", + pw_log_warn("error id:%u seq:%d res:%d (%s): %s", id, seq, res, spa_strerror(res), message); if (id == PW_ID_CORE) { diff --git a/src/gst/gstpipewiredeviceprovider.c b/src/gst/gstpipewiredeviceprovider.c index f7c7724c6..95488ffa0 100644 --- a/src/gst/gstpipewiredeviceprovider.c +++ b/src/gst/gstpipewiredeviceprovider.c @@ -295,7 +295,7 @@ on_core_error(void *data, uint32_t id, int seq, int res, const char *message) struct core_data *rd = data; GstPipeWireDeviceProvider *self = rd->self; - pw_log_error("error id:%u seq:%d res:%d (%s): %s", + pw_log_warn("error id:%u seq:%d res:%d (%s): %s", id, seq, res, spa_strerror(res), message); if (id == PW_ID_CORE) { diff --git a/src/modules/module-protocol-pulse/pulse-server.c b/src/modules/module-protocol-pulse/pulse-server.c index 5429b5867..c7bf41694 100644 --- a/src/modules/module-protocol-pulse/pulse-server.c +++ b/src/modules/module-protocol-pulse/pulse-server.c @@ -415,7 +415,8 @@ static int reply_error(struct client *client, uint32_t command, uint32_t tag, in else name = "invalid"; - pw_log_warn(NAME" %p: [%s] ERROR command:%d (%s) tag:%u error:%u (%s)", + pw_log(res == -ENOENT ? SPA_LOG_LEVEL_INFO : SPA_LOG_LEVEL_WARN, + NAME" %p: [%s] ERROR command:%d (%s) tag:%u error:%u (%s)", client, client->name, command, name, tag, error, spa_strerror(res)); reply = message_alloc(client, -1, 0); diff --git a/src/modules/spa/module-device-factory.c b/src/modules/spa/module-device-factory.c index 4cbc81acd..3001359a6 100644 --- a/src/modules/spa/module-device-factory.c +++ b/src/modules/spa/module-device-factory.c @@ -164,10 +164,11 @@ error_properties: "usage: "FACTORY_USAGE); goto error_exit; error_device: - pw_log_debug("can't create device: %s", spa_strerror(res)); + pw_log_debug("can't create device %s: %s", factory_name, spa_strerror(res)); if (resource) pw_resource_errorf_id(resource, new_id, res, - "can't create device: %s", spa_strerror(res)); + "can't create device %s: %s", factory_name, + spa_strerror(res)); goto error_exit; error_bind: pw_resource_errorf_id(resource, new_id, res, "can't bind device"); diff --git a/src/modules/spa/spa-device.c b/src/modules/spa/spa-device.c index 684126755..a4aaccd43 100644 --- a/src/modules/spa/spa-device.c +++ b/src/modules/spa/spa-device.c @@ -30,6 +30,7 @@ #include #include +#include #include #include #include @@ -141,15 +142,16 @@ struct pw_impl_device *pw_spa_device_load(struct pw_context *context, error_load: res = -errno; - pw_log_debug("can't load device handle: %m"); + pw_log_debug("can't load device handle %s: %m", factory_name); goto error_exit; error_interface: - pw_log_debug("can't get device interface %d", res); + pw_log_debug("can't get device interface %s: %s", factory_name, + spa_strerror(res)); goto error_exit_unload; error_device: properties = NULL; res = -errno; - pw_log_debug("can't create device: %m"); + pw_log_debug("can't create device %s: %m", factory_name); goto error_exit_unload; error_exit_unload: