| Message ID | 20210809121931.2519-1-caihuoqing@baidu.com (mailing list archive) |
|---|---|
| State | Awaiting Upstream |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [v2] net/mlx5e: Make use of mlx5_core_warn() | expand |
| Context | Check | Description |
|---|---|---|
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Guessed tree name to be net-next |
| netdev/subject_prefix | warning | Target tree name not specified in the subject |
| netdev/cc_maintainers | warning | 2 maintainers not CCed: davem@davemloft.net kuba@kernel.org |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 0 this patch: 0 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | warning | WARNING: line length of 98 exceeds 80 columns |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 0 this patch: 0 |
| netdev/header_inline | success | Link |
On Mon, Aug 09, 2021 at 08:19:31PM +0800, Cai Huoqing wrote: > to replace printk(KERN_WARNING ...) with mlx5_core_warn() kindly > if we use mlx5_core_warn(), the prefix "mlx5:" not needed > > Signed-off-by: Cai Huoqing <caihuoqing@baidu.com> > --- > drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c Thanks, Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
On Mon, 2021-08-09 at 20:19 +0800, Cai Huoqing wrote: > to replace printk(KERN_WARNING ...) with mlx5_core_warn() kindly > if we use mlx5_core_warn(), the prefix "mlx5:" not needed in mlx5e it is netdev stack so netdev_warn(priv->netdev, "foo bar"); please.
On Mon, Aug 09, 2021 at 08:39:10PM +0000, Saeed Mahameed wrote: > On Mon, 2021-08-09 at 20:19 +0800, Cai Huoqing wrote: > > to replace printk(KERN_WARNING ...) with mlx5_core_warn() kindly > > if we use mlx5_core_warn(), the prefix "mlx5:" not needed > > in mlx5e it is netdev stack so netdev_warn(priv->netdev, "foo bar"); Saeed, That file is full of mlx5_core_* prints, even in the same function where Cai is changing, you will find mlx5_core_warn(). Thanks > please.
On Tue, 2021-08-10 at 09:58 +0300, Leon Romanovsky wrote: > On Mon, Aug 09, 2021 at 08:39:10PM +0000, Saeed Mahameed wrote: > > On Mon, 2021-08-09 at 20:19 +0800, Cai Huoqing wrote: > > > to replace printk(KERN_WARNING ...) with mlx5_core_warn() kindly > > > if we use mlx5_core_warn(), the prefix "mlx5:" not needed > > > > in mlx5e it is netdev stack so netdev_warn(priv->netdev, "foo bar"); > > Saeed, > > That file is full of mlx5_core_* prints, even in the same function > where > Cai is changing, you will find mlx5_core_warn(). > $ git grep -E "mlx5_core_(warn|info|err)" drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | wc -l 4 $ git grep -E "netdev_(warn|info|err)" drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | wc -l 24 4 vs 24, a big win for netdev ;-) 4 is not full. Many of mlx5 files has some leftovers historical code, doesn't mean we need to keep the old habits ..
diff --git a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c index e5c4344a114e..304cca0f54d3 100644 --- a/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c +++ b/drivers/net/ethernet/mellanox/mlx5/core/en_tc.c @@ -2702,7 +2702,8 @@ static int offload_pedit_fields(struct mlx5e_priv *priv, if (s_mask && a_mask) { NL_SET_ERR_MSG_MOD(extack, "can't set and add to the same HW field"); - printk(KERN_WARNING "mlx5: can't set and add to the same HW field (%x)\n", f->field); + mlx5_core_warn(priv->mdev, + "can't set and add to the same HW field (%x)\n", f->field); return -EOPNOTSUPP; } @@ -2741,8 +2742,9 @@ static int offload_pedit_fields(struct mlx5e_priv *priv, if (first < next_z && next_z < last) { NL_SET_ERR_MSG_MOD(extack, "rewrite of few sub-fields isn't supported"); - printk(KERN_WARNING "mlx5: rewrite of few sub-fields (mask %lx) isn't offloaded\n", - mask); + mlx5_core_warn(priv->mdev, + "rewrite of few sub-fields (mask %lx) isn't offloaded\n", + mask); return -EOPNOTSUPP; }
to replace printk(KERN_WARNING ...) with mlx5_core_warn() kindly if we use mlx5_core_warn(), the prefix "mlx5:" not needed Signed-off-by: Cai Huoqing <caihuoqing@baidu.com> --- drivers/net/ethernet/mellanox/mlx5/core/en_tc.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)