Re: [PATCH 01/15] devlink: prepare to support region operations

[Jacob Keller <jacob.e.keller@intel.com>]

On 2/3/2020 9:07 AM, Jacob Keller wrote:
> On 2/3/2020 3:35 AM, Jiri Pirko wrote:
>> Thu, Jan 30, 2020 at 11:58:56PM CET, jacob.e.keller@intel.com wrote:
>>> Modify the devlink region code in preparation for adding new operations
>>> on regions.
>>>
>>> Create a devlink_region_ops structure, and move the name pointer from
>>> within the devlink_region structure into the ops structure (similar to
>>> the devlink_health_reporter_ops).
>>>
>>> This prepares the regions to enable support of additional operations in
>>> the future such as requesting snapshots, or accessing the region
>>> directly without a snapshot.
>>>
>>> Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
>>> ---
>>> drivers/net/ethernet/mellanox/mlx4/crdump.c | 25 ++++++++++++---------
>>> drivers/net/netdevsim/dev.c                 |  6 ++++-
>>> include/net/devlink.h                       | 17 ++++++++++----
>>> net/core/devlink.c                          | 23 ++++++++++---------
>>> 4 files changed, 45 insertions(+), 26 deletions(-)
>>>
>>> diff --git a/drivers/net/ethernet/mellanox/mlx4/crdump.c b/drivers/net/ethernet/mellanox/mlx4/crdump.c
>>> index 64ed725aec28..4cea64033919 100644
>>> --- a/drivers/net/ethernet/mellanox/mlx4/crdump.c
>>> +++ b/drivers/net/ethernet/mellanox/mlx4/crdump.c
>>> @@ -38,8 +38,13 @@
>>> #define CR_ENABLE_BIT_OFFSET		0xF3F04
>>> #define MAX_NUM_OF_DUMPS_TO_STORE	(8)
>>>
>>> -static const char *region_cr_space_str = "cr-space";
>>> -static const char *region_fw_health_str = "fw-health";
>>
>> Just leave these as are and use in ops and messages. It is odd to use
>> ops.name in the message.
>>
> 
> So this produces the following errors, not 100% sure how to resolve:
> 
> 
>> drivers/net/ethernet/mellanox/mlx4/crdump.c:45:10: error: initializer element is not constant
>>    45 |  .name = region_cr_space_str,
>>       |          ^~~~~~~~~~~~~~~~~~~
>> drivers/net/ethernet/mellanox/mlx4/crdump.c:45:10: note: (near initialization for ‘region_cr_space_ops.name’)
>> drivers/net/ethernet/mellanox/mlx4/crdump.c:49:10: error: initializer element is not constant
>>    49 |  .name = region_fw_health_str,
>>       |          ^~~~~~~~~~~~~~~~~~~~
> 
> 
> Thanks,
> Jake
> 

Heh, this is fixed by using static const char * const <name>, to ensure
that the compiler realizes both the pointer and the contents are constant.

Thanks,
Jake