From nobody Sat Jan 29 17:57:57 2022 X-Original-To: dev-commits-ports-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id CBC851983943; Sat, 29 Jan 2022 17:57:57 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4JmMVs4FfQz3mCp; Sat, 29 Jan 2022 17:57:57 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1643479077; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=QT50vruxOdR+WVUWWosigtolRF8qEgVkyhyA/mXopgk=; b=o6Vp80iLMhWu2n/X+mhD1kcjLzuoqwono4F6NUKp6FRrGHWVxH1t48Cp9HZ6BjC8XBXLPl /GHQII46vGnsZFmBc3t0CwhBVX1EmtDRVgXj7x7JoeSV1W2SASaPTxUr+/8R2Ov2gZ9yJs aN4uP9Ix4ZLZcOfQL1ZVs2v4xwS7fcmtwqV55sohFS6zUKBHAN2K3z46/I4X+k9qxoHDyA uoXUhgMLWovwiN1dAOwj837i1j8g8h8WVtd3GfYLu9duQUqeAwwYNwX53Sxke1K6s6Pfei /jRdEW20g1HNR++2yym77OCPwIwQLfzsMi+9x4UD7CDiA0r9fSBtcYPEBoJcVA== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 6EF8319E39; Sat, 29 Jan 2022 17:57:57 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 20THvvUv079435; Sat, 29 Jan 2022 17:57:57 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 20THvvWe079434; Sat, 29 Jan 2022 17:57:57 GMT (envelope-from git) Date: Sat, 29 Jan 2022 17:57:57 GMT Message-Id: <202201291757.20THvvWe079434@gitrepo.freebsd.org> To: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-main@FreeBSD.org From: Hans Petter Selasky Subject: git: fedba31a63e5 - main - audio/alsa-plugins: Try to cleanup ALSA-OSS backend patch. List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: hselasky X-Git-Repository: ports X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: fedba31a63e5c76aa5b6caad199530b01e759e34 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1643479077; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=QT50vruxOdR+WVUWWosigtolRF8qEgVkyhyA/mXopgk=; b=JghHuj5AycjWSqWgoQZOQkmlzdKUu0Tt7mZs2Urfu75jeqmfP+Cxn1fbXHqju99qc96ioi S7BtvQtbo0P6xVfatuqZVWL4M3Yy5npmqgRCnZMCzyttcIOz6vsrRvxGM/kzOl+SBy6yB6 YcRZty+pGiZBkg3fHWX1Nq7Qbwr1sPCa+HwOL2pYAbeyU+10aOpf7vcqVO1JZcPkYeryA3 iGx+fWB1epQ7vEixjKqjeW0MN3v/DAmQ4zQ1GATZKOfxm/8fwXDp4y7OUTrylxBYWnKK0/ pyjdSqrKUhUOKvPg5YSgLRPwlwEirvOQJ6zYbov13VnY7hi6Jm2g1B/7gVnTMA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1643479077; a=rsa-sha256; cv=none; b=LrBT8ps4jB5Y9rUGzlalEoq6NxWfgWxEwx+oyQpLq1lk2+dyX1SiBTzm8MZjPOyTH8Yauu XcYxgKa6euC/vwHusxPv+DpWzCd1ktMZ58IBrYTdEW4LImREb2rn8BwcXWf8eW7KySvWo/ vivBHRyxEm3hM+5XNZIphToUN1DKY+D50Amo7gyJnux0kpT9UHs/NeIzVDwDcpGAT67gY2 diXOlQVcPYfcfM+uob/JcD1Y+teNo6pqZgc5vCb77F7tSzVtot/AHOh5qWrpgOBrmBxIl+ Ydx/HPWXSSEEH4xbJF06LK/s6krADt6yASOR50ZQSbGJfcWfNBitgPPKqbR8QQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by hselasky: URL: https://cgit.FreeBSD.org/ports/commit/?id=fedba31a63e5c76aa5b6caad199530b01e759e34 commit fedba31a63e5c76aa5b6caad199530b01e759e34 Author: Hans Petter Selasky AuthorDate: 2022-01-29 17:47:07 +0000 Commit: Hans Petter Selasky CommitDate: 2022-01-29 17:56:52 +0000 audio/alsa-plugins: Try to cleanup ALSA-OSS backend patch. Approved by: pi (implicit) --- audio/alsa-plugins/Makefile | 10 +- audio/alsa-plugins/files/patch-oss_pcm__oss.c | 402 ++++++++++++-------------- 2 files changed, 187 insertions(+), 225 deletions(-) diff --git a/audio/alsa-plugins/Makefile b/audio/alsa-plugins/Makefile index 2c490bc45803..e8a7d1eec77e 100644 --- a/audio/alsa-plugins/Makefile +++ b/audio/alsa-plugins/Makefile @@ -2,7 +2,7 @@ PORTNAME= alsa-plugins PORTVERSION= 1.2.2 -PORTREVISION= 4 +PORTREVISION= 5 DISTVERSIONPREFIX= v CATEGORIES= audio MASTER_SITES= GH @@ -25,22 +25,20 @@ MAKE_ARGS+= RM="${RM}" INSTALL_TARGET= install-strip CPPFLAGS+= -I${.CURDIR}/../alsa-lib/files -OPTIONS_DEFAULT= IO_PTR PULSEAUDIO +OPTIONS_DEFAULT= PULSEAUDIO OPTIONS_DEFINE= BUFSZ_P2 FFMPEG JACK PULSEAUDIO SAMPLERATE SPEEX OPTIONS_SUB= yes OPTIONS_GROUP= OSS -OPTIONS_GROUP_OSS= BLKCNT_P2 BUFSZ_P2 IO_PTR VERBOSE +OPTIONS_GROUP_OSS= BLKCNT_P2 BUFSZ_P2 VERBOSE -IO_PTR_DESC= Precise playback/recording pointer BLKCNT_P2_DESC= Restrict number of fragments to ^2 aligned BUFSZ_P2_DESC= Restrict buffer size to ^2 aligned VERBOSE_DESC= Print debugging messages -IO_PTR_CFLAGS= -DFREEBSD_OSS_USE_IO_PTR BLKCNT_P2_CFLAGS= -DFREEBSD_OSS_BLKCNT_P2 BUFSZ_P2_CFLAGS= -DFREEBSD_OSS_BUFSZ_P2 -VERBOSE_CFLAGS= -DFREEBSD_OSS_DEBUG_VERBOSE +VERBOSE_CFLAGS= -DALSA_OSS_DEBUG_VERBOSE FFMPEG_LIB_DEPENDS= libavcodec.so:multimedia/ffmpeg FFMPEG_CONFIGURE_ENABLE= libav diff --git a/audio/alsa-plugins/files/patch-oss_pcm__oss.c b/audio/alsa-plugins/files/patch-oss_pcm__oss.c index 930bada70b5b..5b2e048e45b1 100644 --- a/audio/alsa-plugins/files/patch-oss_pcm__oss.c +++ b/audio/alsa-plugins/files/patch-oss_pcm__oss.c @@ -1,6 +1,10 @@ ---- oss/pcm_oss.c.orig 2022-01-29 13:06:30 UTC +--- oss/pcm_oss.c.orig 2020-02-19 09:35:42 UTC +++ oss/pcm_oss.c -@@ -22,17 +22,45 @@ +@@ -19,21 +19,45 @@ + */ + + #include ++#include #include #include #include @@ -10,43 +14,44 @@ +#include +#endif -+#define ARRAY_SIZE(x) (sizeof(x) / sizeof(*(x))) ++#ifndef ARRAY_SIZE ++#define ARRAY_SIZE(x) (sizeof(x) / sizeof(*(x))) ++#endif + -+#ifdef __FreeBSD__ -+#define FREEBSD_OSS_RATE_MIN 1 -+#define FREEBSD_OSS_RATE_MAX 384000 ++#define ALSA_OSS_RATE_MIN 1 ++#define ALSA_OSS_RATE_MAX 384000 + -+#define FREEBSD_OSS_CHANNELS_MIN 1 -+#define FREEBSD_OSS_CHANNELS_MAX 8 ++#define ALSA_OSS_CHANNELS_MIN 1 ++#define ALSA_OSS_CHANNELS_MAX 8 + -+#define FREEBSD_OSS_BUFSZ_MAX 131072 -+#define FREEBSD_OSS_BLKCNT_MIN 2 -+#define FREEBSD_OSS_BLKSZ_MIN 16 /* (FREEBSD_OSS_CHANNELS_MAX * 4) */ ++#define ALSA_OSS_BUFSZ_MAX 131072 ++#define ALSA_OSS_BLKCNT_MIN 2 ++#define ALSA_OSS_BLKSZ_MIN 16 /* (ALSA_OSS_CHANNELS_MAX * 4) */ + -+#define FREEBSD_OSS_BUFSZ_MIN (FREEBSD_OSS_BLKCNT_MIN * FREEBSD_OSS_BLKSZ_MIN) -+#define FREEBSD_OSS_BLKCNT_MAX (FREEBSD_OSS_BUFSZ_MAX / FREEBSD_OSS_BUFSZ_MIN) -+#define FREEBSD_OSS_BLKSZ_MAX (FREEBSD_OSS_BUFSZ_MAX / FREEBSD_OSS_BLKCNT_MIN) -+#endif ++#define ALSA_OSS_BUFSZ_MIN (ALSA_OSS_BLKCNT_MIN * ALSA_OSS_BLKSZ_MIN) ++#define ALSA_OSS_BLKCNT_MAX (ALSA_OSS_BUFSZ_MAX / ALSA_OSS_BUFSZ_MIN) ++#define ALSA_OSS_BLKSZ_MAX (ALSA_OSS_BUFSZ_MAX / ALSA_OSS_BLKCNT_MIN) + typedef struct snd_pcm_oss { snd_pcm_ioplug_t io; char *device; int fd; -+#ifdef __FreeBSD__ -+ int bufsz, ptr, ptr_align, last_bytes; -+#else - int fragment_set; - int caps; -+#endif +- int fragment_set; +- int caps; ++ int bufsz; ++ int ptr; ++ int ptr_align; int format; -+#ifndef __FreeBSD__ - unsigned int period_shift; - unsigned int periods; -+#endif - unsigned int frame_bytes; +- unsigned int period_shift; +- unsigned int periods; +- unsigned int frame_bytes; ++ int frame_bytes; ++ int last_bytes; ++ bool buffer_used; } snd_pcm_oss_t; -@@ -49,8 +77,21 @@ static snd_pcm_sframes_t oss_write(snd_pcm_ioplug_t *i + static snd_pcm_sframes_t oss_write(snd_pcm_ioplug_t *io, +@@ -49,8 +73,21 @@ static snd_pcm_sframes_t oss_write(snd_pcm_ioplug_t *i buf = (char *)areas->addr + (areas->first + areas->step * offset) / 8; size *= oss->frame_bytes; result = write(oss->fd, buf, size); @@ -70,7 +75,7 @@ return result / oss->frame_bytes; } -@@ -67,14 +108,88 @@ static snd_pcm_sframes_t oss_read(snd_pcm_ioplug_t *io +@@ -67,24 +104,66 @@ static snd_pcm_sframes_t oss_read(snd_pcm_ioplug_t *io buf = (char *)areas->addr + (areas->first + areas->step * offset) / 8; size *= oss->frame_bytes; result = read(oss->fd, buf, size); @@ -96,29 +101,43 @@ static snd_pcm_sframes_t oss_pointer(snd_pcm_ioplug_t *io) { -+#ifdef __FreeBSD__ snd_pcm_oss_t *oss = io->private_data; -+#ifdef FREEBSD_OSS_USE_IO_PTR +- struct count_info info; +- int ptr; + struct count_info ci; -+#endif + audio_buf_info bi; + +- if (ioctl(oss->fd, io->stream == SND_PCM_STREAM_PLAYBACK ? +- SNDCTL_DSP_GETOPTR : SNDCTL_DSP_GETIPTR, &info) < 0) { +- fprintf(stderr, "*** OSS: oss_pointer error\n"); +- return 0; ++ if (io->stream == SND_PCM_STREAM_PLAYBACK) { ++ if (ioctl(oss->fd, SNDCTL_DSP_GETOPTR, &ci) < 0) ++ return -EINVAL; ++ if (ioctl(oss->fd, SNDCTL_DSP_GETOSPACE, &bi) < 0) ++ return -EINVAL; ++ } else { ++ if (ioctl(oss->fd, SNDCTL_DSP_GETIPTR, &ci) < 0) ++ return -EINVAL; ++ if (ioctl(oss->fd, SNDCTL_DSP_GETISPACE, &bi) < 0) ++ return -EINVAL; + } +- ptr = snd_pcm_bytes_to_frames(io->pcm, info.ptr); +- return ptr; + -+ if (io->state != SND_PCM_STATE_RUNNING) -+ return 0; -+ -+ if (io->state == SND_PCM_STATE_XRUN) -+ return -EPIPE; -+ -+#ifdef FREEBSD_OSS_USE_IO_PTR -+ if (ioctl(oss->fd, (io->stream == SND_PCM_STREAM_PLAYBACK) ? -+ SNDCTL_DSP_GETOPTR : SNDCTL_DSP_GETIPTR, &ci) < 0) -+ return -EINVAL; ++ /* check for over- and under- run */ ++ if (bi.bytes != oss->bufsz) { ++ oss->buffer_used = true; ++ } else { ++ /* only report error once */ ++ if (oss->buffer_used == true) { ++ oss->buffer_used = false; ++ return -EPIPE; ++ } ++ } + -+ if (ci.ptr == oss->last_bytes && -+ ((ioctl(oss->fd, (io->stream == SND_PCM_STREAM_PLAYBACK) ? -+ SNDCTL_DSP_GETOSPACE : SNDCTL_DSP_GETISPACE, &bi) < 0) || -+ bi.bytes == oss->bufsz)) -+ return -EPIPE; ++ if (oss->last_bytes == -1) ++ oss->last_bytes = ci.ptr; + + if (ci.ptr < oss->last_bytes) + oss->ptr += oss->bufsz; @@ -128,59 +147,16 @@ + oss->ptr %= oss->ptr_align; + + oss->last_bytes = ci.ptr; -+#else /* !FREEBSD_OSS_USE_IO_PTR */ -+ if (ioctl(oss->fd, (io->stream == SND_PCM_STREAM_PLAYBACK) ? -+ SNDCTL_DSP_GETOSPACE : SNDCTL_DSP_GETISPACE, &bi) < 0) -+ return -EINVAL; -+ -+ if (bi.bytes == oss->bufsz && bi.bytes == oss->last_bytes) { -+#if 0 -+#ifdef SNDCTL_DSP_GETERROR -+ audio_errinfo ei; -+ if (ioctl(oss->fd, SNDCTL_DSP_GETERROR, &ei) < 0 || -+ (io->stream == SND_PCM_STREAM_PLAYBACK && -+ ei.play_underruns != 0) || -+ (io->stream == SND_PCM_STREAM_CAPTURE && -+ ei.rec_overruns != 0)) -+#endif -+#endif -+ return -EPIPE; -+ } -+ -+ if (bi.bytes > oss->last_bytes) { -+ oss->ptr += bi.bytes - oss->last_bytes; -+ oss->ptr %= oss->ptr_align; -+ } -+ -+ oss->last_bytes = bi.bytes; -+#endif /* FREEBSD_OSS_USE_IO_PTR */ + + return snd_pcm_bytes_to_frames(io->pcm, oss->ptr); -+#else -+ snd_pcm_oss_t *oss = io->private_data; - struct count_info info; - int ptr; - -@@ -85,20 +200,59 @@ static snd_pcm_sframes_t oss_pointer(snd_pcm_ioplug_t - } - ptr = snd_pcm_bytes_to_frames(io->pcm, info.ptr); - return ptr; -+#endif } static int oss_start(snd_pcm_ioplug_t *io) - { - snd_pcm_oss_t *oss = io->private_data; -+#ifdef __FreeBSD__ -+ audio_buf_info bi; -+#ifdef FREEBSD_OSS_USE_IO_PTR -+ struct count_info ci; -+#endif -+#endif +@@ -93,12 +172,25 @@ static int oss_start(snd_pcm_ioplug_t *io) int tmp = io->stream == SND_PCM_STREAM_PLAYBACK ? PCM_ENABLE_OUTPUT : PCM_ENABLE_INPUT; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + @@ -195,48 +171,29 @@ +#endif } + -+#ifdef __FreeBSD__ -+ if (ioctl(oss->fd, (io->stream == SND_PCM_STREAM_PLAYBACK) ? -+ SNDCTL_DSP_GETOSPACE : SNDCTL_DSP_GETISPACE, &bi) < 0) -+ return -EINVAL; -+ -+ if (oss->bufsz != (bi.fragsize * bi.fragstotal)) { -+ fprintf(stderr, "%s(): WARNING - bufsz changed! %d -> %d\n", -+ __func__, oss->bufsz, bi.fragsize * bi.fragstotal); -+ oss->bufsz = bi.fragsize * bi.fragstotal; -+ } -+ -+#ifdef FREEBSD_OSS_USE_IO_PTR -+ if (ioctl(oss->fd, (io->stream == SND_PCM_STREAM_PLAYBACK) ? -+ SNDCTL_DSP_GETOPTR : SNDCTL_DSP_GETIPTR, &ci) < 0) -+ return -EINVAL; -+ -+ oss->last_bytes = ci.ptr; -+#else -+ oss->last_bytes = bi.bytes; -+#endif ++ oss->last_bytes = -1; + oss->ptr = 0; -+#endif ++ oss->buffer_used = false; + return 0; } -@@ -107,6 +261,10 @@ static int oss_stop(snd_pcm_ioplug_t *io) +@@ -107,6 +199,10 @@ static int oss_stop(snd_pcm_ioplug_t *io) snd_pcm_oss_t *oss = io->private_data; int tmp = 0; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + ioctl(oss->fd, SNDCTL_DSP_SETTRIGGER, &tmp); return 0; } -@@ -115,18 +273,46 @@ static int oss_drain(snd_pcm_ioplug_t *io) +@@ -115,101 +211,176 @@ static int oss_drain(snd_pcm_ioplug_t *io) { snd_pcm_oss_t *oss = io->private_data; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + @@ -246,48 +203,43 @@ return 0; } +-static int oss_prepare(snd_pcm_ioplug_t *io) +static int oss_delay(snd_pcm_ioplug_t *io, snd_pcm_sframes_t *delayp) -+{ -+ snd_pcm_oss_t *oss = io->private_data; -+ int tmp; -+ + { + snd_pcm_oss_t *oss = io->private_data; + int tmp; + +- ioctl(oss->fd, SNDCTL_DSP_RESET); + if (oss->fd < 0) + return -EBADFD; -+ + +- tmp = io->channels; +- if (ioctl(oss->fd, SNDCTL_DSP_CHANNELS, &tmp) < 0) { +- perror("SNDCTL_DSP_CHANNELS"); +- return -EINVAL; + if (io->stream == SND_PCM_STREAM_PLAYBACK) { + if (ioctl(oss->fd, SNDCTL_DSP_GETODELAY, &tmp) < 0 || tmp < 0) + tmp = 0; + } else { + tmp = 0; -+ } + } +- tmp = oss->format; +- if (ioctl(oss->fd, SNDCTL_DSP_SETFMT, &tmp) < 0) { +- perror("SNDCTL_DSP_SETFMT"); +- return -EINVAL; +- } +- tmp = io->rate; +- if (ioctl(oss->fd, SNDCTL_DSP_SPEED, &tmp) < 0 || +- tmp > io->rate * 1.01 || tmp < io->rate * 0.99) { +- perror("SNDCTL_DSP_SPEED"); +- return -EINVAL; +- } +- return 0; + *delayp = snd_pcm_bytes_to_frames(io->pcm, tmp); + + return (0); -+} -+ -+#ifndef __FreeBSD__ - static int oss_prepare(snd_pcm_ioplug_t *io) - { - snd_pcm_oss_t *oss = io->private_data; - int tmp; - -- ioctl(oss->fd, SNDCTL_DSP_RESET); -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) -+ fprintf(stderr, "%s()\n", __func__); -+#endif - -+ ioctl(oss->fd, SNDCTL_DSP_RESET, NULL); -+ - tmp = io->channels; - if (ioctl(oss->fd, SNDCTL_DSP_CHANNELS, &tmp) < 0) { - perror("SNDCTL_DSP_CHANNELS"); -@@ -145,16 +331,75 @@ static int oss_prepare(snd_pcm_ioplug_t *io) - } - return 0; } -+#endif -+#ifdef __FreeBSD__ +static const struct { + int oss_format; + snd_pcm_format_t alsa_format; @@ -322,50 +274,51 @@ + { AFMT_U32_LE, SND_PCM_FORMAT_U24_LE }, + { AFMT_U32_BE, SND_PCM_FORMAT_U24_BE }, +}; -+#endif + static int oss_hw_params(snd_pcm_ioplug_t *io, snd_pcm_hw_params_t *params ATTRIBUTE_UNUSED) { snd_pcm_oss_t *oss = io->private_data; int i, tmp, err; -+#ifdef __FreeBSD__ +- unsigned int period_bytes; + int blksz_shift, blkcnt; + audio_buf_info bi; -+#else - unsigned int period_bytes; -+#endif long oflags, flags; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + oss->frame_bytes = (snd_pcm_format_physical_width(io->format) * io->channels) / 8; -+#ifdef __FreeBSD__ +- switch (io->format) { +- case SND_PCM_FORMAT_U8: +- oss->format = AFMT_U8; +- break; +- case SND_PCM_FORMAT_S16_LE: +- oss->format = AFMT_S16_LE; +- break; +- case SND_PCM_FORMAT_S16_BE: +- oss->format = AFMT_S16_BE; +- break; +- default: + oss->ptr_align = io->buffer_size * oss->frame_bytes; + + oss->format = 0; -+ for (i = 0; i < ARRAY_SIZE(oss_formats_tab); i++) { ++ for (i = 0; i != ARRAY_SIZE(oss_formats_tab); i++) { + if (oss_formats_tab[i].alsa_format == io->format) { + oss->format = oss_formats_tab[i].oss_format; + break; + } + } + if (oss->format == 0) { -+#else - switch (io->format) { - case SND_PCM_FORMAT_U8: - oss->format = AFMT_U8; -@@ -166,9 +411,93 @@ static int oss_hw_params(snd_pcm_ioplug_t *io, - oss->format = AFMT_S16_BE; - break; - default: -+#endif fprintf(stderr, "*** OSS: unsupported format %s\n", snd_pcm_format_name(io->format)); return -EINVAL; } -+#ifdef __FreeBSD__ +- period_bytes = io->period_size * oss->frame_bytes; +- oss->period_shift = 0; +- for (i = 31; i >= 4; i--) { +- if (period_bytes & (1U << i)) { +- oss->period_shift = i; + + ioctl(oss->fd, SNDCTL_DSP_RESET); + @@ -379,23 +332,52 @@ + /* get logarithmic value */ + for (blksz_shift = 0; blksz_shift < 24; blksz_shift++) { + if (tmp == (1 << blksz_shift)) -+ break; -+ } + break; +- } + } +- if (! oss->period_shift) { +- fprintf(stderr, "*** OSS: invalid period size %d\n", (int)io->period_size); +- return -EINVAL; + + tmp = io->buffer_size * oss->frame_bytes; + + /* compute HW buffer big enough to hold SW buffer */ -+ for (blkcnt = FREEBSD_OSS_BLKCNT_MIN; blkcnt != FREEBSD_OSS_BLKCNT_MAX; blkcnt *= 2) { ++ for (blkcnt = ALSA_OSS_BLKCNT_MIN; blkcnt != ALSA_OSS_BLKCNT_MAX; blkcnt *= 2) { + if ((blkcnt << blksz_shift) >= tmp) + break; -+ } -+ + } +- oss->periods = io->buffer_size / io->period_size; + +- _retry: +- tmp = oss->period_shift | (oss->periods << 16); + tmp = blksz_shift | (blkcnt << 16); -+ if (ioctl(oss->fd, SNDCTL_DSP_SETFRAGMENT, &tmp) < 0) { + if (ioctl(oss->fd, SNDCTL_DSP_SETFRAGMENT, &tmp) < 0) { +- if (! oss->fragment_set) { +- perror("SNDCTL_DSP_SETFRAGMENT"); +- fprintf(stderr, "*** period shift = %d, periods = %d\n", oss->period_shift, oss->periods); +- return -EINVAL; +- } +- /* OSS has no proper way to reinitialize the fragments */ +- /* try to reopen the device */ +- close(oss->fd); +- oss->fd = open(oss->device, io->stream == SND_PCM_STREAM_PLAYBACK ? +- O_WRONLY : O_RDONLY); +- if (oss->fd < 0) { +- err = -errno; +- SNDERR("Cannot reopen the device %s", oss->device); +- return err; +- } +- io->poll_fd = oss->fd; +- io->poll_events = io->stream == SND_PCM_STREAM_PLAYBACK ? +- POLLOUT : POLLIN; +- snd_pcm_ioplug_reinit_status(io); +- oss->fragment_set = 0; +- goto _retry; + perror("SNDCTL_DSP_SETFRAGMENTS"); + return -EINVAL; -+ } -+ + } +- oss->fragment_set = 1; + + tmp = oss->format; + if (ioctl(oss->fd, SNDCTL_DSP_SETFMT, &tmp) < 0 || + tmp != oss->format) { @@ -426,17 +408,16 @@ + oss->bufsz = bi.fragsize * bi.fragstotal; + +#ifdef SNDCTL_DSP_LOW_WATER -+ tmp = ((io->period_size * oss->frame_bytes) * 3) / 4; -+ tmp -= tmp % oss->frame_bytes; -+ if (tmp < oss->frame_bytes) -+ tmp = oss->frame_bytes; -+ if (tmp > bi.fragsize) -+ tmp = bi.fragsize; ++ tmp = io->period_size * oss->frame_bytes; ++ if (tmp > oss->bufsz) ++ tmp = oss->bufsz; ++ else if (tmp == 0) ++ tmp = 1; + if (ioctl(oss->fd, SNDCTL_DSP_LOW_WATER, &tmp) < 0) + perror("SNDCTL_DSP_LOW_WATER"); +#endif + -+#ifdef FREEBSD_OSS_DEBUG_VERBOSE ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, + "\n\n[%lu -> %d] %lu ~ %d -> %d, %lu ~ %d -> %d [d:%ld lw:%d]\n\n", + io->buffer_size / io->period_size, bi.fragstotal, @@ -447,19 +428,10 @@ + (long)(io->buffer_size * oss->frame_bytes) - + oss->bufsz, tmp); +#endif -+#else - period_bytes = io->period_size * oss->frame_bytes; - oss->period_shift = 0; - for (i = 31; i >= 4; i--) { -@@ -209,6 +538,7 @@ static int oss_hw_params(snd_pcm_ioplug_t *io, - goto _retry; - } - oss->fragment_set = 1; -+#endif - if ((flags = fcntl(oss->fd, F_GETFL)) < 0) { err = -errno; -@@ -229,16 +559,152 @@ static int oss_hw_params(snd_pcm_ioplug_t *io, + perror("F_GETFL"); +@@ -229,16 +400,148 @@ static int oss_hw_params(snd_pcm_ioplug_t *io, return 0; } @@ -481,13 +453,9 @@ +#endif unsigned int nformats; + unsigned int format[ARRAY_SIZE(oss_formats_tab)]; -+#if 0 -+ unsigned int nchannels; -+ unsigned int channel[FREEBSD_OSS_CHANNELS_MAX]; -+#endif + int i, err, tmp; + -+#ifdef FREEBSD_OSS_DEBUG_VERBOSE ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + @@ -536,13 +504,13 @@ + return err; +#endif + err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_CHANNELS, -+ FREEBSD_OSS_CHANNELS_MIN, FREEBSD_OSS_CHANNELS_MAX); ++ ALSA_OSS_CHANNELS_MIN, ALSA_OSS_CHANNELS_MAX); + if (err < 0) + return err; + + /* supported rates */ + err = snd_pcm_ioplug_set_param_minmax(io, SND_PCM_IOPLUG_HW_RATE, -+ FREEBSD_OSS_RATE_MIN, FREEBSD_OSS_RATE_MAX); ++ ALSA_OSS_RATE_MIN, ALSA_OSS_RATE_MAX); + if (err < 0) + return err; + @@ -555,9 +523,9 @@ +#ifdef FREEBSD_OSS_BLKCNT_P2 + tmp = 0; + for (i = 1; i < 31 && tmp < ARRAY_SIZE(period_list); i++) { -+ if ((1 << i) > FREEBSD_OSS_BLKCNT_MAX) ++ if ((1 << i) > ALSA_OSS_BLKCNT_MAX) + break; -+ if ((1 << i) < FREEBSD_OSS_BLKCNT_MIN) ++ if ((1 << i) < ALSA_OSS_BLKCNT_MIN) + continue; + period_list[tmp++] = 1 << i; + } @@ -569,24 +537,24 @@ +#endif + /* periods , not strictly ^2 but later on will be refined */ + err = snd_pcm_ioplug_set_param_minmax(io, -+ SND_PCM_IOPLUG_HW_PERIODS, FREEBSD_OSS_BLKCNT_MIN, -+ FREEBSD_OSS_BLKCNT_MAX); ++ SND_PCM_IOPLUG_HW_PERIODS, ALSA_OSS_BLKCNT_MIN, ++ ALSA_OSS_BLKCNT_MAX); + if (err < 0) + return err; + + /* period size , not strictly ^2 */ + err = snd_pcm_ioplug_set_param_minmax(io, -+ SND_PCM_IOPLUG_HW_PERIOD_BYTES, FREEBSD_OSS_BLKSZ_MIN, -+ FREEBSD_OSS_BLKSZ_MAX); ++ SND_PCM_IOPLUG_HW_PERIOD_BYTES, ALSA_OSS_BLKSZ_MIN, ++ ALSA_OSS_BLKSZ_MAX); + if (err < 0) + return err; + +#ifdef FREEBSD_OSS_BUFSZ_P2 + tmp = 0; + for (i = 1; i < 31 && tmp < ARRAY_SIZE(bufsz_list); i++) { -+ if ((1 << i) > FREEBSD_OSS_BUFSZ_MAX) ++ if ((1 << i) > ALSA_OSS_BUFSZ_MAX) + break; -+ if ((1 << i) < FREEBSD_OSS_BUFSZ_MIN) ++ if ((1 << i) < ALSA_OSS_BUFSZ_MIN) + continue; + bufsz_list[tmp++] = 1 << i; + } @@ -598,8 +566,8 @@ +#endif + /* buffer size , not strictly ^2 */ + err = snd_pcm_ioplug_set_param_minmax(io, -+ SND_PCM_IOPLUG_HW_BUFFER_BYTES, FREEBSD_OSS_BUFSZ_MIN, -+ FREEBSD_OSS_BUFSZ_MAX); ++ SND_PCM_IOPLUG_HW_BUFFER_BYTES, ALSA_OSS_BUFSZ_MIN, ++ ALSA_OSS_BUFSZ_MAX); + if (err < 0) + return err; + @@ -614,7 +582,7 @@ unsigned int format[5]; unsigned int nchannels; unsigned int channel[6]; -@@ -317,6 +783,7 @@ static int oss_hw_constraint(snd_pcm_oss_t *oss) +@@ -317,6 +620,7 @@ static int oss_hw_constraint(snd_pcm_oss_t *oss) return err; return 0; @@ -622,46 +590,42 @@ } -@@ -324,6 +791,10 @@ static int oss_close(snd_pcm_ioplug_t *io) +@@ -324,6 +628,10 @@ static int oss_close(snd_pcm_ioplug_t *io) { snd_pcm_oss_t *oss = io->private_data; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif + close(oss->fd); free(oss->device); free(oss); -@@ -337,8 +808,11 @@ static const snd_pcm_ioplug_callback_t oss_playback_ca +@@ -337,8 +645,8 @@ static const snd_pcm_ioplug_callback_t oss_playback_ca .pointer = oss_pointer, .close = oss_close, .hw_params = oss_hw_params, -+#ifndef __FreeBSD__ - .prepare = oss_prepare, -+#endif +- .prepare = oss_prepare, .drain = oss_drain, + .delay = oss_delay, }; static const snd_pcm_ioplug_callback_t oss_capture_callback = { -@@ -348,8 +822,11 @@ static const snd_pcm_ioplug_callback_t oss_capture_cal +@@ -348,8 +656,8 @@ static const snd_pcm_ioplug_callback_t oss_capture_cal .pointer = oss_pointer, .close = oss_close, .hw_params = oss_hw_params, -+#ifndef __FreeBSD__ - .prepare = oss_prepare, -+#endif +- .prepare = oss_prepare, .drain = oss_drain, + .delay = oss_delay, }; -@@ -360,6 +837,10 @@ SND_PCM_PLUGIN_DEFINE_FUNC(oss) +@@ -360,6 +668,10 @@ SND_PCM_PLUGIN_DEFINE_FUNC(oss) int err; snd_pcm_oss_t *oss; -+#if defined(__FreeBSD__) && defined(FREEBSD_OSS_DEBUG_VERBOSE) ++#ifdef ALSA_OSS_DEBUG_VERBOSE + fprintf(stderr, "%s()\n", __func__); +#endif +