From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path: <dev-bounces@dpdk.org>
Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124])
by inbox.dpdk.org (Postfix) with ESMTP id 3595DA052A;
Sun, 24 Jan 2021 21:52:14 +0100 (CET)
Received: from [217.70.189.124] (localhost [127.0.0.1])
by mails.dpdk.org (Postfix) with ESMTP id D4ED5140D49;
Sun, 24 Jan 2021 21:52:08 +0100 (CET)
Received: from mail-lj1-f171.google.com (mail-lj1-f171.google.com
[209.85.208.171])
by mails.dpdk.org (Postfix) with ESMTP id 717FB140D43
for <dev@dpdk.org>; Sun, 24 Jan 2021 21:52:07 +0100 (CET)
Received: by mail-lj1-f171.google.com with SMTP id i17so12902132ljn.1
for <dev@dpdk.org>; Sun, 24 Jan 2021 12:52:07 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;
h=from:to:cc:subject:date:message-id:in-reply-to:references
:mime-version:content-transfer-encoding;
bh=+wwEn0uvC6ZwE2YT5goyQUlpvw6CXN6BT2E3Ec/kmAM=;
b=spVUUFUXpX2pWBohQXBsTXaBEs89gU7RdtWmZFTwks55Gn1o0vP1zbWR4ao7OrONIB
WrFXz4sJvPprK9TkQuRzLvFQnSOE9t5zdDW26RUBXzXXN9pmwTh3a2jzZ/S7YGrCELMZ
N6g8i6EcER/f2DxG1nnO3tZS93uALIzS1FF0rP/o4z9WmRBaXZCWkohnCmvbQZghsUBB
dYfD9j6fNZ47dXQjndW7AN+/6dJEvD9zAAxFAIa0Aj/xwUm8NncMfrFZi6beUt/RufqC
0i+y9Tt3kT2O9QkidpUk8rTbaOcFnFCDNjS/K7CCkq5H6u6GnY3PX8N4Sw5FLH8we2BU
hbfA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
d=1e100.net; s=20161025;
h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to
:references:mime-version:content-transfer-encoding;
bh=+wwEn0uvC6ZwE2YT5goyQUlpvw6CXN6BT2E3Ec/kmAM=;
b=dizJT2an3U9xgxj9B+gRhvOlTMM5msJ05wCxlUEIRPdpKaemfymjz8Uo+7PS1yMYy/
QQhA6hBvkdFH3yOBBZlmsy6esMDznHgqRmVDvfUQ5hGiyCO4wXRIK8qqaGnOyp9uXhJz
6C6SExUMOXoTY0SpEzpUX2lHuShmvGS/cVYody0fKehDZnPqKWJ4C6WaUapQsPxedkgS
4L11dt2E6UdGBY6e1aMKiuIo72GPSwJoAs8re9Ft3R12S839VWwyH/FBZUEN5MF+eIka
+KCUlAurNDmBA4q5lNWhwY6DAi7HgESXcgraC2Je9G+J7cSjda6tjdpuClreyPsEqkvP
etpA==
X-Gm-Message-State: AOAM533BkIQBl06zQqLAloVRwY28IUKSCK4+xhy5L+RZgc92qdvxcSty
eDyrHQ2QLCGAPAV+TorPkHV7EcZrehHWRQ==
X-Google-Smtp-Source: ABdhPJyoFXczD2IGe1zHHE4O7F37ZCOl6E26Lfumi7gweOwJF3EdbmZXVURHqHVl0wtylTwEmu9KBA==
X-Received: by 2002:a05:651c:8d:: with SMTP id 13mr272768ljq.33.1611521526578;
Sun, 24 Jan 2021 12:52:06 -0800 (PST)
Received: from localhost.localdomain (broadband-37-110-65-23.ip.moscow.rt.ru.
[37.110.65.23])
by smtp.gmail.com with ESMTPSA id r72sm1168551lff.214.2021.01.24.12.52.05
(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
Sun, 24 Jan 2021 12:52:05 -0800 (PST)
From: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
To: dev@dpdk.org
Cc: Maxime Coquelin <maxime.coquelin@redhat.com>,
Bruce Richardson <bruce.richardson@intel.com>,
Thomas Monjalon <thomas@monjalon.net>,
Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>,
Neil Horman <nhorman@tuxdriver.com>
Date: Sun, 24 Jan 2021 23:51:55 +0300
Message-Id: <20210124205157.14386-2-dmitry.kozliuk@gmail.com>
X-Mailer: git-send-email 2.29.2
In-Reply-To: <20210124205157.14386-1-dmitry.kozliuk@gmail.com>
References: <20210122224327.5621-1-dmitry.kozliuk@gmail.com>
<20210124205157.14386-1-dmitry.kozliuk@gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Subject: [dpdk-dev] [PATCH v10 1/3] pmdinfogen: add Python implementation
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
<mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
<mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>
Using a high-level, interpreted language simplifies maintenance and
build process. Furthermore, ELF handling is delegated to pyelftools
package. Original logic is kept, the copyright recognizes that.
Signed-off-by: Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
---
buildtools/pmdinfogen.py | 189 +++++++++++++++++++++++++++++++++++++++
1 file changed, 189 insertions(+)
create mode 100755 buildtools/pmdinfogen.py
diff --git a/buildtools/pmdinfogen.py b/buildtools/pmdinfogen.py
new file mode 100755
index 000000000..0cca47ff1
--- /dev/null
+++ b/buildtools/pmdinfogen.py
@@ -0,0 +1,189 @@
+#!/usr/bin/env python3
+# SPDX-License-Identifier: BSD-3-Clause
+# Copyright (c) 2016 Neil Horman <nhorman@tuxdriver.com>
+# Copyright (c) 2020 Dmitry Kozlyuk <dmitry.kozliuk@gmail.com>
+
+import argparse
+import ctypes
+import json
+import sys
+import tempfile
+
+from elftools.elf.elffile import ELFFile
+from elftools.elf.sections import SymbolTableSection
+
+
+class ELFSymbol:
+ def __init__(self, image, symbol):
+ self._image = image
+ self._symbol = symbol
+
+ @property
+ def size(self):
+ return self._symbol["st_size"]
+
+ @property
+ def value(self):
+ data = self._image.get_section_data(self._symbol["st_shndx"])
+ base = self._symbol["st_value"]
+ return data[base:base + self.size]
+
+ @property
+ def string_value(self):
+ value = self.value
+ return value[:-1].decode() if value else ""
+
+
+class ELFImage:
+ def __init__(self, data):
+ self._image = ELFFile(data)
+ self._symtab = self._image.get_section_by_name(".symtab")
+ if not isinstance(self._symtab, SymbolTableSection):
+ raise Exception(".symtab section is not a symbol table")
+
+ @property
+ def is_big_endian(self):
+ return not self._image.little_endian
+
+ def get_section_data(self, name):
+ return self._image.get_section(name).data()
+
+ def find_by_name(self, name):
+ symbol = self._symtab.get_symbol_by_name(name)
+ return ELFSymbol(self, symbol[0]) if symbol else None
+
+ def find_by_prefix(self, prefix):
+ for i in range(self._symtab.num_symbols()):
+ symbol = self._symtab.get_symbol(i)
+ if symbol.name.startswith(prefix):
+ yield ELFSymbol(self, symbol)
+
+
+def define_rte_pci_id(is_big_endian):
+ base_type = ctypes.LittleEndianStructure
+ if is_big_endian:
+ base_type = ctypes.BigEndianStructure
+
+ class rte_pci_id(base_type):
+ _pack_ = True
+ _fields_ = [
+ ("class_id", ctypes.c_uint32),
+ ("vendor_id", ctypes.c_uint16),
+ ("device_id", ctypes.c_uint16),
+ ("subsystem_vendor_id", ctypes.c_uint16),
+ ("subsystem_device_id", ctypes.c_uint16),
+ ]
+
+ return rte_pci_id
+
+
+class Driver:
+ OPTIONS = [
+ ("params", "_param_string_export"),
+ ("kmod", "_kmod_dep_export"),
+ ]
+
+ def __init__(self, name, options):
+ self.name = name
+ for key, value in options.items():
+ setattr(self, key, value)
+ self.pci_ids = []
+
+ @classmethod
+ def load(cls, image, symbol):
+ name = symbol.string_value
+
+ options = {}
+ for key, suffix in cls.OPTIONS:
+ option_symbol = image.find_by_name("__%s%s" % (name, suffix))
+ if option_symbol:
+ value = option_symbol.string_value
+ options[key] = value
+
+ driver = cls(name, options)
+
+ pci_table_name_symbol = image.find_by_name("__%s_pci_tbl_export" % name)
+ if pci_table_name_symbol:
+ driver.pci_ids = cls._load_pci_ids(image, pci_table_name_symbol)
+
+ return driver
+
+ @staticmethod
+ def _load_pci_ids(image, table_name_symbol):
+ table_name = table_name_symbol.string_value
+ table_symbol = image.find_by_name(table_name)
+ if not table_symbol:
+ raise Exception("PCI table declared but not defined: %d" % table_name)
+
+ rte_pci_id = define_rte_pci_id(image.is_big_endian)
+
+ pci_id_size = ctypes.sizeof(rte_pci_id)
+ pci_ids_desc = rte_pci_id * (table_symbol.size // pci_id_size)
+ pci_ids = pci_ids_desc.from_buffer_copy(table_symbol.value)
+ result = []
+ for pci_id in pci_ids:
+ if not pci_id.device_id:
+ break
+ result.append([
+ pci_id.vendor_id,
+ pci_id.device_id,
+ pci_id.subsystem_vendor_id,
+ pci_id.subsystem_device_id,
+ ])
+ return result
+
+ def dump(self, file):
+ dumped = json.dumps(self.__dict__)
+ escaped = dumped.replace('"', '\\"')
+ print(
+ 'const char %s_pmd_info[] __attribute__((used)) = "PMD_INFO_STRING= %s";'
+ % (self.name, escaped),
+ file=file,
+ )
+
+
+def load_drivers(image):
+ drivers = []
+ for symbol in image.find_by_prefix("this_pmd_name"):
+ drivers.append(Driver.load(image, symbol))
+ return drivers
+
+
+def dump_drivers(drivers, file):
+ # Keep legacy order of definitions.
+ for driver in reversed(drivers):
+ driver.dump(file)
+
+
+def parse_args():
+ parser = argparse.ArgumentParser()
+ parser.add_argument("input", help="input object file path or '-' for stdin")
+ parser.add_argument("output", help="output C file path or '-' for stdout")
+ return parser.parse_args()
+
+
+def open_input(path):
+ if path == "-":
+ temp = tempfile.TemporaryFile()
+ temp.write(sys.stdin.buffer.read())
+ return temp
+ return open(path, "rb")
+
+
+def open_output(path):
+ if path == "-":
+ return sys.stdout
+ return open(path, "w")
+
+
+def main():
+ args = parse_args()
+ infile = open_input(args.input)
+ image = ELFImage(infile)
+ drivers = load_drivers(image)
+ output = open_output(args.output)
+ dump_drivers(drivers, output)
+
+
+if __name__ == "__main__":
+ main()
--
2.29.2