From patchwork Mon Nov 24 15:51:02 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sujith Sankar X-Patchwork-Id: 1491 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id E91C280B0; Mon, 24 Nov 2014 16:40:19 +0100 (CET) Received: from rcdn-iport-3.cisco.com (rcdn-iport-3.cisco.com [173.37.86.74]) by dpdk.org (Postfix) with ESMTP id 325BF80A7 for ; Mon, 24 Nov 2014 16:40:16 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=5615; q=dns/txt; s=iport; t=1416844265; x=1418053865; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=Ff29Hgze7flsCkFCKIBt7mS3j0Cbd7gETjl8yrZjbKI=; b=J7AYNk8ekqVuQpMiKMkEMWoMGb8MU6ph5hSrX3xO4T5Wdc29pjJ+4WUq Ci7ix7u4xSmT1w23LpC7DXRDBk0155sTp9UnWoYcfPo6Y94+LK6IHFUP0 iygsADOhq26AzA8gA/KnZP3YuXucitd/0OHaGtnlJcOUvouzbEsm8ottX 8=; X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: AjQFAHdTc1StJV2d/2dsb2JhbABbgkhGgS4E0gACgR0WAQEBAQF9hAIBAQEEJ1IQAgEIEQMBAigHMhQJCAIEDgWIQdAXAQEBAQEBAQECAQEBAQEBAQEBAQEXkHoRB4ROBZA6gi6HN4RilxqDfXiBCCQcgQMBAQE X-IronPort-AV: E=Sophos;i="5.07,449,1413244800"; d="scan'208,217";a="374838927" Received: from rcdn-core-6.cisco.com ([173.37.93.157]) by rcdn-iport-3.cisco.com with ESMTP; 24 Nov 2014 15:51:04 +0000 Received: from xhc-rcd-x10.cisco.com (xhc-rcd-x10.cisco.com [173.37.183.84]) by rcdn-core-6.cisco.com (8.14.5/8.14.5) with ESMTP id sAOFp3b7004879 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Mon, 24 Nov 2014 15:51:03 GMT Received: from xmb-aln-x07.cisco.com ([169.254.2.173]) by xhc-rcd-x10.cisco.com ([173.37.183.84]) with mapi id 14.03.0195.001; Mon, 24 Nov 2014 09:51:03 -0600 From: "Sujith Sankar (ssujith)" To: David Marchand Thread-Topic: [dpdk-dev] [PATCH v3 6/6] DPDK changes for accommodating ENIC PMD Thread-Index: AQHQBuhEwvzmIS/5eU6QDhmBVMOFxJxwA1MAgACsiYA= Date: Mon, 24 Nov 2014 15:51:02 +0000 Message-ID: References: <1416758899-1351-1-git-send-email-ssujith@cisco.com> <1416758899-1351-7-git-send-email-ssujith@cisco.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: user-agent: Microsoft-MacOutlook/14.3.9.131030 x-originating-ip: [10.65.69.234] MIME-Version: 1.0 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" , "Prasad Rao \(prrao\)" Subject: Re: [dpdk-dev] [PATCH v3 6/6] DPDK changes for accommodating ENIC PMD X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list 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" Hi David, During the testing, I saw that the length field was 0. ENIC PMD validates the length of the BAR against a max value. In order to get the length in the resource structure, I added this statement. Thanks, -Sujith From: David Marchand > Date: Monday, 24 November 2014 4:33 pm To: "Sujith Sankar (ssujith)" > Cc: "dev@dpdk.org" >, "Prasad Rao (prrao)" > Subject: Re: [dpdk-dev] [PATCH v3 6/6] DPDK changes for accommodating ENIC PMD Hello Sujith, On Sun, Nov 23, 2014 at 5:08 PM, Sujith Sankar > wrote: Not sure I understand why you need to overwrite the length value. This is supposed to be initialised before by "generic" code. This looks like a hack or a workaround. Can you elaborate on this change ? Thanks. --- David Marchand diff --git a/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c b/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c index c776ddc..6bf8f2e 100644 --- a/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c +++ b/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c @@ -736,6 +736,7 @@ pci_vfio_map_resource(struct rte_pci_device *dev) maps[i].offset = reg.offset; maps[i].size = reg.size; dev->mem_resource[i].addr = bar_addr; + dev->mem_resource[i].len = reg.size; } /* if secondary process, do not set up interrupts */