Re: [PATCH] alsa-info.sh: add dmesg info on ALSA/HDA

[Takashi Iwai <tiwai@suse.de>]

At Thu, 16 Jul 2009 19:24:06 +0800,
Wu Fengguang wrote:
> 
> On Thu, Jul 16, 2009 at 05:42:09PM +0800, Takashi Iwai wrote:
> > At Wed, 08 Jul 2009 21:07:22 +0800,
> > Wu Fengguang wrote:
> > > 
> > > Hi Takashi,
> > > 
> > > This patchset mainly disables the auto-upload of alsa info by default.
> > 
> > I pushed the patches now.  Thanks!
> 
> Thank you!
> 
> btw, I find it often necessary to query user for the dmesg info.
> How about adding dmesg to alsa-info?

A good idea.  Applied now.  Thanks!


Takashi

> 
> Thanks,
> Fengguang
> ---
> alsa-info.sh: add dmesg info on ALSA/HDA
> 
> Add outputs:
> 		dmesg | grep -E 'ALSA|HDA|HDMI|sound|hda.codec|hda.intel'
> which should cover most ALSA HDA kernel messages.
> 
> Signed-off-by: Wu Fengguang <fengguang.wu@intel.com>
> ---
>  utils/alsa-info.sh |   44 ++++++++++++++++++++++++++++++++++++-------
>  1 file changed, 37 insertions(+), 7 deletions(-)
> 
> --- alsa-driver.orig/utils/alsa-info.sh
> +++ alsa-driver/utils/alsa-info.sh
> @@ -243,6 +243,15 @@ withsysfs() {
>      fi
>  }
>  
> +withdmesg() {
> +	echo "!!ALSA/HDA dmesg" >> $FILE
> +	echo "!!------------------" >> $FILE
> +	echo "" >> $FILE
> +	dmesg | grep -E 'ALSA|HDA|HDMI|sound|hda.codec|hda.intel' >> $FILE
> +	echo "" >> $FILE
> +	echo "" >> $FILE
> +}
> +
>  withall() {
>  	withdevices
>  	withconfigs
> @@ -251,6 +260,7 @@ withall() {
>  	withalsactl
>  	withlsmod
>  	withsysfs
> +	withdmesg
>  }
>  
>  get_alsa_library_version() {
> @@ -341,15 +351,31 @@ done
>  
>  #Script header output.
>  if [ "$WELCOME" = "yes" ]; then
> +greeting_message="\
> +
> +This script visits the following commands/files to collect diagnostic
> +information about your ALSA installation and sound related hardware.
> +
> +  dmesg
> +  lspci
> +  lsmod
> +  aplay
> +  amixer
> +  alsactl
> +  /proc/asound/
> +  /sys/class/sound/
> +  ~/.asoundrc (etc.)
> +
> +See '$0 --help' for command line options.
> +"
>  if [[ -n "$DIALOG" ]]; then
> -	dialog --backtitle "$BGTITLE" --title "ALSA-Info script v $SCRIPT_VERSION" --msgbox "\nThis script visits the following commands/files to collect diagnostic information about your ALSA installation and sound related hardware.\n\n  lspci\n  lsmod\n  aplay\n  amixer\n  alsactl\n  /proc/asound/\n  /sys/class/sound/\n  ~/.asoundrc (etc.)\n\nSee '$0 --help' for command line options.\n" 20 80
> +	dialog  --backtitle "$BGTITLE" \
> +		--title "ALSA-Info script v $SCRIPT_VERSION" \
> +		--msgbox "$greeting_message" 20 80
>  else
> -
> -echo "ALSA Information Script v $SCRIPT_VERSION"
> -echo "--------------------------------"
> -echo ""
> -echo "This script will collect information about your ALSA installation and sound related hardware, to help diagnose your problem."
> -echo ""
> +	echo "ALSA Information Script v $SCRIPT_VERSION"
> +	echo "--------------------------------"
> +	echo "$greeting_message"
>  fi # dialog
>  fi # WELCOME
>  
> @@ -619,6 +645,9 @@ then
>  		--with-devices)
>  			withdevices
>  			;;
> +		--with-dmesg)
> +			withdmesg
> +			;;
>  		--with-configs)
>  			if [[ -e $HOME/.asoundrc ]] || [[ -e /etc/asound.conf ]]
>  			then
> @@ -674,6 +703,7 @@ then
>  			echo "	--with-configs (includes the output of ~/.asoundrc and"
>  			echo "	    /etc/asound.conf if they exist)" 
>  			echo "	--with-devices (shows the device nodes in /dev/snd/)"
> +			echo "	--with-dmesg (shows the ALSA/HDA kernel messages)"
>  			echo ""
>  			echo "	--update (check server for script updates)"
>  			echo "	--upload (upload contents to remote server)"
>