* [PATCH v2 0/2] dpaa_eth: add ethtool coalesce control @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: leoyang.li, roy.pledge, linuxppc-dev, linux-arm-kernel, linux-kernel, Madalin Bucur Add control of the DPAA portal interrupt coalescing settings from ethtool. changes from v1: added range checking for the QMan APIs Madalin Bucur (2): soc/qman: add return value to interrupt coalesce changing APIs dpaa_eth: add ethtool coalesce control drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ drivers/soc/fsl/qbman/qman.c | 33 +++++++++++---- include/soc/fsl/qman.h | 8 +++- 3 files changed, 81 insertions(+), 9 deletions(-) -- 2.1.0 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 0/2] dpaa_eth: add ethtool coalesce control @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: linux-arm-kernel Add control of the DPAA portal interrupt coalescing settings from ethtool. changes from v1: added range checking for the QMan APIs Madalin Bucur (2): soc/qman: add return value to interrupt coalesce changing APIs dpaa_eth: add ethtool coalesce control drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ drivers/soc/fsl/qbman/qman.c | 33 +++++++++++---- include/soc/fsl/qman.h | 8 +++- 3 files changed, 81 insertions(+), 9 deletions(-) -- 2.1.0 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 0/2] dpaa_eth: add ethtool coalesce control @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: Madalin Bucur, roy.pledge, linux-kernel, leoyang.li, linuxppc-dev, linux-arm-kernel Add control of the DPAA portal interrupt coalescing settings from ethtool. changes from v1: added range checking for the QMan APIs Madalin Bucur (2): soc/qman: add return value to interrupt coalesce changing APIs dpaa_eth: add ethtool coalesce control drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ drivers/soc/fsl/qbman/qman.c | 33 +++++++++++---- include/soc/fsl/qman.h | 8 +++- 3 files changed, 81 insertions(+), 9 deletions(-) -- 2.1.0 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 1/2] soc/qman: add return value to interrupt coalesce changing APIs 2018-11-13 16:29 ` Madalin Bucur (?) @ 2018-11-13 16:29 ` Madalin Bucur -1 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: leoyang.li, roy.pledge, linuxppc-dev, linux-arm-kernel, linux-kernel, Madalin Bucur Check that the values received by the portal interrupt coalesce change APIs are in range. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> Signed-off-by: Roy Pledge <roy.pledge@nxp.com> --- drivers/soc/fsl/qbman/qman.c | 33 ++++++++++++++++++++++++++------- include/soc/fsl/qman.h | 8 ++++++-- 2 files changed, 32 insertions(+), 9 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c b/drivers/soc/fsl/qbman/qman.c index 5ce24718c2fd..5b9de224193c 100644 --- a/drivers/soc/fsl/qbman/qman.c +++ b/drivers/soc/fsl/qbman/qman.c @@ -36,6 +36,8 @@ #define MAX_IRQNAME 16 /* big enough for "QMan portal %d" */ #define QMAN_POLL_LIMIT 32 #define QMAN_PIRQ_DQRR_ITHRESH 12 +#define QMAN_DQRR_IT_MAX 15 +#define QMAN_ITP_MAX 0xFFF #define QMAN_PIRQ_MR_ITHRESH 4 #define QMAN_PIRQ_IPERIOD 100 @@ -727,9 +729,15 @@ static inline void qm_dqrr_vdqcr_set(struct qm_portal *portal, u32 vdqcr) qm_out(portal, QM_REG_DQRR_VDQCR, vdqcr); } -static inline void qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) +static inline int qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) { + + if (ithresh > QMAN_DQRR_IT_MAX) + return -EINVAL; + qm_out(portal, QM_REG_DQRR_ITR, ithresh); + + return 0; } /* --- MR API --- */ @@ -1012,13 +1020,20 @@ static inline void put_affine_portal(void) static struct workqueue_struct *qm_portal_wq; -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) { + int res; + if (!portal) - return; + return -EINVAL; + + res = qm_dqrr_set_ithresh(&portal->p, ithresh); + if (res) + return res; - qm_dqrr_set_ithresh(&portal->p, ithresh); portal->p.dqrr.ithresh = ithresh; + + return 0; } EXPORT_SYMBOL(qman_dqrr_set_ithresh); @@ -1036,10 +1051,14 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod) } EXPORT_SYMBOL(qman_portal_get_iperiod); -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) { - if (portal) - qm_out(&portal->p, QM_REG_ITPR, iperiod); + if (!portal || iperiod > QMAN_ITP_MAX) + return -EINVAL; + + qm_out(&portal->p, QM_REG_ITPR, iperiod); + + return 0; } EXPORT_SYMBOL(qman_portal_set_iperiod); diff --git a/include/soc/fsl/qman.h b/include/soc/fsl/qman.h index 56877660d5ba..5cc7af06c1ba 100644 --- a/include/soc/fsl/qman.h +++ b/include/soc/fsl/qman.h @@ -1205,8 +1205,10 @@ void qman_dqrr_get_ithresh(struct qman_portal *portal, u8 *ithresh); * qman_dqrr_set_ithresh - Set coalesce interrupt threshold * @portal: portal to set the new value on * @ithresh: new threshold value + * + * Returns 0 on success, or a negative error code. */ -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); /** * qman_dqrr_get_iperiod - Get coalesce interrupt period @@ -1219,7 +1221,9 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod); * qman_dqrr_set_iperiod - Set coalesce interrupt period * @portal: portal to set the new value on * @ithresh: new period value + * + * Returns 0 on success, or a negative error code. */ -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); #endif /* __FSL_QMAN_H */ -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v2 1/2] soc/qman: add return value to interrupt coalesce changing APIs @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: linux-arm-kernel Check that the values received by the portal interrupt coalesce change APIs are in range. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> Signed-off-by: Roy Pledge <roy.pledge@nxp.com> --- drivers/soc/fsl/qbman/qman.c | 33 ++++++++++++++++++++++++++------- include/soc/fsl/qman.h | 8 ++++++-- 2 files changed, 32 insertions(+), 9 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c b/drivers/soc/fsl/qbman/qman.c index 5ce24718c2fd..5b9de224193c 100644 --- a/drivers/soc/fsl/qbman/qman.c +++ b/drivers/soc/fsl/qbman/qman.c @@ -36,6 +36,8 @@ #define MAX_IRQNAME 16 /* big enough for "QMan portal %d" */ #define QMAN_POLL_LIMIT 32 #define QMAN_PIRQ_DQRR_ITHRESH 12 +#define QMAN_DQRR_IT_MAX 15 +#define QMAN_ITP_MAX 0xFFF #define QMAN_PIRQ_MR_ITHRESH 4 #define QMAN_PIRQ_IPERIOD 100 @@ -727,9 +729,15 @@ static inline void qm_dqrr_vdqcr_set(struct qm_portal *portal, u32 vdqcr) qm_out(portal, QM_REG_DQRR_VDQCR, vdqcr); } -static inline void qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) +static inline int qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) { + + if (ithresh > QMAN_DQRR_IT_MAX) + return -EINVAL; + qm_out(portal, QM_REG_DQRR_ITR, ithresh); + + return 0; } /* --- MR API --- */ @@ -1012,13 +1020,20 @@ static inline void put_affine_portal(void) static struct workqueue_struct *qm_portal_wq; -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) { + int res; + if (!portal) - return; + return -EINVAL; + + res = qm_dqrr_set_ithresh(&portal->p, ithresh); + if (res) + return res; - qm_dqrr_set_ithresh(&portal->p, ithresh); portal->p.dqrr.ithresh = ithresh; + + return 0; } EXPORT_SYMBOL(qman_dqrr_set_ithresh); @@ -1036,10 +1051,14 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod) } EXPORT_SYMBOL(qman_portal_get_iperiod); -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) { - if (portal) - qm_out(&portal->p, QM_REG_ITPR, iperiod); + if (!portal || iperiod > QMAN_ITP_MAX) + return -EINVAL; + + qm_out(&portal->p, QM_REG_ITPR, iperiod); + + return 0; } EXPORT_SYMBOL(qman_portal_set_iperiod); diff --git a/include/soc/fsl/qman.h b/include/soc/fsl/qman.h index 56877660d5ba..5cc7af06c1ba 100644 --- a/include/soc/fsl/qman.h +++ b/include/soc/fsl/qman.h @@ -1205,8 +1205,10 @@ void qman_dqrr_get_ithresh(struct qman_portal *portal, u8 *ithresh); * qman_dqrr_set_ithresh - Set coalesce interrupt threshold * @portal: portal to set the new value on * @ithresh: new threshold value + * + * Returns 0 on success, or a negative error code. */ -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); /** * qman_dqrr_get_iperiod - Get coalesce interrupt period @@ -1219,7 +1221,9 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod); * qman_dqrr_set_iperiod - Set coalesce interrupt period * @portal: portal to set the new value on * @ithresh: new period value + * + * Returns 0 on success, or a negative error code. */ -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); #endif /* __FSL_QMAN_H */ -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v2 1/2] soc/qman: add return value to interrupt coalesce changing APIs @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: Madalin Bucur, roy.pledge, linux-kernel, leoyang.li, linuxppc-dev, linux-arm-kernel Check that the values received by the portal interrupt coalesce change APIs are in range. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> Signed-off-by: Roy Pledge <roy.pledge@nxp.com> --- drivers/soc/fsl/qbman/qman.c | 33 ++++++++++++++++++++++++++------- include/soc/fsl/qman.h | 8 ++++++-- 2 files changed, 32 insertions(+), 9 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c b/drivers/soc/fsl/qbman/qman.c index 5ce24718c2fd..5b9de224193c 100644 --- a/drivers/soc/fsl/qbman/qman.c +++ b/drivers/soc/fsl/qbman/qman.c @@ -36,6 +36,8 @@ #define MAX_IRQNAME 16 /* big enough for "QMan portal %d" */ #define QMAN_POLL_LIMIT 32 #define QMAN_PIRQ_DQRR_ITHRESH 12 +#define QMAN_DQRR_IT_MAX 15 +#define QMAN_ITP_MAX 0xFFF #define QMAN_PIRQ_MR_ITHRESH 4 #define QMAN_PIRQ_IPERIOD 100 @@ -727,9 +729,15 @@ static inline void qm_dqrr_vdqcr_set(struct qm_portal *portal, u32 vdqcr) qm_out(portal, QM_REG_DQRR_VDQCR, vdqcr); } -static inline void qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) +static inline int qm_dqrr_set_ithresh(struct qm_portal *portal, u8 ithresh) { + + if (ithresh > QMAN_DQRR_IT_MAX) + return -EINVAL; + qm_out(portal, QM_REG_DQRR_ITR, ithresh); + + return 0; } /* --- MR API --- */ @@ -1012,13 +1020,20 @@ static inline void put_affine_portal(void) static struct workqueue_struct *qm_portal_wq; -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh) { + int res; + if (!portal) - return; + return -EINVAL; + + res = qm_dqrr_set_ithresh(&portal->p, ithresh); + if (res) + return res; - qm_dqrr_set_ithresh(&portal->p, ithresh); portal->p.dqrr.ithresh = ithresh; + + return 0; } EXPORT_SYMBOL(qman_dqrr_set_ithresh); @@ -1036,10 +1051,14 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod) } EXPORT_SYMBOL(qman_portal_get_iperiod); -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod) { - if (portal) - qm_out(&portal->p, QM_REG_ITPR, iperiod); + if (!portal || iperiod > QMAN_ITP_MAX) + return -EINVAL; + + qm_out(&portal->p, QM_REG_ITPR, iperiod); + + return 0; } EXPORT_SYMBOL(qman_portal_set_iperiod); diff --git a/include/soc/fsl/qman.h b/include/soc/fsl/qman.h index 56877660d5ba..5cc7af06c1ba 100644 --- a/include/soc/fsl/qman.h +++ b/include/soc/fsl/qman.h @@ -1205,8 +1205,10 @@ void qman_dqrr_get_ithresh(struct qman_portal *portal, u8 *ithresh); * qman_dqrr_set_ithresh - Set coalesce interrupt threshold * @portal: portal to set the new value on * @ithresh: new threshold value + * + * Returns 0 on success, or a negative error code. */ -void qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); +int qman_dqrr_set_ithresh(struct qman_portal *portal, u8 ithresh); /** * qman_dqrr_get_iperiod - Get coalesce interrupt period @@ -1219,7 +1221,9 @@ void qman_portal_get_iperiod(struct qman_portal *portal, u32 *iperiod); * qman_dqrr_set_iperiod - Set coalesce interrupt period * @portal: portal to set the new value on * @ithresh: new period value + * + * Returns 0 on success, or a negative error code. */ -void qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); +int qman_portal_set_iperiod(struct qman_portal *portal, u32 iperiod); #endif /* __FSL_QMAN_H */ -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control 2018-11-13 16:29 ` Madalin Bucur (?) @ 2018-11-13 16:29 ` Madalin Bucur -1 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: leoyang.li, roy.pledge, linuxppc-dev, linux-arm-kernel, linux-kernel, Madalin Bucur Allow ethtool control of the DPAA QMan portal interrupt coalescing settings. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> --- drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ 1 file changed, 49 insertions(+) diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c index 13d6e2272ece..4df366b05976 100644 --- a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c @@ -529,6 +529,53 @@ static int dpaa_get_ts_info(struct net_device *net_dev, return 0; } +static int dpaa_get_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + struct qman_portal *portal; + u32 period; + u8 thresh; + + portal = qman_get_affine_portal(smp_processor_id()); + qman_portal_get_iperiod(portal, &period); + qman_dqrr_get_ithresh(portal, &thresh); + + c->rx_coalesce_usecs = period; + c->rx_max_coalesced_frames = thresh; + c->use_adaptive_rx_coalesce = false; + + return 0; +} + +static int dpaa_set_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + const cpumask_t *cpus = qman_affine_cpus(); + struct qman_portal *portal; + u32 period; + u8 thresh; + int cpu; + int res; + + if (c->use_adaptive_rx_coalesce) + return -EINVAL; + + period = c->rx_coalesce_usecs; + thresh = c->rx_max_coalesced_frames; + + for_each_cpu(cpu, cpus) { + portal = qman_get_affine_portal(cpu); + res = qman_portal_set_iperiod(portal, period); + if (res) + return res; + res = qman_dqrr_set_ithresh(portal, thresh); + if (res) + return res; + } + + return 0; +} + const struct ethtool_ops dpaa_ethtool_ops = { .get_drvinfo = dpaa_get_drvinfo, .get_msglevel = dpaa_get_msglevel, @@ -545,4 +592,6 @@ const struct ethtool_ops dpaa_ethtool_ops = { .get_rxnfc = dpaa_get_rxnfc, .set_rxnfc = dpaa_set_rxnfc, .get_ts_info = dpaa_get_ts_info, + .get_coalesce = dpaa_get_coalesce, + .set_coalesce = dpaa_set_coalesce, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: linux-arm-kernel Allow ethtool control of the DPAA QMan portal interrupt coalescing settings. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> --- drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ 1 file changed, 49 insertions(+) diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c index 13d6e2272ece..4df366b05976 100644 --- a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c @@ -529,6 +529,53 @@ static int dpaa_get_ts_info(struct net_device *net_dev, return 0; } +static int dpaa_get_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + struct qman_portal *portal; + u32 period; + u8 thresh; + + portal = qman_get_affine_portal(smp_processor_id()); + qman_portal_get_iperiod(portal, &period); + qman_dqrr_get_ithresh(portal, &thresh); + + c->rx_coalesce_usecs = period; + c->rx_max_coalesced_frames = thresh; + c->use_adaptive_rx_coalesce = false; + + return 0; +} + +static int dpaa_set_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + const cpumask_t *cpus = qman_affine_cpus(); + struct qman_portal *portal; + u32 period; + u8 thresh; + int cpu; + int res; + + if (c->use_adaptive_rx_coalesce) + return -EINVAL; + + period = c->rx_coalesce_usecs; + thresh = c->rx_max_coalesced_frames; + + for_each_cpu(cpu, cpus) { + portal = qman_get_affine_portal(cpu); + res = qman_portal_set_iperiod(portal, period); + if (res) + return res; + res = qman_dqrr_set_ithresh(portal, thresh); + if (res) + return res; + } + + return 0; +} + const struct ethtool_ops dpaa_ethtool_ops = { .get_drvinfo = dpaa_get_drvinfo, .get_msglevel = dpaa_get_msglevel, @@ -545,4 +592,6 @@ const struct ethtool_ops dpaa_ethtool_ops = { .get_rxnfc = dpaa_get_rxnfc, .set_rxnfc = dpaa_set_rxnfc, .get_ts_info = dpaa_get_ts_info, + .get_coalesce = dpaa_get_coalesce, + .set_coalesce = dpaa_set_coalesce, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-13 16:29 ` Madalin Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin Bucur @ 2018-11-13 16:29 UTC (permalink / raw) To: davem, netdev Cc: Madalin Bucur, roy.pledge, linux-kernel, leoyang.li, linuxppc-dev, linux-arm-kernel Allow ethtool control of the DPAA QMan portal interrupt coalescing settings. Signed-off-by: Madalin Bucur <madalin.bucur@nxp.com> --- drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c | 49 ++++++++++++++++++++++ 1 file changed, 49 insertions(+) diff --git a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c index 13d6e2272ece..4df366b05976 100644 --- a/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c +++ b/drivers/net/ethernet/freescale/dpaa/dpaa_ethtool.c @@ -529,6 +529,53 @@ static int dpaa_get_ts_info(struct net_device *net_dev, return 0; } +static int dpaa_get_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + struct qman_portal *portal; + u32 period; + u8 thresh; + + portal = qman_get_affine_portal(smp_processor_id()); + qman_portal_get_iperiod(portal, &period); + qman_dqrr_get_ithresh(portal, &thresh); + + c->rx_coalesce_usecs = period; + c->rx_max_coalesced_frames = thresh; + c->use_adaptive_rx_coalesce = false; + + return 0; +} + +static int dpaa_set_coalesce(struct net_device *dev, + struct ethtool_coalesce *c) +{ + const cpumask_t *cpus = qman_affine_cpus(); + struct qman_portal *portal; + u32 period; + u8 thresh; + int cpu; + int res; + + if (c->use_adaptive_rx_coalesce) + return -EINVAL; + + period = c->rx_coalesce_usecs; + thresh = c->rx_max_coalesced_frames; + + for_each_cpu(cpu, cpus) { + portal = qman_get_affine_portal(cpu); + res = qman_portal_set_iperiod(portal, period); + if (res) + return res; + res = qman_dqrr_set_ithresh(portal, thresh); + if (res) + return res; + } + + return 0; +} + const struct ethtool_ops dpaa_ethtool_ops = { .get_drvinfo = dpaa_get_drvinfo, .get_msglevel = dpaa_get_msglevel, @@ -545,4 +592,6 @@ const struct ethtool_ops dpaa_ethtool_ops = { .get_rxnfc = dpaa_get_rxnfc, .set_rxnfc = dpaa_set_rxnfc, .get_ts_info = dpaa_get_ts_info, + .get_coalesce = dpaa_get_coalesce, + .set_coalesce = dpaa_set_coalesce, }; -- 2.1.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control 2018-11-13 16:29 ` Madalin Bucur (?) @ 2018-11-17 3:42 ` David Miller -1 siblings, 0 replies; 16+ messages in thread From: David Miller @ 2018-11-17 3:42 UTC (permalink / raw) To: madalin.bucur Cc: netdev, leoyang.li, roy.pledge, linuxppc-dev, linux-arm-kernel, linux-kernel From: Madalin Bucur <madalin.bucur@nxp.com> Date: Tue, 13 Nov 2018 18:29:51 +0200 > + for_each_cpu(cpu, cpus) { > + portal = qman_get_affine_portal(cpu); > + res = qman_portal_set_iperiod(portal, period); > + if (res) > + return res; > + res = qman_dqrr_set_ithresh(portal, thresh); > + if (res) > + return res; Nope, you can't do it like this. If any intermediate change fails, you have to unwind all of the changes made up until that point. Which means you'll have to store the previous setting somewhere and reinstall those saved values in the error path. ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-17 3:42 ` David Miller 0 siblings, 0 replies; 16+ messages in thread From: David Miller @ 2018-11-17 3:42 UTC (permalink / raw) To: linux-arm-kernel From: Madalin Bucur <madalin.bucur@nxp.com> Date: Tue, 13 Nov 2018 18:29:51 +0200 > + for_each_cpu(cpu, cpus) { > + portal = qman_get_affine_portal(cpu); > + res = qman_portal_set_iperiod(portal, period); > + if (res) > + return res; > + res = qman_dqrr_set_ithresh(portal, thresh); > + if (res) > + return res; Nope, you can't do it like this. If any intermediate change fails, you have to unwind all of the changes made up until that point. Which means you'll have to store the previous setting somewhere and reinstall those saved values in the error path. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-17 3:42 ` David Miller 0 siblings, 0 replies; 16+ messages in thread From: David Miller @ 2018-11-17 3:42 UTC (permalink / raw) To: madalin.bucur Cc: netdev, roy.pledge, linux-kernel, leoyang.li, linuxppc-dev, linux-arm-kernel From: Madalin Bucur <madalin.bucur@nxp.com> Date: Tue, 13 Nov 2018 18:29:51 +0200 > + for_each_cpu(cpu, cpus) { > + portal = qman_get_affine_portal(cpu); > + res = qman_portal_set_iperiod(portal, period); > + if (res) > + return res; > + res = qman_dqrr_set_ithresh(portal, thresh); > + if (res) > + return res; Nope, you can't do it like this. If any intermediate change fails, you have to unwind all of the changes made up until that point. Which means you'll have to store the previous setting somewhere and reinstall those saved values in the error path. ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control 2018-11-17 3:42 ` David Miller (?) (?) @ 2018-11-19 7:11 ` Madalin-cristian Bucur -1 siblings, 0 replies; 16+ messages in thread From: Madalin-cristian Bucur @ 2018-11-19 7:11 UTC (permalink / raw) To: David Miller Cc: netdev, Leo Li, Roy Pledge, linuxppc-dev, linux-arm-kernel, linux-kernel > -----Original Message----- > From: David Miller <davem@davemloft.net> > Sent: Saturday, November 17, 2018 5:42 AM > To: Madalin-cristian Bucur <madalin.bucur@nxp.com> > Subject: Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control > > From: Madalin Bucur <madalin.bucur@nxp.com> > Date: Tue, 13 Nov 2018 18:29:51 +0200 > > > + for_each_cpu(cpu, cpus) { > > + portal = qman_get_affine_portal(cpu); > > + res = qman_portal_set_iperiod(portal, period); > > + if (res) > > + return res; > > + res = qman_dqrr_set_ithresh(portal, thresh); > > + if (res) > > + return res; > > Nope, you can't do it like this. > > If any intermediate change fails, you have to unwind all of the > changes made up until that point. > > Which means you'll have to store the previous setting somewhere > and reinstall those saved values in the error path. Thank you, I'll come back with a v3. Madalin ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-19 7:11 ` Madalin-cristian Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin-cristian Bucur @ 2018-11-19 7:11 UTC (permalink / raw) To: linux-arm-kernel > -----Original Message----- > From: David Miller <davem@davemloft.net> > Sent: Saturday, November 17, 2018 5:42 AM > To: Madalin-cristian Bucur <madalin.bucur@nxp.com> > Subject: Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control > > From: Madalin Bucur <madalin.bucur@nxp.com> > Date: Tue, 13 Nov 2018 18:29:51 +0200 > > > + for_each_cpu(cpu, cpus) { > > + portal = qman_get_affine_portal(cpu); > > + res = qman_portal_set_iperiod(portal, period); > > + if (res) > > + return res; > > + res = qman_dqrr_set_ithresh(portal, thresh); > > + if (res) > > + return res; > > Nope, you can't do it like this. > > If any intermediate change fails, you have to unwind all of the > changes made up until that point. > > Which means you'll have to store the previous setting somewhere > and reinstall those saved values in the error path. Thank you, I'll come back with a v3. Madalin ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-19 7:11 ` Madalin-cristian Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin-cristian Bucur @ 2018-11-19 7:11 UTC (permalink / raw) To: David Miller Cc: netdev, Roy Pledge, linux-kernel, Leo Li, linuxppc-dev, linux-arm-kernel > -----Original Message----- > From: David Miller <davem@davemloft.net> > Sent: Saturday, November 17, 2018 5:42 AM > To: Madalin-cristian Bucur <madalin.bucur@nxp.com> > Subject: Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control > > From: Madalin Bucur <madalin.bucur@nxp.com> > Date: Tue, 13 Nov 2018 18:29:51 +0200 > > > + for_each_cpu(cpu, cpus) { > > + portal = qman_get_affine_portal(cpu); > > + res = qman_portal_set_iperiod(portal, period); > > + if (res) > > + return res; > > + res = qman_dqrr_set_ithresh(portal, thresh); > > + if (res) > > + return res; > > Nope, you can't do it like this. > > If any intermediate change fails, you have to unwind all of the > changes made up until that point. > > Which means you'll have to store the previous setting somewhere > and reinstall those saved values in the error path. Thank you, I'll come back with a v3. Madalin ^ permalink raw reply [flat|nested] 16+ messages in thread
* RE: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control @ 2018-11-19 7:11 ` Madalin-cristian Bucur 0 siblings, 0 replies; 16+ messages in thread From: Madalin-cristian Bucur @ 2018-11-19 7:11 UTC (permalink / raw) To: David Miller Cc: netdev, Leo Li, Roy Pledge, linuxppc-dev, linux-arm-kernel, linux-kernel > -----Original Message----- > From: David Miller <davem@davemloft.net> > Sent: Saturday, November 17, 2018 5:42 AM > To: Madalin-cristian Bucur <madalin.bucur@nxp.com> > Subject: Re: [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control > > From: Madalin Bucur <madalin.bucur@nxp.com> > Date: Tue, 13 Nov 2018 18:29:51 +0200 > > > + for_each_cpu(cpu, cpus) { > > + portal = qman_get_affine_portal(cpu); > > + res = qman_portal_set_iperiod(portal, period); > > + if (res) > > + return res; > > + res = qman_dqrr_set_ithresh(portal, thresh); > > + if (res) > > + return res; > > Nope, you can't do it like this. > > If any intermediate change fails, you have to unwind all of the > changes made up until that point. > > Which means you'll have to store the previous setting somewhere > and reinstall those saved values in the error path. Thank you, I'll come back with a v3. Madalin ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2018-11-19 17:33 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-11-13 16:29 [PATCH v2 0/2] dpaa_eth: add ethtool coalesce control Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-13 16:29 ` [PATCH v2 1/2] soc/qman: add return value to interrupt coalesce changing APIs Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-13 16:29 ` [PATCH v2 2/2] dpaa_eth: add ethtool coalesce control Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-13 16:29 ` Madalin Bucur 2018-11-17 3:42 ` David Miller 2018-11-17 3:42 ` David Miller 2018-11-17 3:42 ` David Miller 2018-11-19 7:11 ` Madalin-cristian Bucur 2018-11-19 7:11 ` Madalin-cristian Bucur 2018-11-19 7:11 ` Madalin-cristian Bucur 2018-11-19 7:11 ` Madalin-cristian Bucur
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.