From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6313DC25B10 for ; Fri, 10 May 2024 07:21:32 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 0EF3D4032A; Fri, 10 May 2024 07:21:32 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id dGaX9K8QOsrc; Fri, 10 May 2024 07:21:31 +0000 (UTC) X-Comment: SPF check N/A for local connections - client-ip=140.211.166.34; helo=ash.osuosl.org; envelope-from=intel-wired-lan-bounces@osuosl.org; receiver= DKIM-Filter: OpenDKIM Filter v2.11.0 smtp2.osuosl.org 3359F4046C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org; s=default; t=1715325691; bh=FNHBuqpasGRzsgLtlwXyhbKdxJCfeuOApWpqnhVfP+g=; h=Date:From:To:References:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: Cc:From; b=2peghUQO7LqP5gOIfqRculMpJ1n9PFIWlKXzynv8RWwm2hy6KgIDC3SJ4yBJrsMLQ Nlfh5EbWaFuFGTSeYbr6N48CaUJMdGHvrUSIhJIqOA3KPP7cLlZN4koKYCkQVkma+T EOX3bDb25Ucx27J9rk4/9MnLkxtrvFtPeruP+TAldn8qp3mx1DmGKWToJ+iC8HzDlr GNFa1OctEuE7rXNo4JEDFCyOqFIWfdal54hl/MCZpihhyj7Z158X0fbacTSRFftUVC 0p0yboyJowSNJIo+oj2EK4IFGA2klbNVxnOm3BrGyZOmsoF+eKR1fKyAERCWM5GmQA SM3t+lwXuLJuQ== Received: from ash.osuosl.org (ash.osuosl.org [140.211.166.34]) by smtp2.osuosl.org (Postfix) with ESMTP id 3359F4046C; Fri, 10 May 2024 07:21:31 +0000 (UTC) Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) by ash.osuosl.org (Postfix) with ESMTP id E74691BF5DC for ; Fri, 10 May 2024 07:21:28 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id DFA9261574 for ; Fri, 10 May 2024 07:21:28 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id VuCqcTbeObjb for ; Fri, 10 May 2024 07:21:27 +0000 (UTC) Received-SPF: None (mailfrom) identity=mailfrom; client-ip=192.198.163.12; helo=mgamail.intel.com; envelope-from=michal.swiatkowski@linux.intel.com; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp3.osuosl.org 6E0826153B DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 6E0826153B Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) by smtp3.osuosl.org (Postfix) with ESMTPS id 6E0826153B for ; Fri, 10 May 2024 07:21:27 +0000 (UTC) X-CSE-ConnectionGUID: ZCJfvUawSuq8WuHPVXIIbQ== X-CSE-MsgGUID: r0L81gQaSIySLKPqKWfAZA== X-IronPort-AV: E=McAfee;i="6600,9927,11068"; a="15100813" X-IronPort-AV: E=Sophos;i="6.08,150,1712646000"; d="scan'208";a="15100813" Received: from orviesa004.jf.intel.com ([10.64.159.144]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2024 00:21:27 -0700 X-CSE-ConnectionGUID: NGjGMmOzRKGiDzTBH8sdAA== X-CSE-MsgGUID: iRd8PrDdQiumgMtOOS1CwA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,150,1712646000"; d="scan'208";a="34381402" Received: from unknown (HELO mev-dev) ([10.237.112.144]) by orviesa004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2024 00:21:24 -0700 Date: Fri, 10 May 2024 09:20:51 +0200 From: Michal Swiatkowski To: Jiri Pirko Message-ID: References: <20240507114516.9765-1-michal.swiatkowski@linux.intel.com> <20240507114516.9765-7-michal.swiatkowski@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1715325687; x=1746861687; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=GStYpJUsqp/2cKsgNBZ5G3Yc4IcXr2kIMIWZMjevH+4=; b=dR1JyYqr6r2qRxwqDh09kozJyt3+q8uCnDa47zhOAAhwFC2gV1kjTN1q +LwTAMMRl9/wNzJSYFhtphWuW9THH5tM3N/1eRv0KulCGDc/zrYkSqzjH G2jNLJSvPFXh7QfzcxqtpB0b+BYkfjsFaSjZ3yaAR0jpQic5ZUHlH65yf XfQo3uWxAOWt9fRQ4EzN6bC5/9FIfIyMBdKdDpIozy1D5OhCzKKfbzCIp ZVDdNJbCDiFYgrVJOlHoEvoBI9tuj7yZuIAzrY2Qoy0lMrlg7jzBPCIec fzrLMgEPRlDNkdhAH5gjbsUzSgat8MIa7/elc8nCqmBFMTQCUYEM4Qnvw g==; X-Mailman-Original-Authentication-Results: smtp3.osuosl.org; dmarc=none (p=none dis=none) header.from=linux.intel.com X-Mailman-Original-Authentication-Results: smtp3.osuosl.org; dkim=pass (2048-bit key, unprotected) header.d=intel.com header.i=@intel.com header.a=rsa-sha256 header.s=Intel header.b=dR1JyYqr Subject: Re: [Intel-wired-lan] [iwl-next v1 06/14] ice: base subfunction aux driver X-BeenThere: intel-wired-lan@osuosl.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Wired Ethernet Linux Kernel Driver Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: shayd@nvidia.com, maciej.fijalkowski@intel.com, mateusz.polchlopek@intel.com, netdev@vger.kernel.org, jiri@nvidia.com, michal.kubiak@intel.com, intel-wired-lan@lists.osuosl.org, pio.raczynski@gmail.com, sridhar.samudrala@intel.com, jacob.e.keller@intel.com, wojciech.drewek@intel.com, przemyslaw.kitszel@intel.com Errors-To: intel-wired-lan-bounces@osuosl.org Sender: "Intel-wired-lan" On Thu, May 09, 2024 at 01:13:55PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:07PM CEST, michal.swiatkowski@linux.intel.com wrote: > >From: Piotr Raczynski > > > >Implement subfunction driver. It is probe when subfunction port is > >activated. > > > >VSI is already created. During the probe VSI is being configured. > >MAC unicast and broadcast filter is added to allow traffic to pass. > > > >Store subfunction pointer in VSI struct. The same is done for VF > >pointer. Make union of subfunction and VF pointer as only one of them > >can be set with one VSI. > > > >Reviewed-by: Jiri Pirko > >Signed-off-by: Piotr Raczynski > >Signed-off-by: Michal Swiatkowski > > Perhaps it would make things clearer for reviewer to have all patches > related to sf auxdev/devlink/netdev at the end of the patchset, after > activation patch. Not sure why you want to mix it here. I need this code to use it in port representor implementation. You suggested in previous review to move activation at the end [1]. [1] https://lore.kernel.org/netdev/Zhje0mQgQTMXwICb@nanopsycho/ From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 910AF15EFB4 for ; Fri, 10 May 2024 07:21:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=192.198.163.12 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715325689; cv=none; b=GnVQwd0WnTgrbN887m2rMMAlRwnyY8X8qo7HhaxwygzRLY3TO04+YI3Jcf6MjhmJ85OsnzOg7TsMJsQ/FXWfCmzZeJkOI7leP28EZuVZd4S8T33jGaovdOYBwnCIkqKa0IwJT7zFVnk3tbGxgRZ/VzEqaX/2KE23bRNgj18EcHY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715325689; c=relaxed/simple; bh=GStYpJUsqp/2cKsgNBZ5G3Yc4IcXr2kIMIWZMjevH+4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=N2nP1ZAERmG+DnFO1z6pGfEnPFwIIz03kQcT8DpWjZMO7wzWIbUKyYb6pByIOsQtlLL8XyQTtFcH+yDOh/QmuvxMJ2Eb3s8IiSC/GwrnJIIJn3rpDWzUa1+Qi7D9xvEdOF4BoKlPg0hF+ZLgvPbJEsYU1a5PdwLFzOQDzRcHsa0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com; spf=none smtp.mailfrom=linux.intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=aek+N2dk; arc=none smtp.client-ip=192.198.163.12 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=linux.intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="aek+N2dk" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1715325688; x=1746861688; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=GStYpJUsqp/2cKsgNBZ5G3Yc4IcXr2kIMIWZMjevH+4=; b=aek+N2dkWInPHTcMr8OXapNjkFoGkBQnCvQNyYFQXwWABghIwDUVCcZU kVTUX70ZzBqgPCxVB1eKdsE84aWifkhekb/+dLiO3WTM/ljV0LoDkdslu dhsB1qHt8yWRz4WVrlV3bnKTAB6n6oDcYutSfFMKef3xYF683OEAHEDut 2z4oFb0FA89vGRfo/a/akuxEQO6p+tI2dk1r3HEHnkpGkToZkPH0iH+r4 FfiSRRfUHoHYOVhxB+abVMhoi7v9Ktvdnfp7Zr0u37n7KzlHOOJ+M/qDE psDj0szOOVjf7iDk5GsarFm6sWU+MxZC1sQfxo0HrvmVfoqWwe79z+YZT g==; X-CSE-ConnectionGUID: 34XcVHOWQymHKbrxCFNeQg== X-CSE-MsgGUID: kp0RtpllSj6wLwNEzVN7Bw== X-IronPort-AV: E=McAfee;i="6600,9927,11068"; a="15100811" X-IronPort-AV: E=Sophos;i="6.08,150,1712646000"; d="scan'208";a="15100811" Received: from orviesa004.jf.intel.com ([10.64.159.144]) by fmvoesa106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2024 00:21:27 -0700 X-CSE-ConnectionGUID: NGjGMmOzRKGiDzTBH8sdAA== X-CSE-MsgGUID: iRd8PrDdQiumgMtOOS1CwA== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.08,150,1712646000"; d="scan'208";a="34381402" Received: from unknown (HELO mev-dev) ([10.237.112.144]) by orviesa004-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 May 2024 00:21:24 -0700 Date: Fri, 10 May 2024 09:20:51 +0200 From: Michal Swiatkowski To: Jiri Pirko Cc: intel-wired-lan@lists.osuosl.org, netdev@vger.kernel.org, jacob.e.keller@intel.com, michal.kubiak@intel.com, maciej.fijalkowski@intel.com, sridhar.samudrala@intel.com, przemyslaw.kitszel@intel.com, wojciech.drewek@intel.com, pio.raczynski@gmail.com, jiri@nvidia.com, mateusz.polchlopek@intel.com, shayd@nvidia.com Subject: Re: [iwl-next v1 06/14] ice: base subfunction aux driver Message-ID: References: <20240507114516.9765-1-michal.swiatkowski@linux.intel.com> <20240507114516.9765-7-michal.swiatkowski@linux.intel.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Thu, May 09, 2024 at 01:13:55PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:07PM CEST, michal.swiatkowski@linux.intel.com wrote: > >From: Piotr Raczynski > > > >Implement subfunction driver. It is probe when subfunction port is > >activated. > > > >VSI is already created. During the probe VSI is being configured. > >MAC unicast and broadcast filter is added to allow traffic to pass. > > > >Store subfunction pointer in VSI struct. The same is done for VF > >pointer. Make union of subfunction and VF pointer as only one of them > >can be set with one VSI. > > > >Reviewed-by: Jiri Pirko > >Signed-off-by: Piotr Raczynski > >Signed-off-by: Michal Swiatkowski > > Perhaps it would make things clearer for reviewer to have all patches > related to sf auxdev/devlink/netdev at the end of the patchset, after > activation patch. Not sure why you want to mix it here. I need this code to use it in port representor implementation. You suggested in previous review to move activation at the end [1]. [1] https://lore.kernel.org/netdev/Zhje0mQgQTMXwICb@nanopsycho/