Nick Kossifidis wrote:
> * Add debug code for displaying EEPROM data. Tested on various chip combinations, on x86 and IXP43x (armv5te). It has many > 80cols warnings but i
> don't think it'll be more readable if i start breaking lines, it's already messy, after all it's debug code. This code also replaces ath_info tool.
I think breaking lines in this case would be rather contraproductive.
> Signed-Off-by: Nick Kossifidis <mickflemm@xxxxxxxxx>
>
> ---
> diff --git a/drivers/net/wireless/ath5k/debug.c b/drivers/net/wireless/ath5k/debug.c
> index ccaeb5c..f0ef5b3 100644
> --- a/drivers/net/wireless/ath5k/debug.c
> +++ b/drivers/net/wireless/ath5k/debug.c
> @@ -324,6 +324,858 @@ static const struct file_operations fops_reset = {
> .owner = THIS_MODULE,
> };
>
> +/* debugfs: EEPROM stuff */
> +
> +/* EEPROM Header (common) */
> +static ssize_t read_file_eeprom_header(struct file *file, char __user *user_buf,
> + size_t count, loff_t *ppos)
> +{
> +
> + struct ath5k_softc *sc = file->private_data;
> + struct ath5k_hw *ah = sc->ah;
> + struct ath5k_eeprom_info ee = ah->ah_capabilities.cap_eeprom;
> + char buf[2000];
Please don't use that much memory from stack. 2k is way too much. Note that
you use yet another bunch of stack (next 3k here on 64-bit) by
ath5k_eeprom_info and 32-bit x86 can still be configured with 4k stacks.
Convert both of them to dynamically allocated buffers.
> +/* EEPROM Header (per mode) */
> +static unsigned int dump_calinfo_for_mode(struct ath5k_hw *ah, int mode,
> + char __user *user_buf, size_t count, loff_t *ppos)
> +{
> + struct ath5k_eeprom_info ee = ah->ah_capabilities.cap_eeprom;
> + unsigned int len = 0;
> + char buf[2000];
dtto
> + int i;
...
> + len = simple_read_from_buffer(user_buf, count, ppos, buf, len);
simple_read_from_buffer can fail and returns negative errno in that case,
change dump_calinfo_for_mode return type appropriately.
> + return len;
> +}
...
> +/* EEPROM channel power calibration info (per mode) */
> +static unsigned int dump_pcalinfo_for_mode_rf5111(struct ath5k_hw *ah, int mode,
> + char __user *user_buf, size_t count, loff_t *ppos)
return ssize_t for the same reason
> +{
> + struct ath5k_eeprom_info ee = ah->ah_capabilities.cap_eeprom;
> + struct ath5k_chan_pcal_info *gen_chan_info;
> + struct ath5k_chan_pcal_info_rf5111 *chan_pcal_info;
> + unsigned int len = 0;
> + char buf[4000];
Here you are almost out of stack even on 8k stacks ;).
> +static unsigned int dump_pcalinfo_for_mode_rf5112(struct ath5k_hw *ah, int mode,
> + char __user *user_buf, size_t count, loff_t *ppos)
dtto and further too
> +{
> + struct ath5k_eeprom_info ee = ah->ah_capabilities.cap_eeprom;
> + struct ath5k_chan_pcal_info *gen_chan_info;
> + struct ath5k_chan_pcal_info_rf5112 *chan_pcal_info;
> + unsigned int len = 0;
> + char buf[4000];
dtto and further too
Happy new year!
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
