From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id BDA48A0548; Fri, 9 Jul 2021 21:24:50 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 39009410FD; Fri, 9 Jul 2021 21:24:50 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 3DFAF40143 for ; Fri, 9 Jul 2021 21:24:49 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1625858688; 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: in-reply-to:in-reply-to:references:references; bh=+X26GGPMNBh2ZHa25I0e6kAkO4s/Y/A81F/v/9v/TEI=; b=bpgXbrLbrrRvLLxQA5XcEWIAwu4juzBIgSAyUboasiUMAycUU1IhSdXMYFtrkN0AxtzMpD jEF7ee3vpRHn+/PgohHv7btTnpTxJfcPGNPT9UKAW0eOeLAxeYt/vcsJkY6QbXKDI/4/ZK X6qkrJLbeCHkx2Yq4NL+Sm90yngWeuA= Received: from mail-ua1-f70.google.com (mail-ua1-f70.google.com [209.85.222.70]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-366--NBO5uejOaur25v-agJy-w-1; Fri, 09 Jul 2021 15:24:45 -0400 X-MC-Unique: -NBO5uejOaur25v-agJy-w-1 Received: by mail-ua1-f70.google.com with SMTP id d7-20020ab066c70000b0290291b95bf303so3995306uaq.16 for ; Fri, 09 Jul 2021 12:24:44 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=+X26GGPMNBh2ZHa25I0e6kAkO4s/Y/A81F/v/9v/TEI=; b=DBX9E2exPaMI2cQOYmyDKLf8CAuRT0QqTf85DrTfRznQNsHsgeoEOTfv5N+dbhUbxE Ak1wL8ZSBWGxJMaENasLBiOOSPEgMPfxFwgEOORwp9bIqH+1P2NwrYGbylkhwm1+AFdO hWWyFN/Qk07bHh1RWFrWJJ4ghy1TOX5DLf0xxUu4KpY/qOKYthnHgam4TTIqUu964qwd 4Jm794SVfcpDgvHW38c4I+FGX8VM4MG0ch0qPGrQoTQQ/cTXok0RJK/VFClFqe0Ic+oG +y9/ghZbWu/HezG0bF3VJKQxVzCvHh0lJ3NuQxMKZtklqVLVXFw2briByrXFV7+OmewP /84Q== X-Gm-Message-State: AOAM533l4Lep+fWa6YJKkWnkpyDjtxHrS6kYciqCY0/jzEmEaEejxDBQ Z1g9js/wQOYUb637OsdCZOuIwyCzgckhc6xTWNd/navg2+5NBTUle4BbMH46jirjjw21aRHIIg5 SLwrnuHwA+XpzwkeV0aQ= X-Received: by 2002:a1f:a655:: with SMTP id p82mr26407616vke.20.1625858684457; Fri, 09 Jul 2021 12:24:44 -0700 (PDT) X-Google-Smtp-Source: ABdhPJykfGnJhk8N6nIlb4Go6d6plQpWb0G2IMN8pRw2e2MkrqEfX4HIWa+2DCbUfR2V/wdSh/uA+RyVOkGBUOW/3K0= X-Received: by 2002:a1f:a655:: with SMTP id p82mr26407602vke.20.1625858684196; Fri, 09 Jul 2021 12:24:44 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: David Marchand Date: Fri, 9 Jul 2021 21:24:32 +0200 Message-ID: To: Anatoly Burakov Cc: dev , David Hunt , "Ananyev, Konstantin" , Ciara Loftus , "Yigit, Ferruh" , Andrew Rybchenko , Thomas Monjalon Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=dmarchan@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH v10 0/8] Enhancements for PMD power management 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" On Fri, Jul 9, 2021 at 6:08 PM Anatoly Burakov wrote: > > This patchset introduces several changes related to PMD power management: > > - Changed monitoring intrinsics to use callbacks as a comparison function, based > on previous patchset [1] but incorporating feedback [2] - this hopefully will > make it possible to add support for .get_monitor_addr in virtio > - Add a new intrinsic to monitor multiple addresses, based on RTM instruction > set and the TPAUSE instruction > - Add support for PMD power management on multiple queues, as well as all > accompanying infrastructure and example apps changes > > v10: > - Added missing changes to NIC feature .ini files > > v9: > - Added all missing Acks and Tests > - Added a new commit with NIC features > - Addressed minor issues raised in review > > v8: > - Fixed checkpatch issue > - Added comment explaining empty poll handling (Konstantin) > > v7: > - Fixed various bugs > > v6: > - Improved the algorithm for multi-queue sleep > - Fixed segfault and addressed other feedback > > v5: > - Removed "power save queue" API and replaced with mechanism suggested by > Konstantin > - Addressed other feedback > > v4: > - Replaced raw number with a macro > - Fixed all the bugs found by Konstantin > - Some other minor corrections > > v3: > - Moved some doc updates to NIC features list > > v2: > - Changed check inversion to callbacks > - Addressed feedback from Konstantin > - Added doc updates where necessary > > [1] http://patches.dpdk.org/project/dpdk/list/?series=16930&state=* > [2] http://patches.dpdk.org/project/dpdk/patch/819ef1ace187365a615d3383e54579e3d9fb216e.1620747068.git.anatoly.burakov@intel.com/#133274 > > Anatoly Burakov (8): > eal: use callbacks for power monitoring comparison > net/af_xdp: add power monitor support > doc: add PMD power management NIC feature > eal: add power monitor for multiple events > power: remove thread safety from PMD power API's > power: support callbacks for multiple Rx queues > power: support monitoring multiple Rx queues > examples/l3fwd-power: support multiq in PMD modes Overall, the series lgtm. I still have a comment on the opaque pointer passed in callbacks. This is not blocking, we can still go with followup patches in this release. It would be great if drivers maintainers could implement this new ops in their driver or give feedback on what should be enhanced. Series applied, thanks. -- David Marchand