Message ID | 1470868957-28770-1-git-send-email-greearb@candelatech.com |
---|---|
State | Changes Requested |
Headers | show |
Hi, the V2 does not address the changes i request in V1. these were * explain why we need this and what it does. right now we only know it "reports certain bugs" which is really very vague John On 11/08/2016 00:42, greearb@candelatech.com wrote: > From: Ben Greear <greearb@candelatech.com> > > This first release is all about checking for ath10k firmware > crashes. Could be extended later for other modules/bugs/etc. > > Signed-off-by: Ben Greear <greearb@candelatech.com> > --- > > v2: Add comments about this being for ath10k firmware > Change name to 'ct-bugcheck' since this is mostly about > ct ath10k firmware at this point. > > package/utils/ct-bugcheck/Makefile | 52 ++++++++++++ > package/utils/ct-bugcheck/src/bugcheck.initd | 16 ++++ > package/utils/ct-bugcheck/src/bugcheck.sh | 115 +++++++++++++++++++++++++++ > package/utils/ct-bugcheck/src/bugchecker.sh | 29 +++++++ > 4 files changed, 212 insertions(+) > create mode 100644 package/utils/ct-bugcheck/Makefile > create mode 100644 package/utils/ct-bugcheck/src/bugcheck.initd > create mode 100755 package/utils/ct-bugcheck/src/bugcheck.sh > create mode 100755 package/utils/ct-bugcheck/src/bugchecker.sh > > diff --git a/package/utils/ct-bugcheck/Makefile b/package/utils/ct-bugcheck/Makefile > new file mode 100644 > index 0000000..5deb1e0 > --- /dev/null > +++ b/package/utils/ct-bugcheck/Makefile > @@ -0,0 +1,52 @@ > +# > +# Copyright (C) 2016 OpenWrt.org > +# > +# This is free software, licensed under the GNU General Public License v2. > +# See /LICENSE for more information. > +# > + > +include $(TOPDIR)/rules.mk > +include $(INCLUDE_DIR)/kernel.mk > + > +PKG_NAME:=ct-bugcheck > +PKG_RELEASE:=2016-07-21 > + > +include $(INCLUDE_DIR)/package.mk > + > +define Package/ct-bugcheck > + SECTION:=utils > + CATEGORY:=Utilities > + TITLE:=Bug checking and reporting utility > + VERSION:=$(PKG_RELEASE) > + MAINTAINER:=Ben Greear <greearb@candelatech.com> > +endef > + > +define Package/ct-bugcheck/description > + Scripts to check for bugs (like firmware crashes) and package them for reporting. > + Currently this script only checks for ath10k firmware crashes. > + Once installed, you can enable this tool by creating a file called > + /etc/config/bugcheck with the following contents: > + DO_BUGCHECK=1 > + export DO_BUGCHECK > + > +endef > + > +define Build/Prepare > + $(CP) src/bugcheck.sh $(PKG_BUILD_DIR)/ > + $(CP) src/bugchecker.sh $(PKG_BUILD_DIR)/ > + $(CP) src/bugcheck.initd $(PKG_BUILD_DIR)/ > +endef > + > +define Build/Compile > + true > +endef > + > +define Package/ct-bugcheck/install > + $(INSTALL_DIR) $(1)/usr/bin > + $(INSTALL_DIR) $(1)/etc/init.d > + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.sh $(1)/usr/bin/ > + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugchecker.sh $(1)/usr/bin/ > + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.initd $(1)/etc/init.d/bugcheck > +endef > + > +$(eval $(call BuildPackage,ct-bugcheck)) > diff --git a/package/utils/ct-bugcheck/src/bugcheck.initd b/package/utils/ct-bugcheck/src/bugcheck.initd > new file mode 100644 > index 0000000..b97a415 > --- /dev/null > +++ b/package/utils/ct-bugcheck/src/bugcheck.initd > @@ -0,0 +1,16 @@ > +#!/bin/sh /etc/rc.common > +# Copyright (C) 2016 OpenWrt.org > + > +START=99 > + > +USE_PROCD=1 > +PROG=/usr/bin/bugchecker.sh > + > +# To actually make bugchecker.sh run, see comments > +# at top of its file. > + > +start_service () { > + procd_open_instance > + procd_set_param command "$PROG" > + procd_close_instance > +} > diff --git a/package/utils/ct-bugcheck/src/bugcheck.sh b/package/utils/ct-bugcheck/src/bugcheck.sh > new file mode 100755 > index 0000000..85f70c5 > --- /dev/null > +++ b/package/utils/ct-bugcheck/src/bugcheck.sh > @@ -0,0 +1,115 @@ > +#!/bin/sh > + > +# Check for ath10k (and maybe other) bugs, package them up, > +# and let user know what to do with them. > + > +TMPLOC=/tmp > +CRASHDIR=$TMPLOC/bugcheck > +FOUND_BUG=0 > + > +# set -x > + > +bugcheck_generic() > +{ > + echo "LEDE crashlog report" > $CRASHDIR/info.txt > + date >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "uname" >> $CRASHDIR/info.txt > + uname -a >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "os-release" >> $CRASHDIR/info.txt > + cat /etc/os-release >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "os-release" >> $CRASHDIR/info.txt > + cat /etc/os-release >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "dmesg output" >> $CRASHDIR/info.txt > + dmesg >> $CRASHDIR/info.txt > + if [ -x /usr/bin/lspci ] > + then > + echo >> $CRASHDIR/info.txt > + echo "lspci" >> $CRASHDIR/info.txt > + lspci >> $CRASHDIR/info.txt > + fi > + echo >> $CRASHDIR/info.txt > + echo "cpuinfo" >> $CRASHDIR/info.txt > + cat /proc/cpuinfo >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "meminfo" >> $CRASHDIR/info.txt > + cat /proc/cpuinfo >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "cmdline" >> $CRASHDIR/info.txt > + cat /proc/cmdline >> $CRASHDIR/info.txt > + echo >> $CRASHDIR/info.txt > + echo "lsmod" >> $CRASHDIR/info.txt > + lsmod >> $CRASHDIR/info.txt > +} > + > +roll_crashes() > +{ > + # Roll any existing crashes > + if [ -d $CRASHDIR ] > + then > + if [ -d $CRASHDIR.1 ] > + then > + rm -fr $CRASHDIR.1 > + fi > + mv $CRASHDIR $CRASHDIR.1 > + fi > + > + # Prepare location > + mkdir -p $CRASHDIR > +} > + > +# ath10k, check debugfs entries. > +for i in /sys/kernel/debug/ieee80211/*/ath10k/fw_crash_dump > +do > + #echo "Checking $i" > + if cat $i > $TMPLOC/ath10k_crash.bin 2>&1 > + then > + FOUND_BUG=1 > + > + #echo "Found ath10k crash data in $i" > + roll_crashes > + > + ADIR=${i/fw_crash_dump/} > + > + CTFW=0 > + if grep -- -ct- $TMPLOC/ath10k_crash.bin > /dev/null 2>&1 > + then > + CTFW=1 > + fi > + > + echo "Send bug reports to:" > $CRASHDIR/report_to.txt > + if [ -f $ADIR/ct_special -o $CTFW == "1" ] > + then > + # Looks like this is CT firmware or driver... > + echo "greearb@candelatech.com" >> $CRASHDIR/report_to.txt > + echo "and/or report or check for duplicates here:" >> $CRASHDIR/report_to.txt > + echo "https://github.com/greearb/ath10k-ct/issues" >> $CRASHDIR/report_to.txt > + else > + # Not sure who would want these bug reports for upstream... > + echo "https://www.lede-project.org/" >> $CRASHDIR/report_to.txt > + fi > + echo >> $CRASHDIR/report_to.txt > + echo "Please attach all files in this directory to bug reports." >> $CRASHDIR/report_to.txt > + > + mv $TMPLOC/ath10k_crash.bin $CRASHDIR > + > + # Add any more ath10k specific stuff here. > + > + # And call generic bug reporting logic > + bugcheck_generic > + fi > +done > + > +if [ $FOUND_BUG == "1" ] > + then > + # Notify LUCI somehow? > + echo "bugcheck.sh found an issue to be reported" > /dev/kmsg > + echo "See $CRASHDIR for details on how to report this" > /dev/kmsg > + # Let calling code know something was wrong. > + exit 1 > +fi > + > +exit 0 > diff --git a/package/utils/ct-bugcheck/src/bugchecker.sh b/package/utils/ct-bugcheck/src/bugchecker.sh > new file mode 100755 > index 0000000..be305af > --- /dev/null > +++ b/package/utils/ct-bugcheck/src/bugchecker.sh > @@ -0,0 +1,29 @@ > +#!/bin/sh > + > +# Periodically call bugcheck.sh script > + > +CHECKER=bugcheck.sh > +SLEEPFOR=60 > + > +DO_BUGCHECK=0 > + > +# So, to enable this, you create an /etc/config/bugcheck file > +# with contents like: > +# DO_BUGCHECK=1 > +# export DO_BUGCHECK > + > +if [ -f /etc/config/bugcheck ] > + then > + . /etc/config/bugcheck > +fi > + > +if [ $DO_BUGCHECK == 0 ] > +then > + exit 0 > +fi > + > +while true > + do > + $CHECKER > + sleep $SLEEPFOR > +done >
On 08/11/2016 11:36 PM, John Crispin wrote: > Hi, > > the V2 does not address the changes i request in V1. these were > > * explain why we need this and what it does. right now we only know it > "reports certain bugs" which is really very vague We need this because ath10k firmware will sometimes crash, and if users have a good way to generate reports, maybe they will and I can fix them so that ath10k firmware will crash less. I thought this part was pretty self evident. If you read the commit description, and the Makefile description, v2 specifies that it checks for ath10k firmware crashes. So, I am not really sure how you want the patch modified to meet your requests. Thanks, Ben > > John > > On 11/08/2016 00:42, greearb@candelatech.com wrote: >> From: Ben Greear <greearb@candelatech.com> >> >> This first release is all about checking for ath10k firmware >> crashes. Could be extended later for other modules/bugs/etc. >> >> Signed-off-by: Ben Greear <greearb@candelatech.com> >> --- >> >> v2: Add comments about this being for ath10k firmware >> Change name to 'ct-bugcheck' since this is mostly about >> ct ath10k firmware at this point. >> >> package/utils/ct-bugcheck/Makefile | 52 ++++++++++++ >> package/utils/ct-bugcheck/src/bugcheck.initd | 16 ++++ >> package/utils/ct-bugcheck/src/bugcheck.sh | 115 +++++++++++++++++++++++++++ >> package/utils/ct-bugcheck/src/bugchecker.sh | 29 +++++++ >> 4 files changed, 212 insertions(+) >> create mode 100644 package/utils/ct-bugcheck/Makefile >> create mode 100644 package/utils/ct-bugcheck/src/bugcheck.initd >> create mode 100755 package/utils/ct-bugcheck/src/bugcheck.sh >> create mode 100755 package/utils/ct-bugcheck/src/bugchecker.sh >> >> diff --git a/package/utils/ct-bugcheck/Makefile b/package/utils/ct-bugcheck/Makefile >> new file mode 100644 >> index 0000000..5deb1e0 >> --- /dev/null >> +++ b/package/utils/ct-bugcheck/Makefile >> @@ -0,0 +1,52 @@ >> +# >> +# Copyright (C) 2016 OpenWrt.org >> +# >> +# This is free software, licensed under the GNU General Public License v2. >> +# See /LICENSE for more information. >> +# >> + >> +include $(TOPDIR)/rules.mk >> +include $(INCLUDE_DIR)/kernel.mk >> + >> +PKG_NAME:=ct-bugcheck >> +PKG_RELEASE:=2016-07-21 >> + >> +include $(INCLUDE_DIR)/package.mk >> + >> +define Package/ct-bugcheck >> + SECTION:=utils >> + CATEGORY:=Utilities >> + TITLE:=Bug checking and reporting utility >> + VERSION:=$(PKG_RELEASE) >> + MAINTAINER:=Ben Greear <greearb@candelatech.com> >> +endef >> + >> +define Package/ct-bugcheck/description >> + Scripts to check for bugs (like firmware crashes) and package them for reporting. >> + Currently this script only checks for ath10k firmware crashes. >> + Once installed, you can enable this tool by creating a file called >> + /etc/config/bugcheck with the following contents: >> + DO_BUGCHECK=1 >> + export DO_BUGCHECK >> + >> +endef >> + >> +define Build/Prepare >> + $(CP) src/bugcheck.sh $(PKG_BUILD_DIR)/ >> + $(CP) src/bugchecker.sh $(PKG_BUILD_DIR)/ >> + $(CP) src/bugcheck.initd $(PKG_BUILD_DIR)/ >> +endef >> + >> +define Build/Compile >> + true >> +endef >> + >> +define Package/ct-bugcheck/install >> + $(INSTALL_DIR) $(1)/usr/bin >> + $(INSTALL_DIR) $(1)/etc/init.d >> + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.sh $(1)/usr/bin/ >> + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugchecker.sh $(1)/usr/bin/ >> + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.initd $(1)/etc/init.d/bugcheck >> +endef >> + >> +$(eval $(call BuildPackage,ct-bugcheck)) >> diff --git a/package/utils/ct-bugcheck/src/bugcheck.initd b/package/utils/ct-bugcheck/src/bugcheck.initd >> new file mode 100644 >> index 0000000..b97a415 >> --- /dev/null >> +++ b/package/utils/ct-bugcheck/src/bugcheck.initd >> @@ -0,0 +1,16 @@ >> +#!/bin/sh /etc/rc.common >> +# Copyright (C) 2016 OpenWrt.org >> + >> +START=99 >> + >> +USE_PROCD=1 >> +PROG=/usr/bin/bugchecker.sh >> + >> +# To actually make bugchecker.sh run, see comments >> +# at top of its file. >> + >> +start_service () { >> + procd_open_instance >> + procd_set_param command "$PROG" >> + procd_close_instance >> +} >> diff --git a/package/utils/ct-bugcheck/src/bugcheck.sh b/package/utils/ct-bugcheck/src/bugcheck.sh >> new file mode 100755 >> index 0000000..85f70c5 >> --- /dev/null >> +++ b/package/utils/ct-bugcheck/src/bugcheck.sh >> @@ -0,0 +1,115 @@ >> +#!/bin/sh >> + >> +# Check for ath10k (and maybe other) bugs, package them up, >> +# and let user know what to do with them. >> + >> +TMPLOC=/tmp >> +CRASHDIR=$TMPLOC/bugcheck >> +FOUND_BUG=0 >> + >> +# set -x >> + >> +bugcheck_generic() >> +{ >> + echo "LEDE crashlog report" > $CRASHDIR/info.txt >> + date >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "uname" >> $CRASHDIR/info.txt >> + uname -a >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "os-release" >> $CRASHDIR/info.txt >> + cat /etc/os-release >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "os-release" >> $CRASHDIR/info.txt >> + cat /etc/os-release >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "dmesg output" >> $CRASHDIR/info.txt >> + dmesg >> $CRASHDIR/info.txt >> + if [ -x /usr/bin/lspci ] >> + then >> + echo >> $CRASHDIR/info.txt >> + echo "lspci" >> $CRASHDIR/info.txt >> + lspci >> $CRASHDIR/info.txt >> + fi >> + echo >> $CRASHDIR/info.txt >> + echo "cpuinfo" >> $CRASHDIR/info.txt >> + cat /proc/cpuinfo >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "meminfo" >> $CRASHDIR/info.txt >> + cat /proc/cpuinfo >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "cmdline" >> $CRASHDIR/info.txt >> + cat /proc/cmdline >> $CRASHDIR/info.txt >> + echo >> $CRASHDIR/info.txt >> + echo "lsmod" >> $CRASHDIR/info.txt >> + lsmod >> $CRASHDIR/info.txt >> +} >> + >> +roll_crashes() >> +{ >> + # Roll any existing crashes >> + if [ -d $CRASHDIR ] >> + then >> + if [ -d $CRASHDIR.1 ] >> + then >> + rm -fr $CRASHDIR.1 >> + fi >> + mv $CRASHDIR $CRASHDIR.1 >> + fi >> + >> + # Prepare location >> + mkdir -p $CRASHDIR >> +} >> + >> +# ath10k, check debugfs entries. >> +for i in /sys/kernel/debug/ieee80211/*/ath10k/fw_crash_dump >> +do >> + #echo "Checking $i" >> + if cat $i > $TMPLOC/ath10k_crash.bin 2>&1 >> + then >> + FOUND_BUG=1 >> + >> + #echo "Found ath10k crash data in $i" >> + roll_crashes >> + >> + ADIR=${i/fw_crash_dump/} >> + >> + CTFW=0 >> + if grep -- -ct- $TMPLOC/ath10k_crash.bin > /dev/null 2>&1 >> + then >> + CTFW=1 >> + fi >> + >> + echo "Send bug reports to:" > $CRASHDIR/report_to.txt >> + if [ -f $ADIR/ct_special -o $CTFW == "1" ] >> + then >> + # Looks like this is CT firmware or driver... >> + echo "greearb@candelatech.com" >> $CRASHDIR/report_to.txt >> + echo "and/or report or check for duplicates here:" >> $CRASHDIR/report_to.txt >> + echo "https://github.com/greearb/ath10k-ct/issues" >> $CRASHDIR/report_to.txt >> + else >> + # Not sure who would want these bug reports for upstream... >> + echo "https://www.lede-project.org/" >> $CRASHDIR/report_to.txt >> + fi >> + echo >> $CRASHDIR/report_to.txt >> + echo "Please attach all files in this directory to bug reports." >> $CRASHDIR/report_to.txt >> + >> + mv $TMPLOC/ath10k_crash.bin $CRASHDIR >> + >> + # Add any more ath10k specific stuff here. >> + >> + # And call generic bug reporting logic >> + bugcheck_generic >> + fi >> +done >> + >> +if [ $FOUND_BUG == "1" ] >> + then >> + # Notify LUCI somehow? >> + echo "bugcheck.sh found an issue to be reported" > /dev/kmsg >> + echo "See $CRASHDIR for details on how to report this" > /dev/kmsg >> + # Let calling code know something was wrong. >> + exit 1 >> +fi >> + >> +exit 0 >> diff --git a/package/utils/ct-bugcheck/src/bugchecker.sh b/package/utils/ct-bugcheck/src/bugchecker.sh >> new file mode 100755 >> index 0000000..be305af >> --- /dev/null >> +++ b/package/utils/ct-bugcheck/src/bugchecker.sh >> @@ -0,0 +1,29 @@ >> +#!/bin/sh >> + >> +# Periodically call bugcheck.sh script >> + >> +CHECKER=bugcheck.sh >> +SLEEPFOR=60 >> + >> +DO_BUGCHECK=0 >> + >> +# So, to enable this, you create an /etc/config/bugcheck file >> +# with contents like: >> +# DO_BUGCHECK=1 >> +# export DO_BUGCHECK >> + >> +if [ -f /etc/config/bugcheck ] >> + then >> + . /etc/config/bugcheck >> +fi >> + >> +if [ $DO_BUGCHECK == 0 ] >> +then >> + exit 0 >> +fi >> + >> +while true >> + do >> + $CHECKER >> + sleep $SLEEPFOR >> +done >> >
Hi Ben, On 12-08-16 16:19, Ben Greear wrote: > > > On 08/11/2016 11:36 PM, John Crispin wrote: >> Hi, >> >> the V2 does not address the changes i request in V1. these were >> >> * explain why we need this and what it does. right now we only know it >> "reports certain bugs" which is really very vague > > We need this because ath10k firmware will sometimes crash, and if > users have > a good way to generate reports, maybe they will and I can fix them so > that > ath10k firmware will crash less. I thought this part was pretty self > evident. I agree that this can be useful. > > If you read the commit description, and the Makefile description, v2 > specifies > that it checks for ath10k firmware crashes. > > So, I am not really sure how you want the patch modified to meet your > requests. Please describe it like this in the commit message. The current one (what I didn't cut below) isn't that clear. > >>> From: Ben Greear <greearb@candelatech.com> >>> >>> This first release is all about checking for ath10k firmware >>> crashes. Could be extended later for other modules/bugs/etc. Thanks, Stijn
diff --git a/package/utils/ct-bugcheck/Makefile b/package/utils/ct-bugcheck/Makefile new file mode 100644 index 0000000..5deb1e0 --- /dev/null +++ b/package/utils/ct-bugcheck/Makefile @@ -0,0 +1,52 @@ +# +# Copyright (C) 2016 OpenWrt.org +# +# This is free software, licensed under the GNU General Public License v2. +# See /LICENSE for more information. +# + +include $(TOPDIR)/rules.mk +include $(INCLUDE_DIR)/kernel.mk + +PKG_NAME:=ct-bugcheck +PKG_RELEASE:=2016-07-21 + +include $(INCLUDE_DIR)/package.mk + +define Package/ct-bugcheck + SECTION:=utils + CATEGORY:=Utilities + TITLE:=Bug checking and reporting utility + VERSION:=$(PKG_RELEASE) + MAINTAINER:=Ben Greear <greearb@candelatech.com> +endef + +define Package/ct-bugcheck/description + Scripts to check for bugs (like firmware crashes) and package them for reporting. + Currently this script only checks for ath10k firmware crashes. + Once installed, you can enable this tool by creating a file called + /etc/config/bugcheck with the following contents: + DO_BUGCHECK=1 + export DO_BUGCHECK + +endef + +define Build/Prepare + $(CP) src/bugcheck.sh $(PKG_BUILD_DIR)/ + $(CP) src/bugchecker.sh $(PKG_BUILD_DIR)/ + $(CP) src/bugcheck.initd $(PKG_BUILD_DIR)/ +endef + +define Build/Compile + true +endef + +define Package/ct-bugcheck/install + $(INSTALL_DIR) $(1)/usr/bin + $(INSTALL_DIR) $(1)/etc/init.d + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.sh $(1)/usr/bin/ + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugchecker.sh $(1)/usr/bin/ + $(INSTALL_BIN) $(PKG_BUILD_DIR)/bugcheck.initd $(1)/etc/init.d/bugcheck +endef + +$(eval $(call BuildPackage,ct-bugcheck)) diff --git a/package/utils/ct-bugcheck/src/bugcheck.initd b/package/utils/ct-bugcheck/src/bugcheck.initd new file mode 100644 index 0000000..b97a415 --- /dev/null +++ b/package/utils/ct-bugcheck/src/bugcheck.initd @@ -0,0 +1,16 @@ +#!/bin/sh /etc/rc.common +# Copyright (C) 2016 OpenWrt.org + +START=99 + +USE_PROCD=1 +PROG=/usr/bin/bugchecker.sh + +# To actually make bugchecker.sh run, see comments +# at top of its file. + +start_service () { + procd_open_instance + procd_set_param command "$PROG" + procd_close_instance +} diff --git a/package/utils/ct-bugcheck/src/bugcheck.sh b/package/utils/ct-bugcheck/src/bugcheck.sh new file mode 100755 index 0000000..85f70c5 --- /dev/null +++ b/package/utils/ct-bugcheck/src/bugcheck.sh @@ -0,0 +1,115 @@ +#!/bin/sh + +# Check for ath10k (and maybe other) bugs, package them up, +# and let user know what to do with them. + +TMPLOC=/tmp +CRASHDIR=$TMPLOC/bugcheck +FOUND_BUG=0 + +# set -x + +bugcheck_generic() +{ + echo "LEDE crashlog report" > $CRASHDIR/info.txt + date >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "uname" >> $CRASHDIR/info.txt + uname -a >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "os-release" >> $CRASHDIR/info.txt + cat /etc/os-release >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "os-release" >> $CRASHDIR/info.txt + cat /etc/os-release >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "dmesg output" >> $CRASHDIR/info.txt + dmesg >> $CRASHDIR/info.txt + if [ -x /usr/bin/lspci ] + then + echo >> $CRASHDIR/info.txt + echo "lspci" >> $CRASHDIR/info.txt + lspci >> $CRASHDIR/info.txt + fi + echo >> $CRASHDIR/info.txt + echo "cpuinfo" >> $CRASHDIR/info.txt + cat /proc/cpuinfo >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "meminfo" >> $CRASHDIR/info.txt + cat /proc/cpuinfo >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "cmdline" >> $CRASHDIR/info.txt + cat /proc/cmdline >> $CRASHDIR/info.txt + echo >> $CRASHDIR/info.txt + echo "lsmod" >> $CRASHDIR/info.txt + lsmod >> $CRASHDIR/info.txt +} + +roll_crashes() +{ + # Roll any existing crashes + if [ -d $CRASHDIR ] + then + if [ -d $CRASHDIR.1 ] + then + rm -fr $CRASHDIR.1 + fi + mv $CRASHDIR $CRASHDIR.1 + fi + + # Prepare location + mkdir -p $CRASHDIR +} + +# ath10k, check debugfs entries. +for i in /sys/kernel/debug/ieee80211/*/ath10k/fw_crash_dump +do + #echo "Checking $i" + if cat $i > $TMPLOC/ath10k_crash.bin 2>&1 + then + FOUND_BUG=1 + + #echo "Found ath10k crash data in $i" + roll_crashes + + ADIR=${i/fw_crash_dump/} + + CTFW=0 + if grep -- -ct- $TMPLOC/ath10k_crash.bin > /dev/null 2>&1 + then + CTFW=1 + fi + + echo "Send bug reports to:" > $CRASHDIR/report_to.txt + if [ -f $ADIR/ct_special -o $CTFW == "1" ] + then + # Looks like this is CT firmware or driver... + echo "greearb@candelatech.com" >> $CRASHDIR/report_to.txt + echo "and/or report or check for duplicates here:" >> $CRASHDIR/report_to.txt + echo "https://github.com/greearb/ath10k-ct/issues" >> $CRASHDIR/report_to.txt + else + # Not sure who would want these bug reports for upstream... + echo "https://www.lede-project.org/" >> $CRASHDIR/report_to.txt + fi + echo >> $CRASHDIR/report_to.txt + echo "Please attach all files in this directory to bug reports." >> $CRASHDIR/report_to.txt + + mv $TMPLOC/ath10k_crash.bin $CRASHDIR + + # Add any more ath10k specific stuff here. + + # And call generic bug reporting logic + bugcheck_generic + fi +done + +if [ $FOUND_BUG == "1" ] + then + # Notify LUCI somehow? + echo "bugcheck.sh found an issue to be reported" > /dev/kmsg + echo "See $CRASHDIR for details on how to report this" > /dev/kmsg + # Let calling code know something was wrong. + exit 1 +fi + +exit 0 diff --git a/package/utils/ct-bugcheck/src/bugchecker.sh b/package/utils/ct-bugcheck/src/bugchecker.sh new file mode 100755 index 0000000..be305af --- /dev/null +++ b/package/utils/ct-bugcheck/src/bugchecker.sh @@ -0,0 +1,29 @@ +#!/bin/sh + +# Periodically call bugcheck.sh script + +CHECKER=bugcheck.sh +SLEEPFOR=60 + +DO_BUGCHECK=0 + +# So, to enable this, you create an /etc/config/bugcheck file +# with contents like: +# DO_BUGCHECK=1 +# export DO_BUGCHECK + +if [ -f /etc/config/bugcheck ] + then + . /etc/config/bugcheck +fi + +if [ $DO_BUGCHECK == 0 ] +then + exit 0 +fi + +while true + do + $CHECKER + sleep $SLEEPFOR +done