]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
iwlwifi: mvm: remove multiple debugfs entries
authorMordechay Goodstein <mordechay.goodstein@intel.com>
Wed, 1 May 2019 17:48:12 +0000 (20:48 +0300)
committerLuca Coelho <luciano.coelho@intel.com>
Sat, 29 Jun 2019 07:09:37 +0000 (10:09 +0300)
Now that we have per station control over amsdu size no need for
multiple entries, especially that the old one is misleading due to not
setting it for all protocols as a limit.

Signed-off-by: Mordechay Goodstein <mordechay.goodstein@intel.com>
Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
drivers/net/wireless/intel/iwlwifi/mvm/debugfs.c
drivers/net/wireless/intel/iwlwifi/mvm/mvm.h
drivers/net/wireless/intel/iwlwifi/mvm/tx.c

index 8483640d56928fad335b62b81ac6e1e5f2753f49..0c188a82cfc1e681cad19384dbaa0ca96e968a4d 100644 (file)
@@ -1396,24 +1396,6 @@ static ssize_t iwl_dbgfs_fw_dbg_collect_write(struct iwl_mvm *mvm,
        return count;
 }
 
-static ssize_t iwl_dbgfs_max_amsdu_len_write(struct iwl_mvm *mvm,
-                                            char *buf, size_t count,
-                                            loff_t *ppos)
-{
-       unsigned int max_amsdu_len;
-       int ret;
-
-       ret = kstrtouint(buf, 0, &max_amsdu_len);
-       if (ret)
-               return ret;
-
-       if (max_amsdu_len > IEEE80211_MAX_MPDU_LEN_VHT_11454)
-               return -EINVAL;
-       mvm->max_amsdu_len = max_amsdu_len;
-
-       return count;
-}
-
 #define ADD_TEXT(...) pos += scnprintf(buf + pos, bufsz - pos, __VA_ARGS__)
 #ifdef CONFIG_IWLWIFI_BCAST_FILTERING
 static ssize_t iwl_dbgfs_bcast_filters_read(struct file *file,
@@ -1913,7 +1895,6 @@ MVM_DEBUGFS_READ_WRITE_FILE_OPS(scan_ant_rxchain, 8);
 MVM_DEBUGFS_READ_WRITE_FILE_OPS(d0i3_refs, 8);
 MVM_DEBUGFS_READ_WRITE_FILE_OPS(fw_dbg_conf, 8);
 MVM_DEBUGFS_WRITE_FILE_OPS(fw_dbg_collect, 64);
-MVM_DEBUGFS_WRITE_FILE_OPS(max_amsdu_len, 8);
 MVM_DEBUGFS_WRITE_FILE_OPS(indirection_tbl,
                           (IWL_RSS_INDIRECTION_TABLE_SIZE * 2));
 MVM_DEBUGFS_WRITE_FILE_OPS(inject_packet, 512);
@@ -2113,7 +2094,6 @@ void iwl_mvm_dbgfs_register(struct iwl_mvm *mvm, struct dentry *dbgfs_dir)
        MVM_DEBUGFS_ADD_FILE(d0i3_refs, mvm->debugfs_dir, 0600);
        MVM_DEBUGFS_ADD_FILE(fw_dbg_conf, mvm->debugfs_dir, 0600);
        MVM_DEBUGFS_ADD_FILE(fw_dbg_collect, mvm->debugfs_dir, 0200);
-       MVM_DEBUGFS_ADD_FILE(max_amsdu_len, mvm->debugfs_dir, 0200);
        MVM_DEBUGFS_ADD_FILE(send_echo_cmd, mvm->debugfs_dir, 0200);
        MVM_DEBUGFS_ADD_FILE(indirection_tbl, mvm->debugfs_dir, 0200);
        MVM_DEBUGFS_ADD_FILE(inject_packet, mvm->debugfs_dir, 0200);
index 02efcf2189c489ad92a0ff1acaf8de436d3cc758..4dbbebd66d27c9c808ac69c39ed5cad7fadaa59b 100644 (file)
@@ -1107,7 +1107,6 @@ struct iwl_mvm {
        u8 ps_disabled; /* u8 instead of bool to ease debugfs_create_* usage */
        /* Indicate if 32Khz external clock is valid */
        u32 ext_clock_valid;
-       unsigned int max_amsdu_len; /* used for debugfs only */
 
        struct ieee80211_vif __rcu *csa_vif;
        struct ieee80211_vif __rcu *csa_tx_blocked_vif;
index ad4760307726fedcdd410cd5b108b1eddf26b8ee..16f7458e2e81dbdaa05578bac650c67bef57ec38 100644 (file)
@@ -893,18 +893,15 @@ static int iwl_mvm_tx_tso(struct iwl_mvm *mvm, struct sk_buff *skb,
        unsigned int mss = skb_shinfo(skb)->gso_size;
        unsigned int num_subframes, tcp_payload_len, subf_len, max_amsdu_len;
        u16 snap_ip_tcp, pad;
-       unsigned int dbg_max_amsdu_len;
        netdev_features_t netdev_flags = NETIF_F_CSUM_MASK | NETIF_F_SG;
        u8 tid;
 
        snap_ip_tcp = 8 + skb_transport_header(skb) - skb_network_header(skb) +
                tcp_hdrlen(skb);
 
-       dbg_max_amsdu_len = READ_ONCE(mvm->max_amsdu_len);
-
        if (!mvmsta->max_amsdu_len ||
            !ieee80211_is_data_qos(hdr->frame_control) ||
-           (!mvmsta->amsdu_enabled && !dbg_max_amsdu_len))
+           !mvmsta->amsdu_enabled)
                return iwl_mvm_tx_tso_segment(skb, 1, netdev_flags, mpdus_skb);
 
        /*
@@ -936,10 +933,6 @@ static int iwl_mvm_tx_tso(struct iwl_mvm *mvm, struct sk_buff *skb,
 
        max_amsdu_len = iwl_mvm_max_amsdu_size(mvm, sta, tid);
 
-       if (unlikely(dbg_max_amsdu_len))
-               max_amsdu_len = min_t(unsigned int, max_amsdu_len,
-                                     dbg_max_amsdu_len);
-
        /*
         * Limit A-MSDU in A-MPDU to 4095 bytes when VHT is not
         * supported. This is a spec requirement (IEEE 802.11-2015