From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: vm_power_manager uses channel_commands.h which is not placed in installed copy of DPDK Date: Mon, 15 Feb 2016 13:56:42 +0200 Message-ID: <56C1BCFA.7050506@redhat.com> References: <0863DE89-580E-4AA2-B949-25179510BD55@mhcomputing.net> <56C18C78.7080307@redhat.com> <6087984.WHOrx6HCKo@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 3714ADE6 for ; Mon, 15 Feb 2016 12:56:45 +0100 (CET) In-Reply-To: <6087984.WHOrx6HCKo@xps13> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 02/15/2016 12:15 PM, Thomas Monjalon wrote: > 2016-02-15 10:29, Panu Matilainen: >> On 02/14/2016 01:38 AM, Matthew Hall wrote: >>> Hello, >>> >>> I found a peculiarity in the vm_power_manager example on DPDK 2.2 if you use an installed copy of DPDK to compile the examples instead of the master copy (while trying to update some outdated stuff in my build system). > [...] >>> Did anybody run into this before? >> >> Yes, the problem exists at least since DPDK 2.1 but somehow never >> remembered to report / do something about it. >> >> Basically either vm_power_manager example is messing with things its not >> supposed to, or librte_power library isn't exporting everything needed >> to fully use the it. Unfortunately both the library and the example are >> lacking a maintainer :-/ > > Exact, you got the point. > I suggest to disable this feature in your config. I take it you mean disabling librte_power entirely, not just the example? > If it's really annoying, you are allowed to request the removal of the code. > I believe someone would maintain it if he has an interest. I can certainly submit a patch to disable it by default in the configs (seems reasonable unmaintained code would default to off) and add a deprecation note along the lines of "if no maintainer steps up, this code will be removed in 16.07" if you like :) - Panu -