Re: [PATCH 1/5] ALSA: pcm: Revert "ALSA: pcm: rewrite snd_pcm_playback_silence()"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 05 May 2023 11:38:45 +0200,
Oswald Buddenhagen wrote:
> 
> On Fri, May 05, 2023 at 11:31:16AM +0200, Takashi Iwai wrote:
> > On Fri, 05 May 2023 09:38:09 +0200,
> > Jaroslav Kysela wrote:
> >> Signed-off-by: Jaroslav Kysela <perex@xxxxxxxx>
> > 
> > Better to add Reported-by and the reference for the original thread
> > reporting the regression,
> > 
> i'll post a slight rework of the series shortly, where i'll include
> this.
> 
> > as well as the Fixes tag.
> > 
> that seems pointless for a revert, as all the info is already included
> anyway, no?

People tend to look at Fixes tag to judge whether the commit can fix
something real or not.  For example, we (at SUSE) regularly check
Fixes tag to pick up the missing fixing.
In the case of a revert, it doesn't mean always a fix.  So it's not
mandatory, but would be still helpful.


thanks,

Takashi



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux