netdev - linux-next: ath9k: build failure, ath_cmn_process

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20141114000747.GA6865@hudson.localdomain>
Date:	Thu, 13 Nov 2014 16:07:47 -0800
From:	Jeremiah Mahler <jmmahler@...il.com>
To:	Oleksij Rempel <linux@...pel-privat.de>
Cc:	Jeremiah Mahler <jmmahler@...il.com>, linux-kernel@...r.kernel.org,
	ath9k-devel@....qualcomm.com, linville@...driver.com,
	linux-wireless@...r.kernel.org, ath9k-devel@...ts.ath9k.org,
	netdev@...r.kernel.org
Subject: linux-next: ath9k: build failure, ath_cmn_process_fft() redefinition


In version 20141113 of the linux-next kernel, if it is compiled with
CONFIG_ATH9K_DEBUGFS unset, an error about ath_cmn_process_fft() being
redefined will be produced.

  make
  ...
    LD [M]  drivers/net/wireless/ath/ath9k/ath9k_hw.o
    CC [M]  drivers/net/wireless/ath/ath9k/common-spectral.o
    CC      lib/debug_locks.o
    CC      lib/random32.o
  drivers/net/wireless/ath/ath9k/common-spectral.c:40:5: error:
  redefinition of ‘ath_cmn_process_fft’
   int ath_cmn_process_fft(struct ath_spec_scan_priv *spec_priv, struct
  ieee80211_hdr *hdr,
       ^
  In file included from drivers/net/wireless/ath/ath9k/common.h:27:0,
                   from drivers/net/wireless/ath/ath9k/ath9k.h:27,
                   from
  drivers/net/wireless/ath/ath9k/common-spectral.c:18:
  drivers/net/wireless/ath/ath9k/common-spectral.h:146:19: note: previous
  definition of ‘ath_cmn_process_fft’ was here
   static inline int ath_cmn_process_fft(struct ath_spec_scan_priv
  *spec_priv,
                     ^
  scripts/Makefile.build:257: recipe for target
  'drivers/net/wireless/ath/ath9k/common-spectral.o' failed
  make[5]: *** [drivers/net/wireless/ath/ath9k/common-spectral.o] Error 1
  scripts/Makefile.build:402: recipe for target
  'drivers/net/wireless/ath/ath9k' failed
  make[4]: *** [drivers/net/wireless/ath/ath9k] Error 2
  scripts/Makefile.build:402: recipe for target 'drivers/net/wireless/ath'
  failed
  make[3]: *** [drivers/net/wireless/ath] Error 2
  scripts/Makefile.build:402: recipe for target 'drivers/net/wireless'
  failed
  make[2]: *** [drivers/net/wireless] Error 2
  scripts/Makefile.build:402: recipe for target 'drivers/net' failed
  make[1]: *** [drivers/net] Error 2
  Makefile:953: recipe for target 'drivers' failed
  make: *** [drivers] Error 2
  make: *** Waiting for unfinished jobs....
    CC      lib/bust_spinlocks.o
  ...

Bisecting the kernel found that the following patch was the cause.

  commit 67dc74f15f147b9f88702de2952d2951e3e000ec
  Author: Oleksij Rempel <linux@...pel-privat.de>
  Date:   Thu Nov 6 08:53:30 2014 +0100
  
      ath9k: move spectral.* to common-spectral.*
      
      and rename exports from ath9k_spectral_* to ath9k_cmn_spectral_*
      
      Signed-off-by: Oleksij Rempel <linux@...pel-privat.de>
      Signed-off-by: John W. Linville <linville@...driver.com>

This patch mostly consists of renaming functions and moving code but
there was a functional change to the Makefile.

common-spectral.h uses CONFIG_ATH9K_DEBUGFS to conditionally provide a
prototype of ath_cmn_process_fft() when set or to define it as a noop
when it is unset.  The Makefile was changed so that CONFIG_ATH9K_DEBUGFS
no longer applied to common-spectral and this will result in two
definitions of ath_cmn_process_fft().

  > --- a/drivers/net/wireless/ath/ath9k/Makefile
  > +++ b/drivers/net/wireless/ath/ath9k/Makefile
  > @@ -16,8 +16,7 @@ ath9k-$(CONFIG_ATH9K_DFS_CERTIFIED) += dfs.o
  >  ath9k-$(CONFIG_ATH9K_TX99) += tx99.o
  >  ath9k-$(CONFIG_ATH9K_WOW) += wow.o
  >
  > -ath9k-$(CONFIG_ATH9K_DEBUGFS) += debug.o \
  > -                                spectral.o
  > +ath9k-$(CONFIG_ATH9K_DEBUGFS) += debug.o
  >
  >  ath9k-$(CONFIG_ATH9K_STATION_STATISTICS) += debug_sta.o
  >
  > @@ -59,7 +58,8 @@ obj-$(CONFIG_ATH9K_COMMON) += ath9k_common.o
  >  ath9k_common-y:=       common.o \
  >                         common-init.o \
  >                         common-beacon.o \
  > -                       common-debug.o
  > +                       common-debug.o \
  > +                       common-spectral.o

Reverting the patch solves one error, but then a new one is produced.

  make
  ...
    MODPOST 185 modules
    CC      arch/x86/boot/edd.o
    VOFFSET arch/x86/boot/voffset.h
  ERROR: "ath9k_cmn_spectral_scan_trigger"
  [drivers/net/wireless/ath/ath9k/ath9k.ko] undefined!
  scripts/Makefile.modpost:90: recipe for target '__modpost' failed
  make[1]: *** [__modpost] Error 1
  ...

This error is caused by the patch before it.

  commit f00a422cc81ef665f5098c0bc43cb0c616e55a9b
  Author: Oleksij Rempel <linux@...pel-privat.de>
  Date:   Thu Nov 6 08:53:29 2014 +0100
  
      ath9k: move ath9k_spectral_scan_ from main.c to spectral.c
      
      Now we should be ready to make this code common.
      
      Signed-off-by: Oleksij Rempel <linux@...pel-privat.de>
      Signed-off-by: John W. Linville <linville@...driver.com>

Since the code was moved from main.c to spectral.c, it is now involved
with CONFIG_ATH9K_DEBUGFS, which causes it to break.

Reverting both the above patches resolves the build errors.

-- 
Jeremiah Mahler
jmmahler@...il.com
http://github.com/jmahler
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html