* [Intel-wired-lan] [PATCH net v4 0/2] i40e: Fix for link-flapping
@ 2021-01-18 19:34 Arkadiusz Kubalewski
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues" Arkadiusz Kubalewski
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code Arkadiusz Kubalewski
0 siblings, 2 replies; 7+ messages in thread
From: Arkadiusz Kubalewski @ 2021-01-18 19:34 UTC (permalink / raw)
To: intel-wired-lan
VF queues were not brought up when PF was brought up after being downed
if the VF driver disabled VFs queues during PF down. This could happen
in some older or external VF driver implementations.
The problem was that PF driver used vf->queues_enabled as a condition
to decide what link-state it would send out which caused the issue.
Remove the check for vf->queues_enabled in the VF link notify. Now VF
will always be notified of the current link status. Also remove
the queues_enabled member from i40e_vf structure as it is not used
anymore. Also refactor repeated link state reporting code into a
separate helper function i40e_set_vf_link_state().
Otherwise VNF implementation was broken and caused a link flap.
Fixes: 2ad1274fa35a ("i40e: don't report link up for a VF who hasn't enabled")
v2: improve commit message
v3: net tree target
v4: split into 2 patches
Arkadiusz Kubalewski (2):
i40e: Revert "i40e: don't report link up for a VF who hasn't enabled
queues"
i40e: refactor repeated link state reporting code
.../ethernet/intel/i40e/i40e_virtchnl_pf.c | 115 ++++++++++--------
.../ethernet/intel/i40e/i40e_virtchnl_pf.h | 1 -
2 files changed, 67 insertions(+), 49 deletions(-)
base-commit: 82edab16ec1fc048ef6a9a96ec5a0d1374f5bf3b
--
2.18.4
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues"
2021-01-18 19:34 [Intel-wired-lan] [PATCH net v4 0/2] i40e: Fix for link-flapping Arkadiusz Kubalewski
@ 2021-01-18 19:34 ` Arkadiusz Kubalewski
2021-01-20 1:29 ` Nguyen, Anthony L
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code Arkadiusz Kubalewski
1 sibling, 1 reply; 7+ messages in thread
From: Arkadiusz Kubalewski @ 2021-01-18 19:34 UTC (permalink / raw)
To: intel-wired-lan
From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
This reverts commit 2ad1274fa35ace5c6360762ba48d33b63da2396c
Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
---
v2: improve commit message
v3: net tree target
v4: split into 2 patches
---
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c | 13 +------------
drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h | 1 -
2 files changed, 1 insertion(+), 13 deletions(-)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
index 7efc61a..1b6ec9b 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
@@ -55,12 +55,7 @@ static void i40e_vc_notify_vf_link_state(struct i40e_vf *vf)
pfe.event = VIRTCHNL_EVENT_LINK_CHANGE;
pfe.severity = PF_EVENT_SEVERITY_INFO;
-
- /* Always report link is down if the VF queues aren't enabled */
- if (!vf->queues_enabled) {
- pfe.event_data.link_event.link_status = false;
- pfe.event_data.link_event.link_speed = 0;
- } else if (vf->link_forced) {
+ if (vf->link_forced) {
pfe.event_data.link_event.link_status = vf->link_up;
pfe.event_data.link_event.link_speed =
(vf->link_up ? i40e_virtchnl_link_speed(ls->link_speed) : 0);
@@ -70,7 +65,6 @@ static void i40e_vc_notify_vf_link_state(struct i40e_vf *vf)
pfe.event_data.link_event.link_speed =
i40e_virtchnl_link_speed(ls->link_speed);
}
-
i40e_aq_send_msg_to_vf(hw, abs_vf_id, VIRTCHNL_OP_EVENT,
0, (u8 *)&pfe, sizeof(pfe), NULL);
}
@@ -2443,8 +2437,6 @@ static int i40e_vc_enable_queues_msg(struct i40e_vf *vf, u8 *msg)
}
}
- vf->queues_enabled = true;
-
error_param:
/* send the response to the VF */
return i40e_vc_send_resp_to_vf(vf, VIRTCHNL_OP_ENABLE_QUEUES,
@@ -2466,9 +2458,6 @@ static int i40e_vc_disable_queues_msg(struct i40e_vf *vf, u8 *msg)
struct i40e_pf *pf = vf->pf;
i40e_status aq_ret = 0;
- /* Immediately mark queues as disabled */
- vf->queues_enabled = false;
-
if (!test_bit(I40E_VF_STATE_ACTIVE, &vf->vf_states)) {
aq_ret = I40E_ERR_PARAM;
goto error_param;
diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h
index 5491215..091e32c 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.h
@@ -98,7 +98,6 @@ struct i40e_vf {
unsigned int tx_rate; /* Tx bandwidth limit in Mbps */
bool link_forced;
bool link_up; /* only valid if VF link is forced */
- bool queues_enabled; /* true if the VF queues are enabled */
bool spoofchk;
u16 num_vlan;
--
2.18.4
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code
2021-01-18 19:34 [Intel-wired-lan] [PATCH net v4 0/2] i40e: Fix for link-flapping Arkadiusz Kubalewski
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues" Arkadiusz Kubalewski
@ 2021-01-18 19:34 ` Arkadiusz Kubalewski
2021-01-20 1:28 ` Nguyen, Anthony L
1 sibling, 1 reply; 7+ messages in thread
From: Arkadiusz Kubalewski @ 2021-01-18 19:34 UTC (permalink / raw)
To: intel-wired-lan
From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Refactor repeated link state reporting code into a separate helper functions:
i40e_set_vf_link_state() i40e_vc_link_speed2mbps().
Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
---
v2: improve commit message
v3: net tree target
v4: split into 2 patches
---
.../ethernet/intel/i40e/i40e_virtchnl_pf.c | 108 +++++++++++-------
1 file changed, 69 insertions(+), 39 deletions(-)
diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
index 1b6ec9b..6621943 100644
--- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
@@ -39,6 +39,66 @@ static void i40e_vc_vf_broadcast(struct i40e_pf *pf,
}
}
+/**
+ * i40e_vc_link_speed2mbps
+ * converts i40e_aq_link_speed to integer value of Mbps
+ * @link_speed: the speed to convert
+ *
+ * return the speed as direct value of Mbps.
+ **/
+static u32
+i40e_vc_link_speed2mbps(enum i40e_aq_link_speed link_speed)
+{
+ switch (link_speed) {
+ case I40E_LINK_SPEED_100MB:
+ return SPEED_100;
+ case I40E_LINK_SPEED_1GB:
+ return SPEED_1000;
+ case I40E_LINK_SPEED_2_5GB:
+ return SPEED_2500;
+ case I40E_LINK_SPEED_5GB:
+ return SPEED_5000;
+ case I40E_LINK_SPEED_10GB:
+ return SPEED_10000;
+ case I40E_LINK_SPEED_20GB:
+ return SPEED_20000;
+ case I40E_LINK_SPEED_25GB:
+ return SPEED_25000;
+ case I40E_LINK_SPEED_40GB:
+ return SPEED_40000;
+ case I40E_LINK_SPEED_UNKNOWN:
+ return SPEED_UNKNOWN;
+ }
+ return SPEED_UNKNOWN;
+}
+
+/**
+ * i40e_set_vf_link_state
+ * @vf: pointer to the VF structure
+ * @pfe: pointer to PF event structure
+ * @ls: pointer to link status structure
+ *
+ * set a link state on a single vf
+ **/
+static void i40e_set_vf_link_state(struct i40e_vf *vf,
+ struct virtchnl_pf_event *pfe, struct i40e_link_status *ls)
+{
+ u8 link_status = ls->link_info & I40E_AQ_LINK_UP;
+
+ if (vf->link_forced)
+ link_status = vf->link_up;
+
+ if (vf->driver_caps & VIRTCHNL_VF_CAP_ADV_LINK_SPEED) {
+ pfe->event_data.link_event_adv.link_speed = link_status ?
+ i40e_vc_link_speed2mbps(ls->link_speed) : 0;
+ pfe->event_data.link_event_adv.link_status = link_status;
+ } else {
+ pfe->event_data.link_event.link_speed = link_status ?
+ i40e_virtchnl_link_speed(ls->link_speed) : 0;
+ pfe->event_data.link_event.link_status = link_status;
+ }
+}
+
/**
* i40e_vc_notify_vf_link_state
* @vf: pointer to the VF structure
@@ -55,16 +115,9 @@ static void i40e_vc_notify_vf_link_state(struct i40e_vf *vf)
pfe.event = VIRTCHNL_EVENT_LINK_CHANGE;
pfe.severity = PF_EVENT_SEVERITY_INFO;
- if (vf->link_forced) {
- pfe.event_data.link_event.link_status = vf->link_up;
- pfe.event_data.link_event.link_speed =
- (vf->link_up ? i40e_virtchnl_link_speed(ls->link_speed) : 0);
- } else {
- pfe.event_data.link_event.link_status =
- ls->link_info & I40E_AQ_LINK_UP;
- pfe.event_data.link_event.link_speed =
- i40e_virtchnl_link_speed(ls->link_speed);
- }
+
+ i40e_set_vf_link_state(vf, &pfe, ls);
+
i40e_aq_send_msg_to_vf(hw, abs_vf_id, VIRTCHNL_OP_EVENT,
0, (u8 *)&pfe, sizeof(pfe), NULL);
}
@@ -1940,6 +1993,7 @@ static int i40e_vc_get_vf_resources_msg(struct i40e_vf *vf, u8 *msg)
VIRTCHNL_VF_OFFLOAD_VLAN;
vfres->vf_cap_flags = VIRTCHNL_VF_OFFLOAD_L2;
+ vfres->vf_cap_flags |= VIRTCHNL_VF_CAP_ADV_LINK_SPEED;
vsi = pf->vsi[vf->lan_vsi_idx];
if (!vsi->info.pvid)
vfres->vf_cap_flags |= VIRTCHNL_VF_OFFLOAD_VLAN;
@@ -3687,26 +3741,8 @@ static int i40e_vc_add_qch_msg(struct i40e_vf *vf, u8 *msg)
}
/* get link speed in MB to validate rate limit */
- switch (ls->link_speed) {
- case VIRTCHNL_LINK_SPEED_100MB:
- speed = SPEED_100;
- break;
- case VIRTCHNL_LINK_SPEED_1GB:
- speed = SPEED_1000;
- break;
- case VIRTCHNL_LINK_SPEED_10GB:
- speed = SPEED_10000;
- break;
- case VIRTCHNL_LINK_SPEED_20GB:
- speed = SPEED_20000;
- break;
- case VIRTCHNL_LINK_SPEED_25GB:
- speed = SPEED_25000;
- break;
- case VIRTCHNL_LINK_SPEED_40GB:
- speed = SPEED_40000;
- break;
- default:
+ speed = i40e_vc_link_speed2mbps(ls->link_speed);
+ if (speed == SPEED_UNKNOWN) {
dev_err(&pf->pdev->dev,
"Cannot detect link speed\n");
aq_ret = I40E_ERR_PARAM;
@@ -4455,23 +4491,17 @@ int i40e_ndo_set_vf_link_state(struct net_device *netdev, int vf_id, int link)
switch (link) {
case IFLA_VF_LINK_STATE_AUTO:
vf->link_forced = false;
- pfe.event_data.link_event.link_status =
- pf->hw.phy.link_info.link_info & I40E_AQ_LINK_UP;
- pfe.event_data.link_event.link_speed =
- (enum virtchnl_link_speed)
- pf->hw.phy.link_info.link_speed;
+ i40e_set_vf_link_state(vf, &pfe, ls);
break;
case IFLA_VF_LINK_STATE_ENABLE:
vf->link_forced = true;
vf->link_up = true;
- pfe.event_data.link_event.link_status = true;
- pfe.event_data.link_event.link_speed = i40e_virtchnl_link_speed(ls->link_speed);
+ i40e_set_vf_link_state(vf, &pfe, ls);
break;
case IFLA_VF_LINK_STATE_DISABLE:
vf->link_forced = true;
vf->link_up = false;
- pfe.event_data.link_event.link_status = false;
- pfe.event_data.link_event.link_speed = 0;
+ i40e_set_vf_link_state(vf, &pfe, ls);
break;
default:
ret = -EINVAL;
--
2.18.4
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code Arkadiusz Kubalewski
@ 2021-01-20 1:28 ` Nguyen, Anthony L
0 siblings, 0 replies; 7+ messages in thread
From: Nguyen, Anthony L @ 2021-01-20 1:28 UTC (permalink / raw)
To: intel-wired-lan
On Mon, 2021-01-18 at 19:34 +0000, Arkadiusz Kubalewski wrote:
> From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
>
> Refactor repeated link state reporting code into a separate helper
> functions:
> i40e_set_vf_link_state() i40e_vc_link_speed2mbps().
As this is refactoring, this is should be for net-next.
> Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
> Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
> ---
> v2: improve commit message
> v3: net tree target
> v4: split into 2 patches
> ---
> .../ethernet/intel/i40e/i40e_virtchnl_pf.c | 108 +++++++++++-----
> --
> 1 file changed, 69 insertions(+), 39 deletions(-)
>
> diff --git a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> index 1b6ec9b..6621943 100644
> --- a/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> +++ b/drivers/net/ethernet/intel/i40e/i40e_virtchnl_pf.c
> @@ -39,6 +39,66 @@ static void i40e_vc_vf_broadcast(struct i40e_pf
> *pf,
> }
> }
>
> +/**
> + * i40e_vc_link_speed2mbps
> + * converts i40e_aq_link_speed to integer value of Mbps
> + * @link_speed: the speed to convert
> + *
> + * return the speed as direct value of Mbps.
> + **/
> +static u32
> +i40e_vc_link_speed2mbps(enum i40e_aq_link_speed link_speed)
> +{
> + switch (link_speed) {
> + case I40E_LINK_SPEED_100MB:
> + return SPEED_100;
> + case I40E_LINK_SPEED_1GB:
> + return SPEED_1000;
> + case I40E_LINK_SPEED_2_5GB:
> + return SPEED_2500;
> + case I40E_LINK_SPEED_5GB:
> + return SPEED_5000;
> + case I40E_LINK_SPEED_10GB:
> + return SPEED_10000;
> + case I40E_LINK_SPEED_20GB:
> + return SPEED_20000;
> + case I40E_LINK_SPEED_25GB:
> + return SPEED_25000;
> + case I40E_LINK_SPEED_40GB:
> + return SPEED_40000;
> + case I40E_LINK_SPEED_UNKNOWN:
> + return SPEED_UNKNOWN;
> + }
> + return SPEED_UNKNOWN;
> +}
> +
> +/**
> + * i40e_set_vf_link_state
> + * @vf: pointer to the VF structure
> + * @pfe: pointer to PF event structure
> + * @ls: pointer to link status structure
> + *
> + * set a link state on a single vf
> + **/
> +static void i40e_set_vf_link_state(struct i40e_vf *vf,
> + struct virtchnl_pf_event *pfe,
> struct i40e_link_status *ls)
> +{
> + u8 link_status = ls->link_info & I40E_AQ_LINK_UP;
> +
> + if (vf->link_forced)
> + link_status = vf->link_up;
> +
> + if (vf->driver_caps & VIRTCHNL_VF_CAP_ADV_LINK_SPEED) {
> + pfe->event_data.link_event_adv.link_speed = link_status
> ?
> + i40e_vc_link_speed2mbps(ls->link_speed) : 0;
> + pfe->event_data.link_event_adv.link_status =
> link_status;
> + } else {
> + pfe->event_data.link_event.link_speed = link_status ?
> + i40e_virtchnl_link_speed(ls->link_speed) : 0;
> + pfe->event_data.link_event.link_status = link_status;
> + }
> +}
> +
> /**
> * i40e_vc_notify_vf_link_state
> * @vf: pointer to the VF structure
> @@ -55,16 +115,9 @@ static void i40e_vc_notify_vf_link_state(struct
> i40e_vf *vf)
>
> pfe.event = VIRTCHNL_EVENT_LINK_CHANGE;
> pfe.severity = PF_EVENT_SEVERITY_INFO;
> - if (vf->link_forced) {
> - pfe.event_data.link_event.link_status = vf->link_up;
> - pfe.event_data.link_event.link_speed =
> - (vf->link_up ? i40e_virtchnl_link_speed(ls-
> >link_speed) : 0);
> - } else {
> - pfe.event_data.link_event.link_status =
> - ls->link_info & I40E_AQ_LINK_UP;
> - pfe.event_data.link_event.link_speed =
> - i40e_virtchnl_link_speed(ls->link_speed);
> - }
> +
> + i40e_set_vf_link_state(vf, &pfe, ls);
> +
> i40e_aq_send_msg_to_vf(hw, abs_vf_id, VIRTCHNL_OP_EVENT,
> 0, (u8 *)&pfe, sizeof(pfe), NULL);
> }
> @@ -1940,6 +1993,7 @@ static int i40e_vc_get_vf_resources_msg(struct
> i40e_vf *vf, u8 *msg)
> VIRTCHNL_VF_OFFLOAD_VLAN;
>
> vfres->vf_cap_flags = VIRTCHNL_VF_OFFLOAD_L2;
> + vfres->vf_cap_flags |= VIRTCHNL_VF_CAP_ADV_LINK_SPEED;
There is no mention about addding this capability in the commit
message.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues"
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues" Arkadiusz Kubalewski
@ 2021-01-20 1:29 ` Nguyen, Anthony L
2021-01-20 10:17 ` Loktionov, Aleksandr
0 siblings, 1 reply; 7+ messages in thread
From: Nguyen, Anthony L @ 2021-01-20 1:29 UTC (permalink / raw)
To: intel-wired-lan
On Mon, 2021-01-18 at 19:34 +0000, Arkadiusz Kubalewski wrote:
> From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
>
> This reverts commit 2ad1274fa35ace5c6360762ba48d33b63da2396c
Can you provide more info on why we are reverting? Some of the
information from the cover letter would probably be useful here.
> Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
> Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
> ---
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues"
2021-01-20 1:29 ` Nguyen, Anthony L
@ 2021-01-20 10:17 ` Loktionov, Aleksandr
2021-01-20 22:17 ` Nguyen, Anthony L
0 siblings, 1 reply; 7+ messages in thread
From: Loktionov, Aleksandr @ 2021-01-20 10:17 UTC (permalink / raw)
To: intel-wired-lan
Good day Anthony
This is the first time we push a revert commit. /*we thought that CL should be enough, and provided standard revert commit message from GIT*/
Can you link us to example of good revert commit message?
Thank you
Alex
ND ITP Linux 40G base driver TL
-----Original Message-----
From: Nguyen, Anthony L <anthony.l.nguyen@intel.com>
Sent: Wednesday, January 20, 2021 2:30 AM
To: Kubalewski, Arkadiusz <arkadiusz.kubalewski@intel.com>; intel-wired-lan at lists.osuosl.org
Cc: Loktionov, Aleksandr <aleksandr.loktionov@intel.com>
Subject: Re: [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues"
On Mon, 2021-01-18 at 19:34 +0000, Arkadiusz Kubalewski wrote:
> From: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
>
> This reverts commit 2ad1274fa35ace5c6360762ba48d33b63da2396c
Can you provide more info on why we are reverting? Some of the information from the cover letter would probably be useful here.
> Signed-off-by: Aleksandr Loktionov <aleksandr.loktionov@intel.com>
> Signed-off-by: Arkadiusz Kubalewski <arkadiusz.kubalewski@intel.com>
> ---
---------------------------------------------------------------------
Intel Technology Poland sp. z o.o.
ul. Sowackiego 173 | 80-298 Gdask | Sd Rejonowy Gdask Pnoc | VII Wydzia Gospodarczy Krajowego Rejestru Sdowego - KRS 101882 | NIP 957-07-52-316 | Kapita zakadowy 200.000 PLN.
Ta wiadomo wraz z zacznikami jest przeznaczona dla okrelonego adresata i moe zawiera informacje poufne. W razie przypadkowego otrzymania tej wiadomoci, prosimy o powiadomienie nadawcy oraz trwae jej usunicie; jakiekolwiek przegldanie lub rozpowszechnianie jest zabronione.
This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). If you are not the intended recipient, please contact the sender and delete all copies; any review or distribution by others is strictly prohibited.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues"
2021-01-20 10:17 ` Loktionov, Aleksandr
@ 2021-01-20 22:17 ` Nguyen, Anthony L
0 siblings, 0 replies; 7+ messages in thread
From: Nguyen, Anthony L @ 2021-01-20 22:17 UTC (permalink / raw)
To: intel-wired-lan
On Wed, 2021-01-20 at 10:17 +0000, Loktionov, Aleksandr wrote:
> Good day Anthony
>
> This is the first time we push a revert commit. /*we thought that CL
> should be enough, and provided standard revert commit message from
> GIT*/
> Can you link us to example of good revert commit message?
Here are a few links to some reverts to Intel drivers:
https://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-
queue.git/commit/?id=6cecf02e77ab9bf97e9252f9fcb8f0738a6de12c
https://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-queue.git/commit/?id=caff422ea81e144842bc44bab408d85ac449377b
https://git.kernel.org/pub/scm/linux/kernel/git/tnguy/next-queue.git/commit/?id=0a92892c69bd01b1a9481ef49587f60e97c64042
Thanks,
Tony
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-01-20 22:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-18 19:34 [Intel-wired-lan] [PATCH net v4 0/2] i40e: Fix for link-flapping Arkadiusz Kubalewski
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 1/2] i40e: Revert "i40e: don't report link up for a VF who hasn't enabled queues" Arkadiusz Kubalewski
2021-01-20 1:29 ` Nguyen, Anthony L
2021-01-20 10:17 ` Loktionov, Aleksandr
2021-01-20 22:17 ` Nguyen, Anthony L
2021-01-18 19:34 ` [Intel-wired-lan] [PATCH net v4 2/2] i40e: refactor repeated link state reporting code Arkadiusz Kubalewski
2021-01-20 1:28 ` Nguyen, Anthony L
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.