Search Linux Wireless

Re: [PATCH 2/6] mac80211: remove global tsinfo debugfs variables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2007-06-11 at 10:57 +0800, Zhu Yi wrote:

> When I sent the patch the first time which used sysfs, the comment was
> one value per file (Is this still true for debugfs?).

Hah, yeah, that's the regular rule for sysfs, but debugfs basically has
no rules ;)

> The DLS is easier because it only has one parameter (peer mac address)
> now. I programed it the same way as tspec. So when we find to need more
> parameters for DLS setup, we can add another debugfs file for the new
> parameter instead of combining multiple parameters in one file.

Alright. I don't think it matters much anyway.

> I'd agree I didn't pay a lot of attentions to the debugfs interface
> design since I thought it was used for occasional debug only. Please
> tell me what which do you prefer: one value per file or multiple values
> per file so that we can do one shot parameter passing? So I don't need
> to switch them back and forth.

No, don't bother, too much work for too little gain. Maybe gather some
thoughts on the nl80211 interface instead :) I personally prefer
one-shot parameter passing because it's closer to what we'll be doing
with nl80211 so we can model the interface like nl80211, however, the
interface between the debugfs and the regular mac80211 code could still
be modelled after nl80211 (it could even call some cfg80211 stuff when
that's added)

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux