[dpdk-dev] [PATCH] eal: add missing include to debug header

Ferruh Yigit ferruh.yigit at intel.com
Thu May 5 18:07:58 CEST 2016


On 5/5/2016 1:59 PM, Bruce Richardson wrote:
> On Wed, May 04, 2016 at 05:37:56PM +0100, Bruce Richardson wrote:
>> The header file rte_debug.h makes use of the "unlikely" macro which
>> means it should include the rte_branch_prediction.h header file.
>>
>> Fixes: 50705e8e3cdd ("eal: add assert macro for debug")
>>
>> Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
>> ---
>>  lib/librte_eal/common/include/rte_debug.h | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/librte_eal/common/include/rte_debug.h b/lib/librte_eal/common/include/rte_debug.h
>> index 9260eda..cab6fb4 100644
>> --- a/lib/librte_eal/common/include/rte_debug.h
>> +++ b/lib/librte_eal/common/include/rte_debug.h
>> @@ -44,6 +44,7 @@
>>   */
>>  
>>  #include "rte_log.h"
>> +#include "rte_branch_prediction.h"
>>  
>>  #ifdef __cplusplus
>>  extern "C" {
> 
> Ping on this patch - any comments or concerns?
> It's a fairly trivial fix that prevents issues with testing some other patches
> on the mainline.
> 

Tested on top of David Marchand's 12123-12139 patchset.

Acked-by: Ferruh Yigit <ferruh.yigit at intel.com>



More information about the dev mailing list