Skip to content

Commit

Permalink
Add configure option to enable gcov analysis
Browse files Browse the repository at this point in the history
* Add configure option to enable gcov analysis.
* Includes a few minor ctime fixes.
* Add codecov.yml configuration.

Reviewed-by: Prakash Surya <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #6642
  • Loading branch information
behlendorf authored and tonyhutter committed Mar 14, 2018
1 parent 834815e commit 184087f
Show file tree
Hide file tree
Showing 7 changed files with 48 additions and 5 deletions.
9 changes: 9 additions & 0 deletions .github/codecov.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
codecov:
strict_yaml_branch: master # only use the latest copy on master branch

comment: off

coverage:
status:
project: off
patch: off
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@
*.mod.c
*~
*.swp
*.gcno
*.gcda
.deps
.libs
.dirstamp
Expand Down
8 changes: 4 additions & 4 deletions Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,10 @@ distclean-local::
-o -name .pc -o -name .hg -o -name .git \) -prune -o \
\( -name '*.orig' -o -name '*.rej' -o -name '*~' \
-o -name '*.bak' -o -name '#*#' -o -name '.*.orig' \
-o -name '.*.rej' -o -name '.script-config' -o -size 0 \
-o -name '*%' -o -name '.*.cmd' -o -name 'core' \
-o -name 'Makefile' -o -name 'Module.symvers' \
-o -name '*.order' -o -name '*.markers' \) \
-o -name '.*.rej' -o -size 0 -o -name '*%' -o -name '.*.cmd' \
-o -name 'core' -o -name 'Makefile' -o -name 'Module.symvers' \
-o -name '*.order' -o -name '*.markers' -o -name '*.gcda' \
-o -name '*.gcno' \) \
-type f -print | xargs $(RM)

dist-hook:
Expand Down
31 changes: 31 additions & 0 deletions config/zfs-build.m4
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,37 @@ AC_DEFUN([ZFS_AC_DEBUGINFO], [
AC_MSG_RESULT([$enable_debuginfo])
])

AC_DEFUN([ZFS_AC_GCOV_KERNEL], [
])

AC_DEFUN([ZFS_AC_GCOV_USER], [
DEBUG_CFLAGS="$DEBUG_CFLAGS -fprofile-arcs -ftest-coverage"
])

AC_DEFUN([ZFS_AC_GCOV], [
AC_MSG_CHECKING([whether gcov profiling will be enabled])
AC_ARG_ENABLE([gcov],
[AS_HELP_STRING([--enable-gcov],
[Enable gcov profiling @<:@default=no@:>@])],
[],
[enable_gcov=no])
AS_CASE(["x$enable_gcov"],
["xyes"],
[ZFS_AC_GCOV_KERNEL
ZFS_AC_GCOV_USER],
["xkernel"],
[ZFS_AC_GCOV_KERNEL],
["xuser"],
[ZFS_AC_GCOV_USER],
["xno"],
[],
[AC_MSG_ERROR([Unknown option $enable_gcov])])
AC_SUBST(DEBUG_CFLAGS)
AC_MSG_RESULT([$enable_gcov])
])

AC_DEFUN([ZFS_AC_CONFIG_ALWAYS], [
ZFS_AC_CONFIG_ALWAYS_NO_UNUSED_BUT_SET_VARIABLE
ZFS_AC_CONFIG_ALWAYS_NO_BOOL_COMPARE
Expand Down
1 change: 1 addition & 0 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ ZFS_AC_PACKAGE
ZFS_AC_CONFIG
ZFS_AC_DEBUG
ZFS_AC_DEBUGINFO
ZFS_AC_GCOV

AC_CONFIG_FILES([
Makefile
Expand Down
2 changes: 1 addition & 1 deletion tests/zfs-tests/tests/functional/ctime/.gitignore
Original file line number Diff line number Diff line change
@@ -1 +1 @@
/ctime_001_pos
/ctime
Empty file modified tests/zfs-tests/tests/functional/ctime/ctime_001_pos.ksh
100644 → 100755
Empty file.

0 comments on commit 184087f

Please sign in to comment.