* [PATCH 0/2] amidi: fix handling of ignored messages
@ 2016-08-13 10:27 Clemens Ladisch
2016-08-13 10:28 ` [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
` (2 more replies)
0 siblings, 3 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 10:27 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
Clemens Ladisch (2):
amidi: ignore not only Active Sensing but also Clock bytes
amidi: fix timeout handling
amidi/amidi.1 | 19 ++++++++---
amidi/amidi.c | 82 +++++++++++++++++++++++++++++++++++++++-----------
2 files changed, 78 insertions(+), 23 deletions(-)
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes
2016-08-13 10:27 [PATCH 0/2] amidi: fix handling of ignored messages Clemens Ladisch
@ 2016-08-13 10:28 ` Clemens Ladisch
2016-08-13 11:14 ` Takashi Sakamoto
2016-08-13 10:29 ` [PATCH 2/2] amidi: fix timeout handling Clemens Ladisch
2016-08-15 14:10 ` [PATCH 0/2] amidi: fix handling of ignored messages Felipe Ferreri Tonello
2 siblings, 1 reply; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 10:28 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
Active Sensing messages are sent by many devices in the background and
would only interfere with the actual messages that amidi is supposed to
capture. Therefore, amidi ignores them by default. However, there are
also devices that send Clock messages with the same problem, so it is
a better idea to filter them out, too.
Reported-by: Martin Tarenskeen <m.tarenskeen@gmail.com>
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
---
amidi/amidi.1 | 19 ++++++++++++++-----
amidi/amidi.c | 13 +++++++++++--
2 files changed, 25 insertions(+), 7 deletions(-)
diff --git a/amidi/amidi.1 b/amidi/amidi.1
index 1b4cfb1..86beb27 100644
--- a/amidi/amidi.1
+++ b/amidi/amidi.1
@@ -1,4 +1,4 @@
-.TH AMIDI 1 "26 Jun 2006"
+.TH AMIDI 1 "16 Apr 2016"
.SH NAME
amidi \- read from and write to ALSA RawMIDI ports
@@ -80,9 +80,11 @@ to record a Standard MIDI (.mid) file, use
.B arecordmidi(1).
.B amidi
-will filter out any Active Sensing bytes (FEh), unless the
+will filter out any Active Sensing and Clock bytes (FEh, F8h), unless the
.I \-a
-option has been given.
+or
+.I \-c
+options have been given.
.TP
.I \-S, \-\-send\-hex="..."
@@ -91,9 +93,11 @@ Sends the bytes specified as hexadecimal numbers to the MIDI port.
.TP
.I \-d, \-\-dump
Prints data received from the MIDI port as hexadecimal bytes.
-Active Sensing bytes (FEh) will not be shown, unless the
+Active Sensing and Clock bytes (FEh, F8h) will not be shown, unless the
.I \-a
-option has been given.
+or
+.I \-c
+options have been given.
This option is useful for debugging.
@@ -111,6 +115,11 @@ to stop receiving data.
Does not ignore Active Sensing bytes (FEh) when saving or printing
received MIDI commands.
+.TP
+.I \-c, \-\-clock
+Does not ignore Clock bytes (F8h) when saving or printing received
+MIDI commands.
+
.SH EXAMPLES
.TP
diff --git a/amidi/amidi.c b/amidi/amidi.c
index cedf18c..58ac814 100644
--- a/amidi/amidi.c
+++ b/amidi/amidi.c
@@ -77,7 +77,8 @@ static void usage(void)
"-d, --dump print received data as hexadecimal bytes\n"
"-t, --timeout=seconds exits when no data has been received\n"
" for the specified duration\n"
- "-a, --active-sensing don't ignore active sensing bytes\n");
+ "-a, --active-sensing include active sensing bytes\n"
+ "-c, --clock include clock bytes\n");
}
static void version(void)
@@ -419,10 +420,12 @@ int main(int argc, char *argv[])
{"dump", 0, NULL, 'd'},
{"timeout", 1, NULL, 't'},
{"active-sensing", 0, NULL, 'a'},
+ {"clock", 0, NULL, 'c'},
{ }
};
int c, err, ok = 0;
int ignore_active_sensing = 1;
+ int ignore_clock = 1;
int do_send_hex = 0;
while ((c = getopt_long(argc, argv, short_options,
@@ -463,6 +466,9 @@ int main(int argc, char *argv[])
case 'a':
ignore_active_sensing = 0;
break;
+ case 'c':
+ ignore_clock = 0;
+ break;
default:
error("Try `amidi --help' for more information.");
return 1;
@@ -589,7 +595,10 @@ int main(int argc, char *argv[])
}
length = 0;
for (i = 0; i < err; ++i)
- if (!ignore_active_sensing || buf[i] != 0xfe)
+ if ((buf[i] != MIDI_CMD_COMMON_CLOCK &&
+ buf[i] != MIDI_CMD_COMMON_SENSING) ||
+ (buf[i] == MIDI_CMD_COMMON_CLOCK && !ignore_clock) ||
+ (buf[i] == MIDI_CMD_COMMON_SENSING && !ignore_active_sensing))
buf[length++] = buf[i];
if (length == 0)
continue;
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] amidi: fix timeout handling
2016-08-13 10:27 [PATCH 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 10:28 ` [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
@ 2016-08-13 10:29 ` Clemens Ladisch
2016-08-15 14:10 ` [PATCH 0/2] amidi: fix handling of ignored messages Felipe Ferreri Tonello
2 siblings, 0 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 10:29 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
The timeout is not supposed to expire when ignored messages are
received. This cannot be handled with the poll() timeout, so add
a separate timer.
Reported-by: Martin Tarenskeen <m.tarenskeen@gmail.com>
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
---
amidi/amidi.c | 69 ++++++++++++++++++++++++++++++++++++++++++++-------------
1 file changed, 53 insertions(+), 16 deletions(-)
diff --git a/amidi/amidi.c b/amidi/amidi.c
index 58ac814..290df48 100644
--- a/amidi/amidi.c
+++ b/amidi/amidi.c
@@ -25,9 +25,11 @@
#include <stdarg.h>
#include <string.h>
#include <ctype.h>
+#include <math.h>
#include <getopt.h>
#include <errno.h>
#include <signal.h>
+#include <sys/timerfd.h>
#include <sys/types.h>
#include <sys/poll.h>
#include <sys/stat.h>
@@ -37,6 +39,8 @@
#include "aconfig.h"
#include "version.h"
+#define NSEC_PER_SEC 1000000000L
+
static int do_device_list, do_rawmidi_list;
static char *port_name = "default";
static char *send_file_name;
@@ -46,7 +50,7 @@ static char *send_data;
static int send_data_length;
static int receive_file;
static int dump;
-static int timeout;
+static float timeout;
static int stop;
static snd_rawmidi_t *input, **inputp;
static snd_rawmidi_t *output, **outputp;
@@ -427,6 +431,7 @@ int main(int argc, char *argv[])
int ignore_active_sensing = 1;
int ignore_clock = 1;
int do_send_hex = 0;
+ struct itimerspec itimerspec = { .it_interval = { 0, 0 } };
while ((c = getopt_long(argc, argv, short_options,
long_options, NULL)) != -1) {
@@ -461,7 +466,7 @@ int main(int argc, char *argv[])
dump = 1;
break;
case 't':
- timeout = atoi(optarg);
+ timeout = atof(optarg);
break;
case 'a':
ignore_active_sensing = 0;
@@ -552,40 +557,64 @@ int main(int argc, char *argv[])
if (inputp) {
int read = 0;
- int npfds, time = 0;
+ int npfds;
struct pollfd *pfds;
- timeout *= 1000;
- npfds = snd_rawmidi_poll_descriptors_count(input);
+ npfds = 1 + snd_rawmidi_poll_descriptors_count(input);
pfds = alloca(npfds * sizeof(struct pollfd));
- snd_rawmidi_poll_descriptors(input, pfds, npfds);
+
+ if (timeout > 0) {
+ pfds[0].fd = timerfd_create(CLOCK_MONOTONIC, 0);
+ if (pfds[0].fd == -1) {
+ error("cannot create timer: %s", strerror(errno));
+ goto _exit;
+ }
+ pfds[0].events = POLLIN;
+ } else {
+ pfds[0].fd = -1;
+ }
+
+ snd_rawmidi_poll_descriptors(input, &pfds[1], npfds - 1);
+
signal(SIGINT, sig_handler);
+
+ if (timeout > 0) {
+ float timeout_int;
+
+ itimerspec.it_value.tv_nsec = modff(timeout, &timeout_int) * NSEC_PER_SEC;
+ itimerspec.it_value.tv_sec = timeout_int;
+ err = timerfd_settime(pfds[0].fd, 0, &itimerspec, NULL);
+ if (err < 0) {
+ error("cannot set timer: %s", strerror(errno));
+ goto _exit;
+ }
+ }
for (;;) {
unsigned char buf[256];
int i, length;
unsigned short revents;
- err = poll(pfds, npfds, 200);
+ err = poll(pfds, npfds, -1);
if (stop || (err < 0 && errno == EINTR))
break;
if (err < 0) {
error("poll failed: %s", strerror(errno));
break;
}
- if (err == 0) {
- time += 200;
- if (timeout && time >= timeout)
- break;
- continue;
- }
- if ((err = snd_rawmidi_poll_descriptors_revents(input, pfds, npfds, &revents)) < 0) {
+
+ err = snd_rawmidi_poll_descriptors_revents(input, &pfds[1], npfds - 1, &revents);
+ if (err < 0) {
error("cannot get poll events: %s", snd_strerror(errno));
break;
}
if (revents & (POLLERR | POLLHUP))
break;
- if (!(revents & POLLIN))
+ if (!(revents & POLLIN)) {
+ if (pfds[0].revents & POLLIN)
+ break;
continue;
+ }
+
err = snd_rawmidi_read(input, buf, sizeof(buf));
if (err == -EAGAIN)
continue;
@@ -603,7 +632,7 @@ int main(int argc, char *argv[])
if (length == 0)
continue;
read += length;
- time = 0;
+
if (receive_file != -1)
write(receive_file, buf, length);
if (dump) {
@@ -611,6 +640,14 @@ int main(int argc, char *argv[])
print_byte(buf[i]);
fflush(stdout);
}
+
+ if (timeout > 0) {
+ err = timerfd_settime(pfds[0].fd, 0, &itimerspec, NULL);
+ if (err < 0) {
+ error("cannot set timer: %s", strerror(errno));
+ break;
+ }
+ }
}
if (isatty(fileno(stdout)))
printf("\n%d bytes read\n", read);
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes
2016-08-13 10:28 ` [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
@ 2016-08-13 11:14 ` Takashi Sakamoto
2016-08-13 11:27 ` Takashi Sakamoto
0 siblings, 1 reply; 12+ messages in thread
From: Takashi Sakamoto @ 2016-08-13 11:14 UTC (permalink / raw)
To: Clemens Ladisch, Takashi Iwai; +Cc: alsa-devel
Hi Clemens,
On Aug 13 2016 19:28, Clemens Ladisch wrote:
> Active Sensing messages are sent by many devices in the background and
> would only interfere with the actual messages that amidi is supposed to
> capture. Therefore, amidi ignores them by default. However, there are
> also devices that send Clock messages with the same problem, so it is
> a better idea to filter them out, too.
>
> Reported-by: Martin Tarenskeen <m.tarenskeen@gmail.com>
> Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
> ---
> amidi/amidi.1 | 19 ++++++++++++++-----
> amidi/amidi.c | 13 +++++++++++--
> 2 files changed, 25 insertions(+), 7 deletions(-)
>
> ...
>
> diff --git a/amidi/amidi.c b/amidi/amidi.c
> index cedf18c..58ac814 100644
> --- a/amidi/amidi.c
> +++ b/amidi/amidi.c
> @@ -77,7 +77,8 @@ static void usage(void)
> "-d, --dump print received data as hexadecimal bytes\n"
> "-t, --timeout=seconds exits when no data has been received\n"
> " for the specified duration\n"
> - "-a, --active-sensing don't ignore active sensing bytes\n");
> + "-a, --active-sensing include active sensing bytes\n"
> + "-c, --clock include clock bytes\n");
> }
>
> static void version(void)
> @@ -419,10 +420,12 @@ int main(int argc, char *argv[])
> {"dump", 0, NULL, 'd'},
> {"timeout", 1, NULL, 't'},
> {"active-sensing", 0, NULL, 'a'},
> + {"clock", 0, NULL, 'c'},
> { }
Need to modify a table for short options.
diff --git a/amidi/amidi.c b/amidi/amidi.c
index 290df48..ea61cba 100644
--- a/amidi/amidi.c
+++ b/amidi/amidi.c
@@ -411,7 +411,7 @@ static void add_send_hex_data(const char *str)
int main(int argc, char *argv[])
{
- static const char short_options[] = "hVlLp:s:r:S::dt:a";
+ static const char short_options[] = "hVlLp:s:r:S::dt:a:c";
static const struct option long_options[] = {
{"help", 0, NULL, 'h'},
{"version", 0, NULL, 'V'},
Regards
Takashi Sakamoto
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes
2016-08-13 11:14 ` Takashi Sakamoto
@ 2016-08-13 11:27 ` Takashi Sakamoto
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
0 siblings, 1 reply; 12+ messages in thread
From: Takashi Sakamoto @ 2016-08-13 11:27 UTC (permalink / raw)
To: Clemens Ladisch, Takashi Iwai; +Cc: alsa-devel
On Aug 13 2016 20:14, Takashi Sakamoto wrote:
> Need to modify a table for short options.
>
> diff --git a/amidi/amidi.c b/amidi/amidi.c
> index 290df48..ea61cba 100644
> --- a/amidi/amidi.c
> +++ b/amidi/amidi.c
> @@ -411,7 +411,7 @@ static void add_send_hex_data(const char *str)
>
> int main(int argc, char *argv[])
> {
> - static const char short_options[] = "hVlLp:s:r:S::dt:a";
> + static const char short_options[] = "hVlLp:s:r:S::dt:a:c";
Oops. This should be "hVlLp:s:r:S::dt:ac"...
> static const struct option long_options[] = {
> {"help", 0, NULL, 'h'},
> {"version", 0, NULL, 'V'},
Regards
Takashi Sakamoto
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 0/2] amidi: fix handling of ignored messages
2016-08-13 11:27 ` Takashi Sakamoto
@ 2016-08-13 14:40 ` Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
` (3 more replies)
0 siblings, 4 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 14:40 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
Clemens Ladisch (2):
amidi: ignore not only Active Sensing but also Clock bytes
amidi: fix timeout handling
v2: fix short_options
amidi/amidi.1 | 19 ++++++++---
amidi/amidi.c | 84 ++++++++++++++++++++++++++++++++++++++------------
2 files changed, 79 insertions(+), 24 deletions(-)
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
@ 2016-08-13 14:41 ` Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 2/2] amidi: fix timeout handling Clemens Ladisch
` (2 subsequent siblings)
3 siblings, 0 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 14:41 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
Active Sensing messages are sent by many devices in the background and
would only interfere with the actual messages that amidi is supposed to
capture. Therefore, amidi ignores them by default. However, there are
also devices that send Clock messages with the same problem, so it is
a better idea to filter them out, too.
Reported-by: Martin Tarenskeen <m.tarenskeen@gmail.com>
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
---
amidi/amidi.1 | 19 ++++++++++++++-----
amidi/amidi.c | 15 ++++++++++++---
2 files changed, 26 insertions(+), 8 deletions(-)
diff --git a/amidi/amidi.1 b/amidi/amidi.1
index 1b4cfb1..86beb27 100644
--- a/amidi/amidi.1
+++ b/amidi/amidi.1
@@ -1,4 +1,4 @@
-.TH AMIDI 1 "26 Jun 2006"
+.TH AMIDI 1 "16 Apr 2016"
.SH NAME
amidi \- read from and write to ALSA RawMIDI ports
@@ -80,9 +80,11 @@ to record a Standard MIDI (.mid) file, use
.B arecordmidi(1).
.B amidi
-will filter out any Active Sensing bytes (FEh), unless the
+will filter out any Active Sensing and Clock bytes (FEh, F8h), unless the
.I \-a
-option has been given.
+or
+.I \-c
+options have been given.
.TP
.I \-S, \-\-send\-hex="..."
@@ -91,9 +93,11 @@ Sends the bytes specified as hexadecimal numbers to the MIDI port.
.TP
.I \-d, \-\-dump
Prints data received from the MIDI port as hexadecimal bytes.
-Active Sensing bytes (FEh) will not be shown, unless the
+Active Sensing and Clock bytes (FEh, F8h) will not be shown, unless the
.I \-a
-option has been given.
+or
+.I \-c
+options have been given.
This option is useful for debugging.
@@ -111,6 +115,11 @@ to stop receiving data.
Does not ignore Active Sensing bytes (FEh) when saving or printing
received MIDI commands.
+.TP
+.I \-c, \-\-clock
+Does not ignore Clock bytes (F8h) when saving or printing received
+MIDI commands.
+
.SH EXAMPLES
.TP
diff --git a/amidi/amidi.c b/amidi/amidi.c
index cedf18c..5871512 100644
--- a/amidi/amidi.c
+++ b/amidi/amidi.c
@@ -77,7 +77,8 @@ static void usage(void)
"-d, --dump print received data as hexadecimal bytes\n"
"-t, --timeout=seconds exits when no data has been received\n"
" for the specified duration\n"
- "-a, --active-sensing don't ignore active sensing bytes\n");
+ "-a, --active-sensing include active sensing bytes\n"
+ "-c, --clock include clock bytes\n");
}
static void version(void)
@@ -406,7 +407,7 @@ static void add_send_hex_data(const char *str)
int main(int argc, char *argv[])
{
- static const char short_options[] = "hVlLp:s:r:S::dt:a";
+ static const char short_options[] = "hVlLp:s:r:S::dt:ac";
static const struct option long_options[] = {
{"help", 0, NULL, 'h'},
{"version", 0, NULL, 'V'},
@@ -419,10 +420,12 @@ int main(int argc, char *argv[])
{"dump", 0, NULL, 'd'},
{"timeout", 1, NULL, 't'},
{"active-sensing", 0, NULL, 'a'},
+ {"clock", 0, NULL, 'c'},
{ }
};
int c, err, ok = 0;
int ignore_active_sensing = 1;
+ int ignore_clock = 1;
int do_send_hex = 0;
while ((c = getopt_long(argc, argv, short_options,
@@ -463,6 +466,9 @@ int main(int argc, char *argv[])
case 'a':
ignore_active_sensing = 0;
break;
+ case 'c':
+ ignore_clock = 0;
+ break;
default:
error("Try `amidi --help' for more information.");
return 1;
@@ -589,7 +595,10 @@ int main(int argc, char *argv[])
}
length = 0;
for (i = 0; i < err; ++i)
- if (!ignore_active_sensing || buf[i] != 0xfe)
+ if ((buf[i] != MIDI_CMD_COMMON_CLOCK &&
+ buf[i] != MIDI_CMD_COMMON_SENSING) ||
+ (buf[i] == MIDI_CMD_COMMON_CLOCK && !ignore_clock) ||
+ (buf[i] == MIDI_CMD_COMMON_SENSING && !ignore_active_sensing))
buf[length++] = buf[i];
if (length == 0)
continue;
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v2 2/2] amidi: fix timeout handling
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
@ 2016-08-13 14:41 ` Clemens Ladisch
2016-08-14 10:19 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Takashi Sakamoto
2016-08-22 9:20 ` Takashi Iwai
3 siblings, 0 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-13 14:41 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel
The timeout is not supposed to expire when ignored messages are
received. This cannot be handled with the poll() timeout, so add
a separate timer.
Reported-by: Martin Tarenskeen <m.tarenskeen@gmail.com>
Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
---
amidi/amidi.c | 69 ++++++++++++++++++++++++++++++++++++++++++++-------------
1 file changed, 53 insertions(+), 16 deletions(-)
diff --git a/amidi/amidi.c b/amidi/amidi.c
index 5871512..c20512c 100644
--- a/amidi/amidi.c
+++ b/amidi/amidi.c
@@ -25,9 +25,11 @@
#include <stdarg.h>
#include <string.h>
#include <ctype.h>
+#include <math.h>
#include <getopt.h>
#include <errno.h>
#include <signal.h>
+#include <sys/timerfd.h>
#include <sys/types.h>
#include <sys/poll.h>
#include <sys/stat.h>
@@ -37,6 +39,8 @@
#include "aconfig.h"
#include "version.h"
+#define NSEC_PER_SEC 1000000000L
+
static int do_device_list, do_rawmidi_list;
static char *port_name = "default";
static char *send_file_name;
@@ -46,7 +50,7 @@ static char *send_data;
static int send_data_length;
static int receive_file;
static int dump;
-static int timeout;
+static float timeout;
static int stop;
static snd_rawmidi_t *input, **inputp;
static snd_rawmidi_t *output, **outputp;
@@ -427,6 +431,7 @@ int main(int argc, char *argv[])
int ignore_active_sensing = 1;
int ignore_clock = 1;
int do_send_hex = 0;
+ struct itimerspec itimerspec = { .it_interval = { 0, 0 } };
while ((c = getopt_long(argc, argv, short_options,
long_options, NULL)) != -1) {
@@ -461,7 +466,7 @@ int main(int argc, char *argv[])
dump = 1;
break;
case 't':
- timeout = atoi(optarg);
+ timeout = atof(optarg);
break;
case 'a':
ignore_active_sensing = 0;
@@ -552,40 +557,64 @@ int main(int argc, char *argv[])
if (inputp) {
int read = 0;
- int npfds, time = 0;
+ int npfds;
struct pollfd *pfds;
- timeout *= 1000;
- npfds = snd_rawmidi_poll_descriptors_count(input);
+ npfds = 1 + snd_rawmidi_poll_descriptors_count(input);
pfds = alloca(npfds * sizeof(struct pollfd));
- snd_rawmidi_poll_descriptors(input, pfds, npfds);
+
+ if (timeout > 0) {
+ pfds[0].fd = timerfd_create(CLOCK_MONOTONIC, 0);
+ if (pfds[0].fd == -1) {
+ error("cannot create timer: %s", strerror(errno));
+ goto _exit;
+ }
+ pfds[0].events = POLLIN;
+ } else {
+ pfds[0].fd = -1;
+ }
+
+ snd_rawmidi_poll_descriptors(input, &pfds[1], npfds - 1);
+
signal(SIGINT, sig_handler);
+
+ if (timeout > 0) {
+ float timeout_int;
+
+ itimerspec.it_value.tv_nsec = modff(timeout, &timeout_int) * NSEC_PER_SEC;
+ itimerspec.it_value.tv_sec = timeout_int;
+ err = timerfd_settime(pfds[0].fd, 0, &itimerspec, NULL);
+ if (err < 0) {
+ error("cannot set timer: %s", strerror(errno));
+ goto _exit;
+ }
+ }
for (;;) {
unsigned char buf[256];
int i, length;
unsigned short revents;
- err = poll(pfds, npfds, 200);
+ err = poll(pfds, npfds, -1);
if (stop || (err < 0 && errno == EINTR))
break;
if (err < 0) {
error("poll failed: %s", strerror(errno));
break;
}
- if (err == 0) {
- time += 200;
- if (timeout && time >= timeout)
- break;
- continue;
- }
- if ((err = snd_rawmidi_poll_descriptors_revents(input, pfds, npfds, &revents)) < 0) {
+
+ err = snd_rawmidi_poll_descriptors_revents(input, &pfds[1], npfds - 1, &revents);
+ if (err < 0) {
error("cannot get poll events: %s", snd_strerror(errno));
break;
}
if (revents & (POLLERR | POLLHUP))
break;
- if (!(revents & POLLIN))
+ if (!(revents & POLLIN)) {
+ if (pfds[0].revents & POLLIN)
+ break;
continue;
+ }
+
err = snd_rawmidi_read(input, buf, sizeof(buf));
if (err == -EAGAIN)
continue;
@@ -603,7 +632,7 @@ int main(int argc, char *argv[])
if (length == 0)
continue;
read += length;
- time = 0;
+
if (receive_file != -1)
write(receive_file, buf, length);
if (dump) {
@@ -611,6 +640,14 @@ int main(int argc, char *argv[])
print_byte(buf[i]);
fflush(stdout);
}
+
+ if (timeout > 0) {
+ err = timerfd_settime(pfds[0].fd, 0, &itimerspec, NULL);
+ if (err < 0) {
+ error("cannot set timer: %s", strerror(errno));
+ break;
+ }
+ }
}
if (isatty(fileno(stdout)))
printf("\n%d bytes read\n", read);
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 0/2] amidi: fix handling of ignored messages
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 2/2] amidi: fix timeout handling Clemens Ladisch
@ 2016-08-14 10:19 ` Takashi Sakamoto
2016-08-22 9:20 ` Takashi Iwai
3 siblings, 0 replies; 12+ messages in thread
From: Takashi Sakamoto @ 2016-08-14 10:19 UTC (permalink / raw)
To: Clemens Ladisch, Takashi Iwai; +Cc: alsa-devel
On Aug 13 2016 23:40, Clemens Ladisch wrote:
> Clemens Ladisch (2):
> amidi: ignore not only Active Sensing but also Clock bytes
> amidi: fix timeout handling
>
> v2: fix short_options
>
> amidi/amidi.1 | 19 ++++++++---
> amidi/amidi.c | 84 ++++++++++++++++++++++++++++++++++++++------------
> 2 files changed, 79 insertions(+), 24 deletions(-)
Reviewd-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Tested-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
I tested these patches with loopbacked MIDI interface and this program.
Then, This command finishes as expected.
$ ./amidi -p hw:1,0,0 -d -t 5
-------- 8< --------
#include <stdio.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <stdint.h>
#include <unistd.h>
int main(void)
{
int fd;
uint8_t buf = 0xfe;
fd = open("/dev/snd/midiC1D0", O_WRONLY);
if (fd < 0) {
return EXIT_FAILURE;
}
while (write(fd, &buf, sizeof(buf)) > 0)
;
close(fd);
return EXIT_FAILURE;
}
-------- 8< --------
Regards
Takashi Sakamoto
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] amidi: fix handling of ignored messages
2016-08-13 10:27 [PATCH 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 10:28 ` [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
2016-08-13 10:29 ` [PATCH 2/2] amidi: fix timeout handling Clemens Ladisch
@ 2016-08-15 14:10 ` Felipe Ferreri Tonello
2016-08-15 14:31 ` Clemens Ladisch
2 siblings, 1 reply; 12+ messages in thread
From: Felipe Ferreri Tonello @ 2016-08-15 14:10 UTC (permalink / raw)
To: Clemens Ladisch, Takashi Iwai; +Cc: alsa-devel
[-- Attachment #1: Type: text/plain, Size: 397 bytes --]
Hi Clemens,
On 13/08/16 11:27, Clemens Ladisch wrote:
> Clemens Ladisch (2):
> amidi: ignore not only Active Sensing but also Clock bytes
> amidi: fix timeout handling
>
> amidi/amidi.1 | 19 ++++++++---
> amidi/amidi.c | 82 +++++++++++++++++++++++++++++++++++++++-----------
> 2 files changed, 78 insertions(+), 23 deletions(-)
>
I can't see the patches emails.
--
Felipe
[-- Attachment #2: 0x92698E6A.asc --]
[-- Type: application/pgp-keys, Size: 7195 bytes --]
[-- Attachment #3: Type: text/plain, Size: 0 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/2] amidi: fix handling of ignored messages
2016-08-15 14:10 ` [PATCH 0/2] amidi: fix handling of ignored messages Felipe Ferreri Tonello
@ 2016-08-15 14:31 ` Clemens Ladisch
0 siblings, 0 replies; 12+ messages in thread
From: Clemens Ladisch @ 2016-08-15 14:31 UTC (permalink / raw)
To: Felipe Ferreri Tonello; +Cc: alsa-devel
Felipe Ferreri Tonello wrote:
> On 13/08/16 11:27, Clemens Ladisch wrote:
>> Clemens Ladisch (2):
>> amidi: ignore not only Active Sensing but also Clock bytes
>> amidi: fix timeout handling
>>
>> amidi/amidi.1 | 19 ++++++++---
>> amidi/amidi.c | 82 +++++++++++++++++++++++++++++++++++++++-----------
>> 2 files changed, 78 insertions(+), 23 deletions(-)
>>
>
> I can't see the patches emails.
http://mailman.alsa-project.org/pipermail/alsa-devel/2016-August/111737.html
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 0/2] amidi: fix handling of ignored messages
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
` (2 preceding siblings ...)
2016-08-14 10:19 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Takashi Sakamoto
@ 2016-08-22 9:20 ` Takashi Iwai
3 siblings, 0 replies; 12+ messages in thread
From: Takashi Iwai @ 2016-08-22 9:20 UTC (permalink / raw)
To: Clemens Ladisch; +Cc: alsa-devel
On Sat, 13 Aug 2016 16:40:43 +0200,
Clemens Ladisch wrote:
>
> Clemens Ladisch (2):
> amidi: ignore not only Active Sensing but also Clock bytes
> amidi: fix timeout handling
>
> v2: fix short_options
Applied both patches now. Thanks!
Takashi
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2016-08-22 9:20 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-13 10:27 [PATCH 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 10:28 ` [PATCH 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
2016-08-13 11:14 ` Takashi Sakamoto
2016-08-13 11:27 ` Takashi Sakamoto
2016-08-13 14:40 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 1/2] amidi: ignore not only Active Sensing but also Clock bytes Clemens Ladisch
2016-08-13 14:41 ` [PATCH v2 2/2] amidi: fix timeout handling Clemens Ladisch
2016-08-14 10:19 ` [PATCH v2 0/2] amidi: fix handling of ignored messages Takashi Sakamoto
2016-08-22 9:20 ` Takashi Iwai
2016-08-13 10:29 ` [PATCH 2/2] amidi: fix timeout handling Clemens Ladisch
2016-08-15 14:10 ` [PATCH 0/2] amidi: fix handling of ignored messages Felipe Ferreri Tonello
2016-08-15 14:31 ` Clemens Ladisch
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.