Some wireless cards are not reported as being in the WIRELESS category by udev_resource

Bug #855382 reported by Brendan Donegan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
High
Sylvain Pineau

Bug Description

At least on this Samsung N310, I've noticed that the tests in wireless.txt.in have been skipped. Checking their dependencies shows that they depend on a device of the WIRELESS category being detected by udev_resource. If I run udev_resource manually and do a grep for WIRELESS I get nothing. It turns out the Atheros network card is reporting itself as an Ethernet adapter:

02:00.0 Ethernet controller: Atheros Communications Inc. AR242x / AR542x Wireless Network Adapter (PCI-Express) (rev 01)
 Subsystem: Askey Computer Corp. Device 7130
 Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx-
 Status: Cap+ 66MHz- UDF- FastB2B- ParErr- DEVSEL=fast >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
 Latency: 0, Cache Line Size: 64 bytes
 Interrupt: pin A routed to IRQ 16
 Region 0: Memory at f0100000 (64-bit, non-prefetchable) [size=64K]
 Capabilities: <access denied>
 Kernel driver in use: ath5k
 Kernel modules: ath5k

I don't know why this is. It might be that udev_resource just has to be less strict on what it classes as a WIRELESS device.

Related branches

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

There might be no way to get around this other than being less strict with the requires in the wireless tests (i.e. require NETWORK instead of WIRELESS specifically). The PCI_CLASS given by udev is 20000 for the Atheros cards in both Samsung netbooks I've tried this on and 28000 anywhere else. If the devices are reporting the wrong PCI class is there anything more we can do?

Changed in checkbox:
status: New → Confirmed
status: Confirmed → New
Revision history for this message
Marc Tardif (cr3) wrote :

Please attach the output of these two commands:

# udevadm info --export-db

# for i in `udevadm info --export-db | sed -n 's/^P: //p'`; do
  echo "P: $i"
  udevadm info --attribute-walk --path=/sys$i 2>/dev/null | sed -n 's/ ATTR{\(.*\)}=="\(.*\)"/A: \1=\2/p'
  echo
done

Changed in checkbox:
assignee: nobody → Marc Tardif (cr3)
status: New → Incomplete
Ara Pulido (ara)
Changed in checkbox:
milestone: none → 0.13
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Here are the two logs requested

Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

and the other one

Changed in checkbox:
status: Incomplete → New
status: New → Confirmed
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

I've set the important of this to High because I believe it is probably impacting our certification efforts right now, but we have little way of telling.

Changed in checkbox:
importance: Undecided → High
Ara Pulido (ara)
Changed in checkbox:
assignee: Marc Tardif (cr3) → nobody
Changed in checkbox:
assignee: nobody → Sylvain Pineau (sylvain-pineau)
status: Confirmed → In Progress
Zygmunt Krynicki (zyga)
Changed in checkbox:
status: In Progress → Fix Committed
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.