From patchwork Fri Oct 8 00:22:43 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: William Tu X-Patchwork-Id: 100739 X-Patchwork-Delegate: thomas@monjalon.net Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id F369CA0C43; Fri, 8 Oct 2021 02:23:05 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7B6BB40042; Fri, 8 Oct 2021 02:23:05 +0200 (CEST) Received: from mail-pf1-f180.google.com (mail-pf1-f180.google.com [209.85.210.180]) by mails.dpdk.org (Postfix) with ESMTP id E1C464003C for ; Fri, 8 Oct 2021 02:23:03 +0200 (CEST) Received: by mail-pf1-f180.google.com with SMTP id i65so3758591pfe.12 for ; Thu, 07 Oct 2021 17:23:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=TtC7GiooRM8/CywQUiG0LeLd9kd7ebaMkd8kZ6fFA/g=; b=gCUfnnQ0AO0Ab3QtHDtQ+8HgzEpWgxCMHPnpX8iwrq4LNZ0CQ1FjyJIWBjLCDc2h/h kXbJe84dRVZe4gOBGKLO2slcydkFEp0kihFcy98UbDeFhBEg9WBouSexMb0qtiF0Fiys 3bEBVJgYVP7kOaYQOw6NomAyw6NzkQO7Vwjlp1OwT7c5HWqUBgSIJKE6Y4T8/QbIpOkL +2WDOE0TTsMU5bgvgtTBCbB5Y1lULNq8pTy7lHMeX47TRQVV4k6SmnPohCXtV1tpX8sQ qa+C6d4mvKitdoxTPdvU5EoZdnCZw/6a8s3TkfkkOeKZE0LdPkyqrIQ5h9JQzv6N6ex5 4D4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=TtC7GiooRM8/CywQUiG0LeLd9kd7ebaMkd8kZ6fFA/g=; b=wwuQb5Fakw2F6nv4T4Cqe6/kuPpXp9mn/80OsnFITGPk7SKYUFRL+hJC3zCuKiI7yH alHKGHq3IOvi/N4VGo0rINFfwfb3poY+sagEXL7G7i2csRNZ0r+KiQ2dtDxKlAi9yKYo u4B2mnaYn0Dx8CmUKbAd2LxvtjL3C+JtgKnP6App6nkOePoqmnR2ECYImIbmL06DDXJq IQO+m6F4iJHNGCNPnHuYt5bS0XbvSxW+VhiUby541cmWFAvM0GJ8B654t+W+L7sVMotJ 4rMQ9LO0X/WFvqMQBsx93kuSmipuNNYw3L6D+iozqTWl/YOTOxsqIfHcQuDUSoMfMAhi xdeA== X-Gm-Message-State: AOAM5321YUDXfL53/140C3Q6ApFWJnJmnJSnAMhpP0D4eZwLIacK5Z7a ljau1O4EGoWX7k7YHBsO4EVhB1vbS4E= X-Google-Smtp-Source: ABdhPJyVNW7rjBpS9fZ8Qak6a/6ZAdxHz88URRug7vFlHrCC3OEBrzVW3yUpGudlfGkXvW0yLPIKaA== X-Received: by 2002:a63:4464:: with SMTP id t36mr2174696pgk.4.1633652582550; Thu, 07 Oct 2021 17:23:02 -0700 (PDT) Received: from tuc-a01.vmware.com.com (c-67-160-105-174.hsd1.wa.comcast.net. [67.160.105.174]) by smtp.gmail.com with ESMTPSA id k190sm448972pgc.11.2021.10.07.17.23.01 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 07 Oct 2021 17:23:01 -0700 (PDT) From: William Tu To: dev@dpdk.org Cc: navasile@microsoft.com, pallavi.kadam@intel.com, dmitry.kozliuk@gmail.com Date: Thu, 7 Oct 2021 17:22:43 -0700 Message-Id: <20211008002243.88955-1-u9012063@gmail.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH v1] windows/netuio: fix pci resouce translation X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" When BAR array is filled like this * Bar[4] = c844000c * Bar[5] = null ex: while ((descriptor->Type != CmResourceTypeMemory) || !(descriptor->Flags & CM_RESOURCE_MEMORY_BAR)); returns true, the loop continue and the decriptor will be NULL, but it's not an error. So I fix it by setting the status to STATUS_SUCCESS. Signed-off-by: William Tu --- Found the issue when adding the e1000 PMD support. http://inbox.dpdk.org/dev/20211007202451.90271-1-u9012063@gmail.com/T/#u --- windows/netuio/netuio_dev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/windows/netuio/netuio_dev.c b/windows/netuio/netuio_dev.c index b2deb10..0dd0055 100644 --- a/windows/netuio/netuio_dev.c +++ b/windows/netuio/netuio_dev.c @@ -207,7 +207,7 @@ netuio_map_hw_resources(WDFDEVICE Device, WDFCMRESLIST Resources, WDFCMRESLIST R next_descriptor++; if (descriptor == NULL) { - status = STATUS_DEVICE_CONFIGURATION_ERROR; + status = STATUS_SUCCESS; goto end; } } while ((descriptor->Type != CmResourceTypeMemory) ||