Commit 7bc93821 authored by Takashi Sakamoto's avatar Takashi Sakamoto Committed by Takashi Iwai

ALSA: firewire-lib: split allocation of isochronous resources from establishment of connection

In current implementation, establishment connection corresponds to
allocation of isochronous resources. Although this is an ideal
implementation of CMP described in IEC 61883-1, it's not enough
efficient to recover PCM substream multiplexed in packet streaming.
The packet streaming can always restart on the same allocated
isochronous resources even if the previous packet streaming
corrupted.

This commit splits allocation of isochronous resources from
establishment of connection so that CMP runs with allocated
isochronous resources.
Signed-off-by: default avatarTakashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent c6b84ffb
...@@ -404,13 +404,11 @@ static int make_both_connections(struct snd_bebob *bebob) ...@@ -404,13 +404,11 @@ static int make_both_connections(struct snd_bebob *bebob)
{ {
int err = 0; int err = 0;
err = cmp_connection_establish(&bebob->out_conn, err = cmp_connection_establish(&bebob->out_conn);
amdtp_stream_get_max_payload(&bebob->tx_stream));
if (err < 0) if (err < 0)
return err; return err;
err = cmp_connection_establish(&bebob->in_conn, err = cmp_connection_establish(&bebob->in_conn);
amdtp_stream_get_max_payload(&bebob->rx_stream));
if (err < 0) { if (err < 0) {
cmp_connection_break(&bebob->out_conn); cmp_connection_break(&bebob->out_conn);
return err; return err;
...@@ -533,14 +531,23 @@ static int keep_resources(struct snd_bebob *bebob, struct amdtp_stream *stream, ...@@ -533,14 +531,23 @@ static int keep_resources(struct snd_bebob *bebob, struct amdtp_stream *stream,
unsigned int rate, unsigned int index) unsigned int rate, unsigned int index)
{ {
struct snd_bebob_stream_formation *formation; struct snd_bebob_stream_formation *formation;
struct cmp_connection *conn;
int err;
if (stream == &bebob->tx_stream) if (stream == &bebob->tx_stream) {
formation = bebob->tx_stream_formations + index; formation = bebob->tx_stream_formations + index;
else conn = &bebob->out_conn;
} else {
formation = bebob->rx_stream_formations + index; formation = bebob->rx_stream_formations + index;
conn = &bebob->in_conn;
}
err = amdtp_am824_set_parameters(stream, rate, formation->pcm,
formation->midi, false);
if (err < 0)
return err;
return amdtp_am824_set_parameters(stream, rate, formation->pcm, return cmp_connection_reserve(conn, amdtp_stream_get_max_payload(stream));
formation->midi, false);
} }
int snd_bebob_stream_reserve_duplex(struct snd_bebob *bebob, unsigned int rate) int snd_bebob_stream_reserve_duplex(struct snd_bebob *bebob, unsigned int rate)
...@@ -591,8 +598,10 @@ int snd_bebob_stream_reserve_duplex(struct snd_bebob *bebob, unsigned int rate) ...@@ -591,8 +598,10 @@ int snd_bebob_stream_reserve_duplex(struct snd_bebob *bebob, unsigned int rate)
return err; return err;
err = keep_resources(bebob, &bebob->rx_stream, rate, index); err = keep_resources(bebob, &bebob->rx_stream, rate, index);
if (err < 0) if (err < 0) {
cmp_connection_release(&bebob->out_conn);
return err; return err;
}
} }
return 0; return 0;
...@@ -685,6 +694,9 @@ void snd_bebob_stream_stop_duplex(struct snd_bebob *bebob) ...@@ -685,6 +694,9 @@ void snd_bebob_stream_stop_duplex(struct snd_bebob *bebob)
amdtp_stream_stop(&bebob->tx_stream); amdtp_stream_stop(&bebob->tx_stream);
break_both_connections(bebob); break_both_connections(bebob);
cmp_connection_release(&bebob->out_conn);
cmp_connection_release(&bebob->in_conn);
} }
} }
......
...@@ -185,6 +185,37 @@ void cmp_connection_destroy(struct cmp_connection *c) ...@@ -185,6 +185,37 @@ void cmp_connection_destroy(struct cmp_connection *c)
} }
EXPORT_SYMBOL(cmp_connection_destroy); EXPORT_SYMBOL(cmp_connection_destroy);
int cmp_connection_reserve(struct cmp_connection *c,
unsigned int max_payload_bytes)
{
int err;
mutex_lock(&c->mutex);
if (WARN_ON(c->resources.allocated)) {
err = -EBUSY;
goto end;
}
c->speed = min(c->max_speed,
fw_parent_device(c->resources.unit)->max_speed);
err = fw_iso_resources_allocate(&c->resources, max_payload_bytes,
c->speed);
end:
mutex_unlock(&c->mutex);
return err;
}
EXPORT_SYMBOL(cmp_connection_reserve);
void cmp_connection_release(struct cmp_connection *c)
{
mutex_lock(&c->mutex);
fw_iso_resources_free(&c->resources);
mutex_unlock(&c->mutex);
}
EXPORT_SYMBOL(cmp_connection_release);
static __be32 ipcr_set_modify(struct cmp_connection *c, __be32 ipcr) static __be32 ipcr_set_modify(struct cmp_connection *c, __be32 ipcr)
{ {
...@@ -270,25 +301,18 @@ static int pcr_set_check(struct cmp_connection *c, __be32 pcr) ...@@ -270,25 +301,18 @@ static int pcr_set_check(struct cmp_connection *c, __be32 pcr)
* When this function succeeds, the caller is responsible for starting * When this function succeeds, the caller is responsible for starting
* transmitting packets. * transmitting packets.
*/ */
int cmp_connection_establish(struct cmp_connection *c, int cmp_connection_establish(struct cmp_connection *c)
unsigned int max_payload_bytes)
{ {
int err; int err;
if (WARN_ON(c->connected))
return -EISCONN;
c->speed = min(c->max_speed,
fw_parent_device(c->resources.unit)->max_speed);
mutex_lock(&c->mutex); mutex_lock(&c->mutex);
retry_after_bus_reset: if (WARN_ON(c->connected)) {
err = fw_iso_resources_allocate(&c->resources, mutex_unlock(&c->mutex);
max_payload_bytes, c->speed); return -EISCONN;
if (err < 0) }
goto err_mutex;
retry_after_bus_reset:
if (c->direction == CMP_OUTPUT) if (c->direction == CMP_OUTPUT)
err = pcr_modify(c, opcr_set_modify, pcr_set_check, err = pcr_modify(c, opcr_set_modify, pcr_set_check,
ABORT_ON_BUS_RESET); ABORT_ON_BUS_RESET);
...@@ -297,21 +321,13 @@ int cmp_connection_establish(struct cmp_connection *c, ...@@ -297,21 +321,13 @@ int cmp_connection_establish(struct cmp_connection *c,
ABORT_ON_BUS_RESET); ABORT_ON_BUS_RESET);
if (err == -EAGAIN) { if (err == -EAGAIN) {
fw_iso_resources_free(&c->resources); err = fw_iso_resources_update(&c->resources);
goto retry_after_bus_reset; if (err >= 0)
goto retry_after_bus_reset;
} }
if (err < 0) if (err >= 0)
goto err_resources; c->connected = true;
c->connected = true;
mutex_unlock(&c->mutex);
return 0;
err_resources:
fw_iso_resources_free(&c->resources);
err_mutex:
mutex_unlock(&c->mutex); mutex_unlock(&c->mutex);
return err; return err;
...@@ -351,14 +367,12 @@ int cmp_connection_update(struct cmp_connection *c) ...@@ -351,14 +367,12 @@ int cmp_connection_update(struct cmp_connection *c)
SUCCEED_ON_BUS_RESET); SUCCEED_ON_BUS_RESET);
if (err < 0) if (err < 0)
goto err_resources; goto err_unconnect;
mutex_unlock(&c->mutex); mutex_unlock(&c->mutex);
return 0; return 0;
err_resources:
fw_iso_resources_free(&c->resources);
err_unconnect: err_unconnect:
c->connected = false; c->connected = false;
mutex_unlock(&c->mutex); mutex_unlock(&c->mutex);
...@@ -395,8 +409,6 @@ void cmp_connection_break(struct cmp_connection *c) ...@@ -395,8 +409,6 @@ void cmp_connection_break(struct cmp_connection *c)
if (err < 0) if (err < 0)
cmp_error(c, "plug is still connected\n"); cmp_error(c, "plug is still connected\n");
fw_iso_resources_free(&c->resources);
c->connected = false; c->connected = false;
mutex_unlock(&c->mutex); mutex_unlock(&c->mutex);
......
...@@ -42,8 +42,11 @@ int cmp_connection_init(struct cmp_connection *connection, ...@@ -42,8 +42,11 @@ int cmp_connection_init(struct cmp_connection *connection,
int cmp_connection_check_used(struct cmp_connection *connection, bool *used); int cmp_connection_check_used(struct cmp_connection *connection, bool *used);
void cmp_connection_destroy(struct cmp_connection *connection); void cmp_connection_destroy(struct cmp_connection *connection);
int cmp_connection_establish(struct cmp_connection *connection, int cmp_connection_reserve(struct cmp_connection *connection,
unsigned int max_payload); unsigned int max_payload);
void cmp_connection_release(struct cmp_connection *connection);
int cmp_connection_establish(struct cmp_connection *connection);
int cmp_connection_update(struct cmp_connection *connection); int cmp_connection_update(struct cmp_connection *connection);
void cmp_connection_break(struct cmp_connection *connection); void cmp_connection_break(struct cmp_connection *connection);
......
...@@ -63,8 +63,7 @@ static int start_stream(struct snd_efw *efw, struct amdtp_stream *stream, ...@@ -63,8 +63,7 @@ static int start_stream(struct snd_efw *efw, struct amdtp_stream *stream,
conn = &efw->in_conn; conn = &efw->in_conn;
// Establish connection via CMP. // Establish connection via CMP.
err = cmp_connection_establish(conn, err = cmp_connection_establish(conn);
amdtp_stream_get_max_payload(stream));
if (err < 0) if (err < 0)
return err; return err;
...@@ -177,17 +176,25 @@ static int keep_resources(struct snd_efw *efw, struct amdtp_stream *stream, ...@@ -177,17 +176,25 @@ static int keep_resources(struct snd_efw *efw, struct amdtp_stream *stream,
{ {
unsigned int pcm_channels; unsigned int pcm_channels;
unsigned int midi_ports; unsigned int midi_ports;
struct cmp_connection *conn;
int err;
if (stream == &efw->tx_stream) { if (stream == &efw->tx_stream) {
pcm_channels = efw->pcm_capture_channels[mode]; pcm_channels = efw->pcm_capture_channels[mode];
midi_ports = efw->midi_out_ports; midi_ports = efw->midi_out_ports;
conn = &efw->out_conn;
} else { } else {
pcm_channels = efw->pcm_playback_channels[mode]; pcm_channels = efw->pcm_playback_channels[mode];
midi_ports = efw->midi_in_ports; midi_ports = efw->midi_in_ports;
conn = &efw->in_conn;
} }
return amdtp_am824_set_parameters(stream, rate, pcm_channels, err = amdtp_am824_set_parameters(stream, rate, pcm_channels,
midi_ports, false); midi_ports, false);
if (err < 0)
return err;
return cmp_connection_reserve(conn, amdtp_stream_get_max_payload(stream));
} }
int snd_efw_stream_reserve_duplex(struct snd_efw *efw, unsigned int rate) int snd_efw_stream_reserve_duplex(struct snd_efw *efw, unsigned int rate)
...@@ -228,8 +235,10 @@ int snd_efw_stream_reserve_duplex(struct snd_efw *efw, unsigned int rate) ...@@ -228,8 +235,10 @@ int snd_efw_stream_reserve_duplex(struct snd_efw *efw, unsigned int rate)
return err; return err;
err = keep_resources(efw, &efw->rx_stream, rate, mode); err = keep_resources(efw, &efw->rx_stream, rate, mode);
if (err < 0) if (err < 0) {
cmp_connection_release(&efw->in_conn);
return err; return err;
}
} }
return 0; return 0;
...@@ -285,6 +294,9 @@ void snd_efw_stream_stop_duplex(struct snd_efw *efw) ...@@ -285,6 +294,9 @@ void snd_efw_stream_stop_duplex(struct snd_efw *efw)
if (efw->substreams_counter == 0) { if (efw->substreams_counter == 0) {
stop_stream(efw, &efw->tx_stream); stop_stream(efw, &efw->tx_stream);
stop_stream(efw, &efw->rx_stream); stop_stream(efw, &efw->rx_stream);
cmp_connection_release(&efw->out_conn);
cmp_connection_release(&efw->in_conn);
} }
} }
......
...@@ -111,8 +111,7 @@ static int start_stream(struct snd_oxfw *oxfw, struct amdtp_stream *stream) ...@@ -111,8 +111,7 @@ static int start_stream(struct snd_oxfw *oxfw, struct amdtp_stream *stream)
else else
conn = &oxfw->out_conn; conn = &oxfw->out_conn;
err = cmp_connection_establish(conn, err = cmp_connection_establish(conn);
amdtp_stream_get_max_payload(stream));
if (err < 0) if (err < 0)
return err; return err;
...@@ -203,15 +202,18 @@ static int keep_resources(struct snd_oxfw *oxfw, struct amdtp_stream *stream) ...@@ -203,15 +202,18 @@ static int keep_resources(struct snd_oxfw *oxfw, struct amdtp_stream *stream)
enum avc_general_plug_dir dir; enum avc_general_plug_dir dir;
u8 **formats; u8 **formats;
struct snd_oxfw_stream_formation formation; struct snd_oxfw_stream_formation formation;
struct cmp_connection *conn;
int i; int i;
int err; int err;
if (stream == &oxfw->rx_stream) { if (stream == &oxfw->rx_stream) {
dir = AVC_GENERAL_PLUG_DIR_IN; dir = AVC_GENERAL_PLUG_DIR_IN;
formats = oxfw->rx_stream_formats; formats = oxfw->rx_stream_formats;
conn = &oxfw->in_conn;
} else { } else {
dir = AVC_GENERAL_PLUG_DIR_OUT; dir = AVC_GENERAL_PLUG_DIR_OUT;
formats = oxfw->tx_stream_formats; formats = oxfw->tx_stream_formats;
conn = &oxfw->out_conn;
} }
err = snd_oxfw_stream_get_current_formation(oxfw, dir, &formation); err = snd_oxfw_stream_get_current_formation(oxfw, dir, &formation);
...@@ -239,8 +241,12 @@ static int keep_resources(struct snd_oxfw *oxfw, struct amdtp_stream *stream) ...@@ -239,8 +241,12 @@ static int keep_resources(struct snd_oxfw *oxfw, struct amdtp_stream *stream)
if (formation.pcm == 0) if (formation.pcm == 0)
return -EINVAL; return -EINVAL;
return amdtp_am824_set_parameters(stream, formation.rate, formation.pcm, err = amdtp_am824_set_parameters(stream, formation.rate, formation.pcm,
formation.midi * 8, false); formation.midi * 8, false);
if (err < 0)
return err;
return cmp_connection_reserve(conn, amdtp_stream_get_max_payload(stream));
} }
int snd_oxfw_stream_reserve_duplex(struct snd_oxfw *oxfw, int snd_oxfw_stream_reserve_duplex(struct snd_oxfw *oxfw,
...@@ -299,8 +305,10 @@ int snd_oxfw_stream_reserve_duplex(struct snd_oxfw *oxfw, ...@@ -299,8 +305,10 @@ int snd_oxfw_stream_reserve_duplex(struct snd_oxfw *oxfw,
if (oxfw->has_output) { if (oxfw->has_output) {
err = keep_resources(oxfw, &oxfw->tx_stream); err = keep_resources(oxfw, &oxfw->tx_stream);
if (err < 0) if (err < 0) {
cmp_connection_release(&oxfw->in_conn);
return err; return err;
}
} }
} }
...@@ -361,10 +369,12 @@ void snd_oxfw_stream_stop_duplex(struct snd_oxfw *oxfw) ...@@ -361,10 +369,12 @@ void snd_oxfw_stream_stop_duplex(struct snd_oxfw *oxfw)
if (oxfw->substreams_count == 0) { if (oxfw->substreams_count == 0) {
amdtp_stream_stop(&oxfw->rx_stream); amdtp_stream_stop(&oxfw->rx_stream);
cmp_connection_break(&oxfw->in_conn); cmp_connection_break(&oxfw->in_conn);
cmp_connection_release(&oxfw->in_conn);
if (oxfw->has_output) { if (oxfw->has_output) {
amdtp_stream_stop(&oxfw->tx_stream); amdtp_stream_stop(&oxfw->tx_stream);
cmp_connection_break(&oxfw->out_conn); cmp_connection_break(&oxfw->out_conn);
cmp_connection_release(&oxfw->out_conn);
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment