All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/5] snd-usb endpoint rework, take 5
@ 2011-12-20 23:34 Daniel Mack
  2011-12-20 23:34 ` [PATCH 1/5] ALSA: snd-usb: add snd_usb_audio-wide mutex Daniel Mack
                   ` (4 more replies)
  0 siblings, 5 replies; 20+ messages in thread
From: Daniel Mack @ 2011-12-20 23:34 UTC (permalink / raw)
  To: alsa-devel; +Cc: gdiffey, tiwai, clemens, linuxaudio, Daniel Mack, blablack

I actually wanted to wait for more input, but as the flaws reported so
far were rather minor, I addressed to following details in this new
round as mentioned by Takashi and Clemens:

 - removed the faulty "(!u || u->urb)" check
 - guard the result of min() with the spinlock inside next_packet_size()
 - removed the tasklet and made the handler a function that is called
   in-place

Thanks again for the reviews, and keep them coming :)

Daniel

Daniel Mack (5):
  ALSA: snd-usb: add snd_usb_audio-wide mutex
  ALSA: snd-usb: implement new endpoint streaming model
  ALSA: snd-usb: switch over to new endpoint streaming logic
  ALSA: snd-usb: remove old streaming logic
  ALSA: snd-usb: add support for implicit feedback

 sound/usb/card.c     |   10 +-
 sound/usb/card.h     |   62 +++
 sound/usb/endpoint.c | 1454 +++++++++++++++++++++++++------------------------
 sound/usb/endpoint.h |   32 +-
 sound/usb/pcm.c      |  437 +++++++++++++---
 sound/usb/stream.c   |   31 +-
 sound/usb/usbaudio.h |    2 +
 7 files changed, 1224 insertions(+), 804 deletions(-)

-- 
1.7.5.4

^ permalink raw reply	[flat|nested] 20+ messages in thread
* [PATCH 0/5] snd-usb endpoint rework, take 4
@ 2011-12-20  9:48 Daniel Mack
  2011-12-20  9:48 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
  0 siblings, 1 reply; 20+ messages in thread
From: Daniel Mack @ 2011-12-20  9:48 UTC (permalink / raw)
  To: alsa-devel; +Cc: gdiffey, tiwai, clemens, linuxaudio, Daniel Mack, blablack

As stated in reply to my previous series, the last version caused deadlocks
when quickly starting and stopping the PCM streams. These should finally be
fixed now. Tested with both the FTU and a sync-endpoint driven device.

Please check and report :)

Daniel


Daniel Mack (5):
  ALSA: snd-usb: add snd_usb_audio-wide mutex
  ALSA: snd-usb: implement new endpoint streaming model
  ALSA: snd-usb: switch over to new endpoint streaming logic
  ALSA: snd-usb: remove old streaming logic
  ALSA: snd-usb: add support for implicit feedback

 sound/usb/card.c     |   10 +-
 sound/usb/card.h     |   63 +++
 sound/usb/endpoint.c | 1458 ++++++++++++++++++++++++++------------------------
 sound/usb/endpoint.h |   32 +-
 sound/usb/pcm.c      |  435 +++++++++++++---
 sound/usb/stream.c   |   31 +-
 sound/usb/usbaudio.h |    2 +
 7 files changed, 1229 insertions(+), 802 deletions(-)

-- 
1.7.5.4

^ permalink raw reply	[flat|nested] 20+ messages in thread
* [PATCH 0/5] snd-usb endpoint logic rework, version 3
@ 2011-12-18 23:02 Daniel Mack
  2011-12-18 23:02 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
  0 siblings, 1 reply; 20+ messages in thread
From: Daniel Mack @ 2011-12-18 23:02 UTC (permalink / raw)
  To: alsa-devel; +Cc: gdiffey, tiwai, clemens, linuxaudio, Daniel Mack, blablack

This is the 3rd round, and I didn't mark it RFC anymore as Clemens and
Takashi agreed on the way of implementation in general.

So in comparison to the last version, this round incorporates some
spinlock fixes, a lot less debugging output and my S-o-b lines.

Please test this extensively with different sound cards.

I didn't experience a single problem over the last hours of testing,
but the more testing the better.


Thanks,
Daniel


Daniel Mack (5):
  ALSA: snd-usb: add snd_usb_audio-wide mutex
  ALSA: snd-usb: implement new endpoint streaming model
  ALSA: snd-usb: switch over to new endpoint streaming logic
  ALSA: snd-usb: remove old streaming logic
  ALSA: snd-usb: add support for implicit feedback

 sound/usb/card.c     |    8 +-
 sound/usb/card.h     |   65 +++
 sound/usb/endpoint.c | 1427 +++++++++++++++++++++++++-------------------------
 sound/usb/endpoint.h |   31 +-
 sound/usb/pcm.c      |  428 +++++++++++++---
 sound/usb/stream.c   |   31 +-
 sound/usb/usbaudio.h |    2 +
 7 files changed, 1191 insertions(+), 801 deletions(-)

-- 
1.7.5.4

^ permalink raw reply	[flat|nested] 20+ messages in thread
* [PATCH 0/5] RFC v2 for snd-usb endpoint logic rework
@ 2011-11-18 20:05 Daniel Mack
  2011-11-18 20:05 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
  0 siblings, 1 reply; 20+ messages in thread
From: Daniel Mack @ 2011-11-18 20:05 UTC (permalink / raw)
  To: alsa-devel; +Cc: gdiffey, tiwai, clemens, linuxaudio, Daniel Mack, blablack

This is v2 of the patch bulk I posted some weeks ago, which incoporates
the points that were discussed. Thanks to everyone who commented and
helped with sugesstions.

This version work really well here, but as last time, it needs a lot
more testing. What's still missing are some cleanups and more
documentation, I'll do that once the acutal content is agreed upon.

Patches should apply cleanly to the latest mainline git + Takashi's
sound tree.

Thanks for any feedback for this next round,
Daniel


Daniel Mack (5):
  ALSA: snd-usb: add snd_usb_audio-wide mutex
  ALSA: snd-usb: implement new endpoint streaming model
  ALSA: snd-usb: switch over to new endpoint streaming logic
  ALSA: snd-usb: remove old streaming logic
  ALSA: snd-usb: add support for implicit feedback

 sound/usb/card.c     |    8 +-
 sound/usb/card.h     |   65 +++
 sound/usb/endpoint.c | 1434 +++++++++++++++++++++++++-------------------------
 sound/usb/endpoint.h |   31 +-
 sound/usb/pcm.c      |  444 +++++++++++++---
 sound/usb/stream.c   |   31 +-
 sound/usb/usbaudio.h |    2 +
 7 files changed, 1214 insertions(+), 801 deletions(-)

-- 
1.7.5.4

^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2011-12-22 19:04 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-20 23:34 [PATCH 0/5] snd-usb endpoint rework, take 5 Daniel Mack
2011-12-20 23:34 ` [PATCH 1/5] ALSA: snd-usb: add snd_usb_audio-wide mutex Daniel Mack
2011-12-20 23:34 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
2011-12-21 16:16   ` Aurélien Leblond
2011-12-21 16:34     ` Daniel Mack
2011-12-21 17:11       ` Aurélien Leblond
2011-12-21 17:18         ` Daniel Mack
2011-12-22 17:55           ` Aurélien Leblond
2011-12-22 19:04             ` Daniel Mack
2011-12-20 23:34 ` [PATCH 3/5] ALSA: snd-usb: switch over to new endpoint streaming logic Daniel Mack
2011-12-20 23:34 ` [PATCH 4/5] ALSA: snd-usb: remove old " Daniel Mack
2011-12-20 23:34 ` [PATCH 5/5] ALSA: snd-usb: add support for implicit feedback Daniel Mack
  -- strict thread matches above, loose matches on Subject: below --
2011-12-20  9:48 [PATCH 0/5] snd-usb endpoint rework, take 4 Daniel Mack
2011-12-20  9:48 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
2011-12-20 15:08   ` Takashi Iwai
2011-12-20 15:32     ` Clemens Ladisch
2011-12-20 20:16       ` Daniel Mack
2011-12-20 15:42   ` Takashi Iwai
2011-12-20 20:08     ` Daniel Mack
2011-12-18 23:02 [PATCH 0/5] snd-usb endpoint logic rework, version 3 Daniel Mack
2011-12-18 23:02 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack
2011-11-18 20:05 [PATCH 0/5] RFC v2 for snd-usb endpoint logic rework Daniel Mack
2011-11-18 20:05 ` [PATCH 2/5] ALSA: snd-usb: implement new endpoint streaming model Daniel Mack

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.