On Fri, Feb 14, 2025 at 09:50:35AM +0100, Larysa Zaremba wrote: > From: Mateusz Pacuszka <mateuszx.pacus...@intel.com> > > In case the rule already exists and another VSI wants to subscribe to it > new VSI list is being created and both VSIs are moved to it. > Currently, the check for already existing VSI with the same rule is done > based on fdw_id.hw_vsi_id, which applies only to LOOKUP_RX flag. > Change it to vsi_handle. This is software VSI ID, but it can be applied > here, because vsi_map itself is also based on it. > > Additionally change return status in case the VSI already exists in the > VSI map to "Already exists". Such case should be handled by the caller.
FWIIW, I might have made this two patches, but I don't feel particularly strongly about it. > > Signed-off-by: Mateusz Pacuszka <mateuszx.pacus...@intel.com> > Reviewed-by: Przemek Kitszel <przemyslaw.kits...@intel.com> > Reviewed-by: Michal Swiatkowski <michal.swiatkow...@linux.intel.com> > Signed-off-by: Larysa Zaremba <larysa.zare...@intel.com> Reviewed-by: Simon Horman <ho...@kernel.org>