From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (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 AA78B13DDA4 for ; Thu, 12 Mar 2026 11:59:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=170.10.129.124 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773316800; cv=none; b=Kh8ybMIYBNbc9L7UaGGKGXXLEL5iUXAt8K+uLiHjM4Hv/rzjfmOs7kOe3TPi6s9WU+Zs6skXZRopjF1KbgFnf1vt5dIxlPBrNBud+eiZbQXqE28mk54OXHg0y+ODtfn1oXaUHp54Keo8BRYVbQOGSbHcqG6yz08otbzNpJW79kQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773316800; c=relaxed/simple; bh=pSM4c5SlwbfLlCY6MVDWcwT0Dj8gikrgvwZsevPv/bo=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=bJjal3vUYyJOspizexzAPLRLIKPZpKpn8jGTDkhlt2mPe0YCsx8Abr4kaWlZv2QdZdK/O/xRIWMX0/yJqHCvQHjHbqoAMyXoqX4J6pGmfTykvkmAfBPy2VO0r+KHzRY+PCKs/6u4R4e78l+ahomEUm0WdtZTI8+zmS464bi7MdM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com; spf=pass smtp.mailfrom=redhat.com; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b=I5haxr+Y; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b=l5gq7hCh; arc=none smtp.client-ip=170.10.129.124 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=redhat.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=redhat.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="I5haxr+Y"; dkim=pass (2048-bit key) header.d=redhat.com header.i=@redhat.com header.b="l5gq7hCh" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1773316798; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=oRa4LvyU+A60q9Rpv8+AHl1L4aAB6JpU7wa5lWrjEDs=; b=I5haxr+YCJowGQKZoRfeL6fGlOjMIFmRfQEApwqIFvlD9dStQe67UoZrY420W2rNLc8ZGx 37HBKL2ISUjPAyHHWOv0bJmiWGd7VzPfN2fYs2X61v0zR0BXmwB8fum39ojVxNOm7DLPp9 eO6PrUyeD1ARKwf1oDGzkjub+VtJozc= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-189-sz14nCtePKCwUYEG5JQGYg-1; Thu, 12 Mar 2026 07:59:57 -0400 X-MC-Unique: sz14nCtePKCwUYEG5JQGYg-1 X-Mimecast-MFC-AGG-ID: sz14nCtePKCwUYEG5JQGYg_1773316797 Received: by mail-wm1-f71.google.com with SMTP id 5b1f17b1804b1-48532df52c5so8246965e9.1 for ; Thu, 12 Mar 2026 04:59:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=google; t=1773316796; x=1773921596; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=oRa4LvyU+A60q9Rpv8+AHl1L4aAB6JpU7wa5lWrjEDs=; b=l5gq7hChyeyzzxYeDL5J57Hfkt4Cb3oU6QSJ1DO8hGkYINELnShHDbW3xYtnXVvUM2 DOI1K1olalkbVal53hbpC+PfqWzCA2QQNPaNDNJb5uCBqOonCvJscUafKYQAyV3QdXhk Si2Nm+OVwaCW7imf/+uKTI/jUS3VaTSiS7m7ogMXSVddn69LTorpEosTC7bTzc2kF4Qy FZbYIP09mSkHZSN0BlGsMK2bqSItt3ufOoRunq4UAPh+IDH61OcKD6kMcsehr+vxtrnS B6/zDKAqtgweYBBc7WahGF4Py+szSz1YsGF6hty+vAnj1oTNzX1yJdZjLJehOTywbkiA 8yMg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1773316796; x=1773921596; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=oRa4LvyU+A60q9Rpv8+AHl1L4aAB6JpU7wa5lWrjEDs=; b=TzWFn3DzT1FHn0KA+pVFsg57bbhU2VDifA52LLTkeKdmzdkSUjzUMguLExg9ON6Sms I74zPjjpafpAoXnRZ6410jUBAP2MvKUafr/KRp0FgUwkQE/y1mNIXgOoLbMnkF8AxZMo rXfhwNSRr/7j4HvIiFyyjWMa4FOq911D1wwuswCnramtbzdd4+yEScAAm0AC93kIiC4E ZQQXxVksl1zy2/NSnWAmNSovQZ6TV/2ERTUYLWmcpGQBCr/12al4g/YsbrbAtsQy4wCm geEJ2ScEeea3mPVBzBhCAdZJCAiFjynh7g8Qo1HC9h7S1fgPmVI1PVIvZ4xXpvmf+ZUn WrFw== X-Forwarded-Encrypted: i=1; AJvYcCW4+tq0LfGtM7k6stM3nFm/yro+XW2cW2CWjsgsazFtWi2AO2WCfLZQjsyxsoe7sr/z0oQKVrLHWl0wndk=@vger.kernel.org X-Gm-Message-State: AOJu0YyfCYFMGStRdEwbAIlNClKh2b1I+4ExeN50Xsz14cNAPI09gU6i Q8q//SaN/yMvgybgpVUgffdRiWHh4xJIDhhAMer9xTblQAmdMUBNR8SQgau6fJp8biO1XhHuExA p2LMdD4imesQkc5w8C7XBqInWedjgUrs2lYSYLiOYR47Ml/v+7AMCvByw3yUuBPhLD02XelCBJw == X-Gm-Gg: ATEYQzwOl6FIcmwDFYyHpjDyK3KZzsmmn/0QAYQWrSDSxDBguzx/MQhtRJZlT9nmcaJ RtprJu41TUfk7+vg1KQy6CoJZFvlmyGzRhmU92uCvaUbszT/GGCuHD/94hOMoYw8daRYk+ldaN9 j8dxihp8AFiugA8RBZX7T7/qF9GyC+LPBpsLdm0b5Se4Uri/NVzqkDNENG7dhyCBYeOozmxoGq+ 4OZFvg81DH68pB7SgENBOYyfb/uuSpOJpt91zfVBf65M4FTbMGMfy+mdufKqy7iNxdVapHIU7Kf MU/mxITv96icCVqSbz8pyP5BlIKHRJGL2S8Y0CPmc6cQ2vXpFt909Snu7cfedBrTurvjrIS2uIw IJW1JZ86RPteKGFCcz6oZXEs6W0y0dHNAbQzqaPzsmodJa5vygDD8Fk8= X-Received: by 2002:a05:600c:4ed0:b0:485:40a6:442e with SMTP id 5b1f17b1804b1-4854ad71905mr116049295e9.0.1773316795929; Thu, 12 Mar 2026 04:59:55 -0700 (PDT) X-Received: by 2002:a05:600c:4ed0:b0:485:40a6:442e with SMTP id 5b1f17b1804b1-4854ad71905mr116048315e9.0.1773316795201; Thu, 12 Mar 2026 04:59:55 -0700 (PDT) Received: from [192.168.88.32] ([216.128.11.95]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4854b6070acsm106712865e9.8.2026.03.12.04.59.53 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 12 Mar 2026 04:59:54 -0700 (PDT) Message-ID: Date: Thu, 12 Mar 2026 12:59:53 +0100 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v7 net-next 06/11] net/nebula-matrix: add common resource implementation To: "illusion.wang" , dimon.zhao@nebula-matrix.com, alvin.wang@nebula-matrix.com, sam.chen@nebula-matrix.com, netdev@vger.kernel.org Cc: andrew+netdev@lunn.ch, corbet@lwn.net, kuba@kernel.org, linux-doc@vger.kernel.org, lorenzo@kernel.org, horms@kernel.org, vadim.fedorenko@linux.dev, lukas.bulwahn@redhat.com, edumazet@google.com, open list References: <20260310120959.22015-1-illusion.wang@nebula-matrix.com> <20260310120959.22015-7-illusion.wang@nebula-matrix.com> Content-Language: en-US From: Paolo Abeni In-Reply-To: <20260310120959.22015-7-illusion.wang@nebula-matrix.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On 3/10/26 1:09 PM, illusion.wang wrote: > +int nbl_res_vsi_id_to_pf_id(struct nbl_resource_mgt *res_mgt, u16 vsi_id) > +{ > + struct nbl_vsi_info *vsi_info = res_mgt->resource_info->vsi_info; > + struct nbl_common_info *common = res_mgt->common; > + bool vsi_find = false; > + u32 pf_id = U32_MAX; > + int i, j; > + > + for (i = 0; i < vsi_info->num; i++) { > + for (j = 0; j < NBL_VSI_SERV_MAX_TYPE; j++) > + if (vsi_id >= vsi_info->serv_info[i][j].base_id && > + (vsi_id < vsi_info->serv_info[i][j].base_id + > + vsi_info->serv_info[i][j].num)) { > + vsi_find = true; > + break; > + } > + > + if (vsi_find) > + break; > + } > + > + if (vsi_find) { > + if (j >= NBL_VSI_SERV_PF_DATA_TYPE && > + j <= NBL_VSI_SERV_PF_DATA_TYPE) > + pf_id = i + common->mgt_pf; AI review says: This isn't a bug, but the condition checks if j is >= NBL_VSI_SERV_PF_DATA_TYPE && j <= NBL_VSI_SERV_PF_DATA_TYPE, which is unnecessarily complex. This is equivalent to checking if j == NBL_VSI_SERV_PF_DATA_TYPE. The confusing comment was removed in v5, but the redundant comparison logic remains. Could this be simplified to: if (j == NBL_VSI_SERV_PF_DATA_TYPE) This was previously noted by Jakub Kicinski in v4 review. Reference: https://lore.kernel.org/netdev/20260207022455.4017894-1-kuba@kernel.org/