mirror of
https://github.com/torvalds/linux.git
synced 2026-05-27 08:33:17 +02:00
net: ena: Control PHC enable through devlink
Add the capability to set parameters through the devlink framework.
The parameter used for controlling PHC (enable/disable) details
are as follows:
- Name: enable_phc
- Type: Boolean (true - enable/false - disable)
- Mode: DEVLINK_PARAM_CMODE_DRIVERINIT
- Effect: Changes take place during driver initialization,
any changes require a devlink reload to take effect.
Signed-off-by: David Arinzon <darinzon@amazon.com>
Link: https://patch.msgid.link/20250617110545.5659-7-darinzon@amazon.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
parent
cea465a96a
commit
816b52624c
|
|
@ -5,6 +5,59 @@
|
||||||
|
|
||||||
#include "linux/pci.h"
|
#include "linux/pci.h"
|
||||||
#include "ena_devlink.h"
|
#include "ena_devlink.h"
|
||||||
|
#include "ena_phc.h"
|
||||||
|
|
||||||
|
static int ena_devlink_enable_phc_validate(struct devlink *devlink, u32 id,
|
||||||
|
union devlink_param_value val,
|
||||||
|
struct netlink_ext_ack *extack)
|
||||||
|
{
|
||||||
|
struct ena_adapter *adapter = ENA_DEVLINK_PRIV(devlink);
|
||||||
|
|
||||||
|
if (!val.vbool)
|
||||||
|
return 0;
|
||||||
|
|
||||||
|
if (!ena_com_phc_supported(adapter->ena_dev)) {
|
||||||
|
NL_SET_ERR_MSG_MOD(extack, "Device doesn't support PHC");
|
||||||
|
return -EOPNOTSUPP;
|
||||||
|
}
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
static const struct devlink_param ena_devlink_params[] = {
|
||||||
|
DEVLINK_PARAM_GENERIC(ENABLE_PHC,
|
||||||
|
BIT(DEVLINK_PARAM_CMODE_DRIVERINIT),
|
||||||
|
NULL,
|
||||||
|
NULL,
|
||||||
|
ena_devlink_enable_phc_validate),
|
||||||
|
};
|
||||||
|
|
||||||
|
void ena_devlink_params_get(struct devlink *devlink)
|
||||||
|
{
|
||||||
|
struct ena_adapter *adapter = ENA_DEVLINK_PRIV(devlink);
|
||||||
|
union devlink_param_value val;
|
||||||
|
int err;
|
||||||
|
|
||||||
|
err = devl_param_driverinit_value_get(devlink,
|
||||||
|
DEVLINK_PARAM_GENERIC_ID_ENABLE_PHC,
|
||||||
|
&val);
|
||||||
|
if (err) {
|
||||||
|
netdev_err(adapter->netdev, "Failed to query PHC param\n");
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
ena_phc_enable(adapter, val.vbool);
|
||||||
|
}
|
||||||
|
|
||||||
|
void ena_devlink_disable_phc_param(struct devlink *devlink)
|
||||||
|
{
|
||||||
|
union devlink_param_value value;
|
||||||
|
|
||||||
|
value.vbool = false;
|
||||||
|
devl_param_driverinit_value_set(devlink,
|
||||||
|
DEVLINK_PARAM_GENERIC_ID_ENABLE_PHC,
|
||||||
|
value);
|
||||||
|
}
|
||||||
|
|
||||||
static void ena_devlink_port_register(struct devlink *devlink)
|
static void ena_devlink_port_register(struct devlink *devlink)
|
||||||
{
|
{
|
||||||
|
|
@ -79,6 +132,27 @@ static const struct devlink_ops ena_devlink_ops = {
|
||||||
.reload_up = ena_devlink_reload_up,
|
.reload_up = ena_devlink_reload_up,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
static int ena_devlink_configure_params(struct devlink *devlink)
|
||||||
|
{
|
||||||
|
struct ena_adapter *adapter = ENA_DEVLINK_PRIV(devlink);
|
||||||
|
union devlink_param_value value;
|
||||||
|
int rc;
|
||||||
|
|
||||||
|
rc = devlink_params_register(devlink, ena_devlink_params,
|
||||||
|
ARRAY_SIZE(ena_devlink_params));
|
||||||
|
if (rc) {
|
||||||
|
netdev_err(adapter->netdev, "Failed to register devlink params\n");
|
||||||
|
return rc;
|
||||||
|
}
|
||||||
|
|
||||||
|
value.vbool = ena_phc_is_enabled(adapter);
|
||||||
|
devl_param_driverinit_value_set(devlink,
|
||||||
|
DEVLINK_PARAM_GENERIC_ID_ENABLE_PHC,
|
||||||
|
value);
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
struct devlink *ena_devlink_alloc(struct ena_adapter *adapter)
|
struct devlink *ena_devlink_alloc(struct ena_adapter *adapter)
|
||||||
{
|
{
|
||||||
struct device *dev = &adapter->pdev->dev;
|
struct device *dev = &adapter->pdev->dev;
|
||||||
|
|
@ -96,11 +170,26 @@ struct devlink *ena_devlink_alloc(struct ena_adapter *adapter)
|
||||||
ENA_DEVLINK_PRIV(devlink) = adapter;
|
ENA_DEVLINK_PRIV(devlink) = adapter;
|
||||||
adapter->devlink = devlink;
|
adapter->devlink = devlink;
|
||||||
|
|
||||||
|
if (ena_devlink_configure_params(devlink))
|
||||||
|
goto free_devlink;
|
||||||
|
|
||||||
return devlink;
|
return devlink;
|
||||||
|
|
||||||
|
free_devlink:
|
||||||
|
devlink_free(devlink);
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
static void ena_devlink_configure_params_clean(struct devlink *devlink)
|
||||||
|
{
|
||||||
|
devlink_params_unregister(devlink, ena_devlink_params,
|
||||||
|
ARRAY_SIZE(ena_devlink_params));
|
||||||
}
|
}
|
||||||
|
|
||||||
void ena_devlink_free(struct devlink *devlink)
|
void ena_devlink_free(struct devlink *devlink)
|
||||||
{
|
{
|
||||||
|
ena_devlink_configure_params_clean(devlink);
|
||||||
|
|
||||||
devlink_free(devlink);
|
devlink_free(devlink);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -15,5 +15,7 @@ struct devlink *ena_devlink_alloc(struct ena_adapter *adapter);
|
||||||
void ena_devlink_free(struct devlink *devlink);
|
void ena_devlink_free(struct devlink *devlink);
|
||||||
void ena_devlink_register(struct devlink *devlink, struct device *dev);
|
void ena_devlink_register(struct devlink *devlink, struct device *dev);
|
||||||
void ena_devlink_unregister(struct devlink *devlink);
|
void ena_devlink_unregister(struct devlink *devlink);
|
||||||
|
void ena_devlink_params_get(struct devlink *devlink);
|
||||||
|
void ena_devlink_disable_phc_param(struct devlink *devlink);
|
||||||
|
|
||||||
#endif /* DEVLINK_H */
|
#endif /* DEVLINK_H */
|
||||||
|
|
|
||||||
|
|
@ -3138,6 +3138,8 @@ static int ena_device_init(struct ena_adapter *adapter, struct pci_dev *pdev,
|
||||||
goto err_mmio_read_less;
|
goto err_mmio_read_less;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
ena_devlink_params_get(adapter->devlink);
|
||||||
|
|
||||||
/* ENA admin level init */
|
/* ENA admin level init */
|
||||||
rc = ena_com_admin_init(ena_dev, &aenq_handlers);
|
rc = ena_com_admin_init(ena_dev, &aenq_handlers);
|
||||||
if (rc) {
|
if (rc) {
|
||||||
|
|
|
||||||
|
|
@ -6,6 +6,7 @@
|
||||||
#include <linux/pci.h>
|
#include <linux/pci.h>
|
||||||
#include "ena_netdev.h"
|
#include "ena_netdev.h"
|
||||||
#include "ena_phc.h"
|
#include "ena_phc.h"
|
||||||
|
#include "ena_devlink.h"
|
||||||
|
|
||||||
static int ena_phc_adjtime(struct ptp_clock_info *clock_info, s64 delta)
|
static int ena_phc_adjtime(struct ptp_clock_info *clock_info, s64 delta)
|
||||||
{
|
{
|
||||||
|
|
@ -213,6 +214,7 @@ int ena_phc_init(struct ena_adapter *adapter)
|
||||||
ena_com_phc_destroy(ena_dev);
|
ena_com_phc_destroy(ena_dev);
|
||||||
err_ena_com_phc_init:
|
err_ena_com_phc_init:
|
||||||
ena_phc_enable(adapter, false);
|
ena_phc_enable(adapter, false);
|
||||||
|
ena_devlink_disable_phc_param(adapter->devlink);
|
||||||
return rc;
|
return rc;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user