* [PATCH 1/2] mlx5: only register devlink when ethernet is available @ 2016-06-15 15:27 ` Arnd Bergmann 0 siblings, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2016-06-15 15:27 UTC (permalink / raw) To: Matan Barak, Leon Romanovsky Cc: Arnd Bergmann, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, netdev-u79uwXL29TY76Z2rM5mHXA, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA We get a build error with the mlx5 driver when the ethernet support (CONFIG_MLX5_CORE_EN) is disabled: drivers/net/ethernet/mellanox/mlx5/core/main.c:1320:22: error: 'mlx5_devlink_eswitch_mode_set' undeclared here (not in a function) drivers/net/ethernet/mellanox/mlx5/core/main.c:1321:22: error: 'mlx5_devlink_eswitch_mode_get' undeclared here (not in a function) drivers/net/built-in.o:(.rodata+0x25a68): undefined reference to `mlx5_devlink_eswitch_mode_get' drivers/net/built-in.o:(.rodata+0x25a6c): undefined reference to `mlx5_devlink_eswitch_mode_set' There are actually two problems here, but they are closely related, so I'm addressing them both: - The header is included under an #ifdef, which is usually a bad idea as it hides the function declarations, so we fail to compile even if we don't actually use the functions in the end. - The references to the functions are kept in the object file because we don't check whether they are built-in or not. As we don't want to add any useless #ifdef here, this uses an IS_ENABLED() check to drop the mlx5_devlink_ops structure when we don't need it, and to skip the register/unregister step. Signed-off-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> Fixes: f7856daf57b9 ("net/mlx5: Add devlink interface") --- drivers/net/ethernet/mellanox/mlx5/core/main.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c b/drivers/net/ethernet/mellanox/mlx5/core/main.c index dc568096b87c..d238e312b123 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/main.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c @@ -54,9 +54,7 @@ #include <net/devlink.h> #include "mlx5_core.h" #include "fs_core.h" -#ifdef CONFIG_MLX5_CORE_EN #include "eswitch.h" -#endif MODULE_AUTHOR("Eli Cohen <eli-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>"); MODULE_DESCRIPTION("Mellanox Connect-IB, ConnectX-4 core driver"); @@ -1329,7 +1327,8 @@ static int init_one(struct pci_dev *pdev, struct mlx5_priv *priv; int err; - devlink = devlink_alloc(&mlx5_devlink_ops, sizeof(*dev)); + devlink = devlink_alloc(IS_ENABLED(CONFIG_MLX5_CORE_EN) ? + &mlx5_devlink_ops : NULL, sizeof(*dev)); if (!devlink) { dev_err(&pdev->dev, "kzalloc failed\n"); return -ENOMEM; @@ -1372,7 +1371,8 @@ static int init_one(struct pci_dev *pdev, goto clean_health; } - err = devlink_register(devlink, &pdev->dev); + if (IS_ENABLED(CONFIG_MLX5_CORE_EN)) + err = devlink_register(devlink, &pdev->dev); if (err) goto clean_load; @@ -1397,7 +1397,8 @@ static void remove_one(struct pci_dev *pdev) struct devlink *devlink = priv_to_devlink(dev); struct mlx5_priv *priv = &dev->priv; - devlink_unregister(devlink); + if (IS_ENABLED(CONFIG_MLX5_CORE_EN)) + devlink_unregister(devlink); if (mlx5_unload_one(dev, priv)) { dev_err(&dev->pdev->dev, "mlx5_unload_one failed\n"); mlx5_health_cleanup(dev); -- 2.9.0 -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 1/2] mlx5: only register devlink when ethernet is available @ 2016-06-15 15:27 ` Arnd Bergmann 0 siblings, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2016-06-15 15:27 UTC (permalink / raw) To: Matan Barak, Leon Romanovsky Cc: Arnd Bergmann, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, netdev, linux-rdma, linux-kernel We get a build error with the mlx5 driver when the ethernet support (CONFIG_MLX5_CORE_EN) is disabled: drivers/net/ethernet/mellanox/mlx5/core/main.c:1320:22: error: 'mlx5_devlink_eswitch_mode_set' undeclared here (not in a function) drivers/net/ethernet/mellanox/mlx5/core/main.c:1321:22: error: 'mlx5_devlink_eswitch_mode_get' undeclared here (not in a function) drivers/net/built-in.o:(.rodata+0x25a68): undefined reference to `mlx5_devlink_eswitch_mode_get' drivers/net/built-in.o:(.rodata+0x25a6c): undefined reference to `mlx5_devlink_eswitch_mode_set' There are actually two problems here, but they are closely related, so I'm addressing them both: - The header is included under an #ifdef, which is usually a bad idea as it hides the function declarations, so we fail to compile even if we don't actually use the functions in the end. - The references to the functions are kept in the object file because we don't check whether they are built-in or not. As we don't want to add any useless #ifdef here, this uses an IS_ENABLED() check to drop the mlx5_devlink_ops structure when we don't need it, and to skip the register/unregister step. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Fixes: f7856daf57b9 ("net/mlx5: Add devlink interface") --- drivers/net/ethernet/mellanox/mlx5/core/main.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/main.c b/drivers/net/ethernet/mellanox/mlx5/core/main.c index dc568096b87c..d238e312b123 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/main.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/main.c @@ -54,9 +54,7 @@ #include <net/devlink.h> #include "mlx5_core.h" #include "fs_core.h" -#ifdef CONFIG_MLX5_CORE_EN #include "eswitch.h" -#endif MODULE_AUTHOR("Eli Cohen <eli@mellanox.com>"); MODULE_DESCRIPTION("Mellanox Connect-IB, ConnectX-4 core driver"); @@ -1329,7 +1327,8 @@ static int init_one(struct pci_dev *pdev, struct mlx5_priv *priv; int err; - devlink = devlink_alloc(&mlx5_devlink_ops, sizeof(*dev)); + devlink = devlink_alloc(IS_ENABLED(CONFIG_MLX5_CORE_EN) ? + &mlx5_devlink_ops : NULL, sizeof(*dev)); if (!devlink) { dev_err(&pdev->dev, "kzalloc failed\n"); return -ENOMEM; @@ -1372,7 +1371,8 @@ static int init_one(struct pci_dev *pdev, goto clean_health; } - err = devlink_register(devlink, &pdev->dev); + if (IS_ENABLED(CONFIG_MLX5_CORE_EN)) + err = devlink_register(devlink, &pdev->dev); if (err) goto clean_load; @@ -1397,7 +1397,8 @@ static void remove_one(struct pci_dev *pdev) struct devlink *devlink = priv_to_devlink(dev); struct mlx5_priv *priv = &dev->priv; - devlink_unregister(devlink); + if (IS_ENABLED(CONFIG_MLX5_CORE_EN)) + devlink_unregister(devlink); if (mlx5_unload_one(dev, priv)) { dev_err(&dev->pdev->dev, "mlx5_unload_one failed\n"); mlx5_health_cleanup(dev); -- 2.9.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/2] mlx5: fix 64-bit division on times 2016-06-15 15:27 ` Arnd Bergmann (?) @ 2016-06-15 15:27 ` Arnd Bergmann 2016-06-17 15:09 ` Saeed Mahameed -1 siblings, 1 reply; 12+ messages in thread From: Arnd Bergmann @ 2016-06-15 15:27 UTC (permalink / raw) To: Matan Barak, Leon Romanovsky Cc: Arnd Bergmann, Saeed Mahameed, David S. Miller, Achiad Shochat, Tariq Toukan, Gal Pressman, Maor Gottlieb, Huy Nguyen, netdev, linux-rdma, linux-kernel The mlx5 driver fails to build on 32-bit architectures after some references to 64-bit divisions got added: drivers/net/built-in.o: In function `mlx5e_rx_am': :(.text+0xf88ac): undefined reference to `__aeabi_ldivmod' The driver even performs three division here, and it uses the obsolete 'struct timespec' that we want to get rid of. Using ktime_t and ktime_us_delta() replaces one of the divisions and is mildly more efficient, aside from working across 'settimeofday' calls and being the right type for the y2038 conversion. Using a u32 instead of s64 to store the number of microseconds limits the maximum time to about 71 minutes, but if we exceed that time, we probably don't care about the result any more for the purpose of rx coalescing. For the number of packets, we are taking the difference between two 'unsigned int', so the result won't ever be greater than that either. After those changes, the other two divisions are done as 32-bit arithmetic operations, which are much faster. Signed-off-by: Arnd Bergmann <arnd@arndb.de> Fixes: 3841f0b3493b ("net/mlx5e: Support adaptive RX coalescing") --- drivers/net/ethernet/mellanox/mlx5/core/en.h | 2 +- drivers/net/ethernet/mellanox/mlx5/core/en_rx_am.c | 12 ++++++------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en.h b/drivers/net/ethernet/mellanox/mlx5/core/en.h index 775b8d02a3dc..37df5728323b 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en.h +++ b/drivers/net/ethernet/mellanox/mlx5/core/en.h @@ -272,7 +272,7 @@ struct mlx5e_rx_am_stats { }; struct mlx5e_rx_am_sample { - struct timespec time; + ktime_t time; unsigned int pkt_ctr; u16 event_ctr; }; diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_rx_am.c b/drivers/net/ethernet/mellanox/mlx5/core/en_rx_am.c index cdff5cace4c2..bd0c70220a80 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_rx_am.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_rx_am.c @@ -267,7 +267,7 @@ static bool mlx5e_am_decision(struct mlx5e_rx_am_stats *curr_stats, static void mlx5e_am_sample(struct mlx5e_rq *rq, struct mlx5e_rx_am_sample *s) { - getnstimeofday(&s->time); + s->time = ktime_get(); s->pkt_ctr = rq->stats.packets; s->event_ctr = rq->cq.event_ctr; } @@ -278,17 +278,17 @@ static void mlx5e_am_calc_stats(struct mlx5e_rx_am_sample *start, struct mlx5e_rx_am_sample *end, struct mlx5e_rx_am_stats *curr_stats) { - struct timespec time = timespec_sub(end->time, start->time); - s64 delta_us = timespec_to_ns(&time) / 1000; - s64 npkts = end->pkt_ctr - start->pkt_ctr; + /* u32 holds up to 71 minutes, should be enough */ + u32 delta_us = ktime_us_delta(end->time, start->time); + unsigned int npkts = end->pkt_ctr - start->pkt_ctr; if (!delta_us) { WARN_ONCE(true, "mlx5e_am_calc_stats: delta_us=0\n"); return; } - curr_stats->ppms = (npkts * 1000) / delta_us; - curr_stats->epms = (MLX5E_AM_NEVENTS * 1000) / delta_us; + curr_stats->ppms = (npkts * USEC_PER_MSEC) / delta_us; + curr_stats->epms = (MLX5E_AM_NEVENTS * USEC_PER_MSEC) / delta_us; } void mlx5e_rx_am_work(struct work_struct *work) -- 2.9.0 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] mlx5: fix 64-bit division on times 2016-06-15 15:27 ` [PATCH 2/2] mlx5: fix 64-bit division on times Arnd Bergmann @ 2016-06-17 15:09 ` Saeed Mahameed 2016-06-17 15:24 ` Arnd Bergmann 0 siblings, 1 reply; 12+ messages in thread From: Saeed Mahameed @ 2016-06-17 15:09 UTC (permalink / raw) To: Arnd Bergmann Cc: Matan Barak, Leon Romanovsky, Saeed Mahameed, David S. Miller, Achiad Shochat, Tariq Toukan, Gal Pressman, Maor Gottlieb, Huy Nguyen, Linux Netdev List, linux-rdma, linux-kernel On Wed, Jun 15, 2016 at 6:27 PM, Arnd Bergmann <arnd@arndb.de> wrote: > The mlx5 driver fails to build on 32-bit architectures after some > references to 64-bit divisions got added: > > drivers/net/built-in.o: In function `mlx5e_rx_am': > :(.text+0xf88ac): undefined reference to `__aeabi_ldivmod' > > The driver even performs three division here, and it uses the > obsolete 'struct timespec' that we want to get rid of. > > Using ktime_t and ktime_us_delta() replaces one of the divisions > and is mildly more efficient, aside from working across 'settimeofday' > calls and being the right type for the y2038 conversion. > > Using a u32 instead of s64 to store the number of microseconds > limits the maximum time to about 71 minutes, but if we exceed that > time, we probably don't care about the result any more for the > purpose of rx coalescing. > > For the number of packets, we are taking the difference between > two 'unsigned int', so the result won't ever be greater than that > either. > > After those changes, the other two divisions are done as 32-bit > arithmetic operations, which are much faster. Nice catch Arnd, we originally fixed this with div_u64, but your solution looks wiser. does ktime_t gives time in a resolution same as timespec ? As discussed before this patch can't be applied on net-next as the original patch which it meant to fix is yet to be submitted, I will CC you once we submit the fixed patch. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/2] mlx5: fix 64-bit division on times 2016-06-17 15:09 ` Saeed Mahameed @ 2016-06-17 15:24 ` Arnd Bergmann 0 siblings, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2016-06-17 15:24 UTC (permalink / raw) To: Saeed Mahameed Cc: Matan Barak, Leon Romanovsky, Saeed Mahameed, David S. Miller, Achiad Shochat, Tariq Toukan, Gal Pressman, Maor Gottlieb, Huy Nguyen, Linux Netdev List, linux-rdma, linux-kernel On Friday, June 17, 2016 6:09:00 PM CEST Saeed Mahameed wrote: > On Wed, Jun 15, 2016 at 6:27 PM, Arnd Bergmann <arnd@arndb.de> wrote: > > The mlx5 driver fails to build on 32-bit architectures after some > > references to 64-bit divisions got added: > > > > drivers/net/built-in.o: In function `mlx5e_rx_am': > > :(.text+0xf88ac): undefined reference to `__aeabi_ldivmod' > > > > The driver even performs three division here, and it uses the > > obsolete 'struct timespec' that we want to get rid of. > > > > Using ktime_t and ktime_us_delta() replaces one of the divisions > > and is mildly more efficient, aside from working across 'settimeofday' > > calls and being the right type for the y2038 conversion. > > > > Using a u32 instead of s64 to store the number of microseconds > > limits the maximum time to about 71 minutes, but if we exceed that > > time, we probably don't care about the result any more for the > > purpose of rx coalescing. > > > > For the number of packets, we are taking the difference between > > two 'unsigned int', so the result won't ever be greater than that > > either. > > > > After those changes, the other two divisions are done as 32-bit > > arithmetic operations, which are much faster. > > Nice catch Arnd, we originally fixed this with div_u64, but your > solution looks wiser. > does ktime_t gives time in a resolution same as timespec ? ktime_t is a 64-bit nanosecond counter, so the resolution is the same as ktime_get_ts64(), which is the "monotonic" equivalent of getnstimeofday(). There are also variants that have the same resolution but are less accurate and don't set the exact lower bits in order to get a faster reading, but the above are all as accurate as the machine allows. Arnd ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available 2016-06-15 15:27 ` Arnd Bergmann (?) (?) @ 2016-06-15 16:04 ` Saeed Mahameed [not found] ` <CALzJLG_CkvbrCehMjMht5KfMK6r-FbvPzzYUEhpA40YHymsh+Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> -1 siblings, 1 reply; 12+ messages in thread From: Saeed Mahameed @ 2016-06-15 16:04 UTC (permalink / raw) To: Arnd Bergmann Cc: Matan Barak, Leon Romanovsky, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma, linux-kernel On Wed, Jun 15, 2016 at 6:27 PM, Arnd Bergmann <arnd@arndb.de> wrote: > We get a build error with the mlx5 driver when the ethernet > support (CONFIG_MLX5_CORE_EN) is disabled: > > drivers/net/ethernet/mellanox/mlx5/core/main.c:1320:22: error: 'mlx5_devlink_eswitch_mode_set' undeclared here (not in a function) > drivers/net/ethernet/mellanox/mlx5/core/main.c:1321:22: error: 'mlx5_devlink_eswitch_mode_get' undeclared here (not in a function) > drivers/net/built-in.o:(.rodata+0x25a68): undefined reference to `mlx5_devlink_eswitch_mode_get' > drivers/net/built-in.o:(.rodata+0x25a6c): undefined reference to `mlx5_devlink_eswitch_mode_set' > > There are actually two problems here, but they are closely related, > so I'm addressing them both: > > - The header is included under an #ifdef, which is usually a bad idea > as it hides the function declarations, so we fail to compile even > if we don't actually use the functions in the end. > - The references to the functions are kept in the object file because > we don't check whether they are built-in or not. > > As we don't want to add any useless #ifdef here, this uses an > IS_ENABLED() check to drop the mlx5_devlink_ops structure when we don't > need it, and to skip the register/unregister step. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > Fixes: f7856daf57b9 ("net/mlx5: Add devlink interface") Hi Arnd, We already took care of those issues, they only apply to Leon's tree https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/, this tree is meant to maintain MLX5 Shared code between netdev and linux-rdma trees prior to submission to both trees. This patch is a non-shared code and it only exists in https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/log/?h=topic/net-next-mlx5. It is yet to be submitted to Dave's net/net-next tree. later on, this patch and all the others will go through the normal submission process. For the future I don't see any reason to CC the whole netdev, rdma and kernel folks. Unless you, Dave and Doug think otherwise. Thanks Saeed. ^ permalink raw reply [flat|nested] 12+ messages in thread
[parent not found: <CALzJLG_CkvbrCehMjMht5KfMK6r-FbvPzzYUEhpA40YHymsh+Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>]
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available 2016-06-15 16:04 ` [PATCH 1/2] mlx5: only register devlink when ethernet is available Saeed Mahameed @ 2016-06-15 20:50 ` Arnd Bergmann 0 siblings, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2016-06-15 20:50 UTC (permalink / raw) To: Saeed Mahameed Cc: Matan Barak, Leon Romanovsky, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > > Hi Arnd, > > We already took care of those issues, they only apply to Leon's tree > https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/, > this tree is meant to maintain MLX5 Shared code between netdev and > linux-rdma trees prior to submission to both trees. > > This patch is a non-shared code and it only exists in > https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/log/?h=topic/net-next-mlx5. > It is yet to be submitted to Dave's net/net-next tree. later on, this > patch and all the others will go through the normal submission > process. Ok, I see. It would be nice if the process had a way to avoid build regressions in linux-next, in particular if you already have a fix by the time a patch that introduces a problem gets added. Can you check if the fix for the second problem correctly removes the unnecessary 64-bit division (as opposed to adding a call to div_s64() or do_div()), and if it removes all traces of 'struct timespec' again? Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available @ 2016-06-15 20:50 ` Arnd Bergmann 0 siblings, 0 replies; 12+ messages in thread From: Arnd Bergmann @ 2016-06-15 20:50 UTC (permalink / raw) To: Saeed Mahameed Cc: Matan Barak, Leon Romanovsky, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma, linux-kernel On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > > Hi Arnd, > > We already took care of those issues, they only apply to Leon's tree > https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/, > this tree is meant to maintain MLX5 Shared code between netdev and > linux-rdma trees prior to submission to both trees. > > This patch is a non-shared code and it only exists in > https://git.kernel.org/cgit/linux/kernel/git/leon/linux-rdma.git/log/?h=topic/net-next-mlx5. > It is yet to be submitted to Dave's net/net-next tree. later on, this > patch and all the others will go through the normal submission > process. Ok, I see. It would be nice if the process had a way to avoid build regressions in linux-next, in particular if you already have a fix by the time a patch that introduces a problem gets added. Can you check if the fix for the second problem correctly removes the unnecessary 64-bit division (as opposed to adding a call to div_s64() or do_div()), and if it removes all traces of 'struct timespec' again? Arnd ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available 2016-06-15 20:50 ` Arnd Bergmann (?) @ 2016-06-17 14:50 ` Saeed Mahameed 2016-06-17 15:02 ` Arnd Bergmann -1 siblings, 1 reply; 12+ messages in thread From: Saeed Mahameed @ 2016-06-17 14:50 UTC (permalink / raw) To: Arnd Bergmann Cc: Matan Barak, Leon Romanovsky, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma, linux-kernel On Wed, Jun 15, 2016 at 11:50 PM, Arnd Bergmann <arnd@arndb.de> wrote: > On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > Ok, I see. It would be nice if the process had a way to avoid build regressions > in linux-next, in particular if you already have a fix by the time a patch > that introduces a problem gets added. > The reason we added this tree is to get 0-day testing but currently it makes some unwanted noise so we will remove it until we figure it out. > > Can you check if the fix for the second problem correctly removes the > unnecessary 64-bit division (as opposed to adding a call to div_s64() > or do_div()), and if it removes all traces of 'struct timespec' again? > Yes, same thing, already fixed, will reply to that thread. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available 2016-06-17 14:50 ` Saeed Mahameed @ 2016-06-17 15:02 ` Arnd Bergmann 2016-06-17 15:40 ` Leon Romanovsky 0 siblings, 1 reply; 12+ messages in thread From: Arnd Bergmann @ 2016-06-17 15:02 UTC (permalink / raw) To: Saeed Mahameed Cc: Matan Barak, Leon Romanovsky, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma, linux-kernel, Fengguang Wu On Friday, June 17, 2016 5:50:14 PM CEST Saeed Mahameed wrote: > On Wed, Jun 15, 2016 at 11:50 PM, Arnd Bergmann <arnd@arndb.de> wrote: > > On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > > Ok, I see. It would be nice if the process had a way to avoid build regressions > > in linux-next, in particular if you already have a fix by the time a patch > > that introduces a problem gets added. > > > > The reason we added this tree is to get 0-day testing but currently it > makes some unwanted noise > so we will remove it until we figure it out. I think you can simply ask Fengguang Wu to add your git tree to the list of trees he pulls from for the 0-day test bot. > > Can you check if the fix for the second problem correctly removes the > > unnecessary 64-bit division (as opposed to adding a call to div_s64() > > or do_div()), and if it removes all traces of 'struct timespec' again? > > > > Yes, same thing, already fixed, will reply to that thread. Ok, thanks for the confirmation! Arnd ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available 2016-06-17 15:02 ` Arnd Bergmann @ 2016-06-17 15:40 ` Leon Romanovsky 0 siblings, 0 replies; 12+ messages in thread From: Leon Romanovsky @ 2016-06-17 15:40 UTC (permalink / raw) To: Arnd Bergmann Cc: Saeed Mahameed, Matan Barak, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA, Fengguang Wu [-- Attachment #1: Type: text/plain, Size: 1103 bytes --] On Fri, Jun 17, 2016 at 05:02:33PM +0200, Arnd Bergmann wrote: > On Friday, June 17, 2016 5:50:14 PM CEST Saeed Mahameed wrote: > > On Wed, Jun 15, 2016 at 11:50 PM, Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org> wrote: > > > On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > > > Ok, I see. It would be nice if the process had a way to avoid build regressions > > > in linux-next, in particular if you already have a fix by the time a patch > > > that introduces a problem gets added. > > > > > > > The reason we added this tree is to get 0-day testing but currently it > > makes some unwanted noise > > so we will remove it until we figure it out. > > I think you can simply ask Fengguang Wu to add your git tree to the list > of trees he pulls from for the 0-day test bot. It is not 0-day only, but linux-next too. It works flawlessly for RDMA topics and Doug receives cleaned and fully tested patches. Sadly enough, it didn't work well for mlx5 net part. Till further notice, I removed mlx5 net part (submission queue) from my tree and from linux-next. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/2] mlx5: only register devlink when ethernet is available @ 2016-06-17 15:40 ` Leon Romanovsky 0 siblings, 0 replies; 12+ messages in thread From: Leon Romanovsky @ 2016-06-17 15:40 UTC (permalink / raw) To: Arnd Bergmann Cc: Saeed Mahameed, Matan Barak, David S. Miller, Saeed Mahameed, Or Gerlitz, Doug Ledford, Eli Cohen, Majd Dibbiny, Linux Netdev List, linux-rdma, linux-kernel, Fengguang Wu [-- Attachment #1: Type: text/plain, Size: 1083 bytes --] On Fri, Jun 17, 2016 at 05:02:33PM +0200, Arnd Bergmann wrote: > On Friday, June 17, 2016 5:50:14 PM CEST Saeed Mahameed wrote: > > On Wed, Jun 15, 2016 at 11:50 PM, Arnd Bergmann <arnd@arndb.de> wrote: > > > On Wednesday, June 15, 2016 7:04:54 PM CEST Saeed Mahameed wrote: > > > Ok, I see. It would be nice if the process had a way to avoid build regressions > > > in linux-next, in particular if you already have a fix by the time a patch > > > that introduces a problem gets added. > > > > > > > The reason we added this tree is to get 0-day testing but currently it > > makes some unwanted noise > > so we will remove it until we figure it out. > > I think you can simply ask Fengguang Wu to add your git tree to the list > of trees he pulls from for the 0-day test bot. It is not 0-day only, but linux-next too. It works flawlessly for RDMA topics and Doug receives cleaned and fully tested patches. Sadly enough, it didn't work well for mlx5 net part. Till further notice, I removed mlx5 net part (submission queue) from my tree and from linux-next. [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2016-06-17 16:11 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-06-15 15:27 [PATCH 1/2] mlx5: only register devlink when ethernet is available Arnd Bergmann 2016-06-15 15:27 ` Arnd Bergmann 2016-06-15 15:27 ` [PATCH 2/2] mlx5: fix 64-bit division on times Arnd Bergmann 2016-06-17 15:09 ` Saeed Mahameed 2016-06-17 15:24 ` Arnd Bergmann 2016-06-15 16:04 ` [PATCH 1/2] mlx5: only register devlink when ethernet is available Saeed Mahameed [not found] ` <CALzJLG_CkvbrCehMjMht5KfMK6r-FbvPzzYUEhpA40YHymsh+Q-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org> 2016-06-15 20:50 ` Arnd Bergmann 2016-06-15 20:50 ` Arnd Bergmann 2016-06-17 14:50 ` Saeed Mahameed 2016-06-17 15:02 ` Arnd Bergmann 2016-06-17 15:40 ` Leon Romanovsky 2016-06-17 15:40 ` Leon Romanovsky
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.