Cover Detail
Show a cover letter.
GET /api/covers/125752/?format=api
{ "id": 125752, "url": "http://patches.dpdk.org/api/covers/125752/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/20230404124840.1898-1-david.marchand@redhat.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20230404124840.1898-1-david.marchand@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20230404124840.1898-1-david.marchand@redhat.com", "date": "2023-04-04T12:48:24", "name": "[v3,00/16] Enable lock annotations on most libraries and drivers", "submitter": { "id": 1173, "url": "http://patches.dpdk.org/api/people/1173/?format=api", "name": "David Marchand", "email": "david.marchand@redhat.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/20230404124840.1898-1-david.marchand@redhat.com/mbox/", "series": [ { "id": 27617, "url": "http://patches.dpdk.org/api/series/27617/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=27617", "date": "2023-04-04T12:48:24", "name": "Enable lock annotations on most libraries and drivers", "version": 3, "mbox": "http://patches.dpdk.org/series/27617/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/125752/comments/", "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id B9902428CD;\n\tTue, 4 Apr 2023 14:48:56 +0200 (CEST)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 45CAE40EF0;\n\tTue, 4 Apr 2023 14:48:56 +0200 (CEST)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.129.124])\n by mails.dpdk.org (Postfix) with ESMTP id 5B4D640A7E\n for <dev@dpdk.org>; Tue, 4 Apr 2023 14:48:55 +0200 (CEST)", "from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com\n [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS\n (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n us-mta-583-Hjypc1TdMpKiFc9DPbIYOQ-1; Tue, 04 Apr 2023 08:48:53 -0400", "from smtp.corp.redhat.com (int-mx02.intmail.prod.int.rdu2.redhat.com\n [10.11.54.2])\n (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n (No client certificate requested)\n by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 201B42807D74;\n Tue, 4 Apr 2023 12:48:53 +0000 (UTC)", "from dmarchan.redhat.com (unknown [10.45.225.38])\n by smtp.corp.redhat.com (Postfix) with ESMTP id 3BC464020C82;\n Tue, 4 Apr 2023 12:48:52 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1680612534;\n h=from:from:reply-to:subject:subject:date:date:message-id:message-id:\n to:to:cc:cc:mime-version:mime-version:content-type:content-type:\n content-transfer-encoding:content-transfer-encoding:\n in-reply-to:in-reply-to:references:references;\n bh=YIzSAk2sOSEYed+sZcE+AYnG4ogBMXnLq9kqKuVOLso=;\n b=D9c4Pk4hESne9mt6yKAtBrEvWYzPsDloRbrEITQ3iQN2YuEonXFHjGJmQ8gMv+4nGfSvf1\n uEKbuUMnoR4SKaQZaSbcNDD1FY8T8Blame7gVlEorXkxFcO2bJCYV8D/T3rcFqa39xg4OQ\n O6Q4Qw4sPfHnK8euJXypwX/7ic2jTDo=", "X-MC-Unique": "Hjypc1TdMpKiFc9DPbIYOQ-1", "From": "David Marchand <david.marchand@redhat.com>", "To": "dev@dpdk.org", "Cc": "thomas@monjalon.net", "Subject": "[PATCH v3 00/16] Enable lock annotations on most libraries and\n drivers", "Date": "Tue, 4 Apr 2023 14:48:24 +0200", "Message-Id": "<20230404124840.1898-1-david.marchand@redhat.com>", "In-Reply-To": "<20230224081642.2566619-1-david.marchand@redhat.com>", "References": "<20230224081642.2566619-1-david.marchand@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.1 on 10.11.54.2", "X-Mimecast-Spam-Score": "0", "X-Mimecast-Originator": "redhat.com", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain; charset=\"US-ASCII\"; x-default=true", "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>,\n <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>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "This is a followup of the series that introduced lock annotations.\nI reworked and made annotations work in what seemed the easier cases.\nIn most cases, I chose to convert inline wrappers around the EAL lock\nAPI to simple macro: I did not see much value in those wrappers and this\nis way simpler than adding __rte_*lock_function tags everywhere.\n\nA list of libraries and drivers still need more work as their code have\nnon obvious locks handling. For those components, the check is opted\nout.\nI leave it to their respective maintainers to enable the checks later.\n\nFreeBSD libc pthread API has lock annotations while Linux glibc has\nnone. Until we get a proper abstraction for mutexes, the simpler is\nto disable the check on FreeBSD.\n\nMaintainers, please review." }