Cover Detail
Show a cover letter.
GET /api/covers/417/?format=api
{ "id": 417, "url": "http://patches.dpdk.org/api/covers/417/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/20231117131824.1977792-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": "<20231117131824.1977792-1-david.marchand@redhat.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20231117131824.1977792-1-david.marchand@redhat.com", "date": "2023-11-17T13:18:21", "name": "[RFC,0/3] Detect superfluous newline in logs", "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/20231117131824.1977792-1-david.marchand@redhat.com/mbox/", "series": [ { "id": 30328, "url": "http://patches.dpdk.org/api/series/30328/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=30328", "date": "2023-11-17T13:18:21", "name": "Detect superfluous newline in logs", "version": 1, "mbox": "http://patches.dpdk.org/series/30328/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/417/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 0081243354;\n\tFri, 17 Nov 2023 14:18:39 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id B7ED840ED1;\n\tFri, 17 Nov 2023 14:18:39 +0100 (CET)", "from us-smtp-delivery-124.mimecast.com\n (us-smtp-delivery-124.mimecast.com [170.10.133.124])\n by mails.dpdk.org (Postfix) with ESMTP id A2C1C40648\n for <dev@dpdk.org>; Fri, 17 Nov 2023 14:18:38 +0100 (CET)", "from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com\n [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS\n (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id\n us-mta-58-_pFRmqsgM9mVBzKZtT6RjA-1; Fri, 17 Nov 2023 08:18:34 -0500", "from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com\n [10.11.54.3])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n (No client certificate requested)\n by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 60926101A52D;\n Fri, 17 Nov 2023 13:18:34 +0000 (UTC)", "from dmarchan.redhat.com (unknown [10.45.226.112])\n by smtp.corp.redhat.com (Postfix) with ESMTP id 3D1001121306;\n Fri, 17 Nov 2023 13:18:33 +0000 (UTC)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com;\n s=mimecast20190719; t=1700227118;\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 bh=ejdJVGFd2Yo8zd7FxsR95jz369X1XEXAIoK1GqKaGos=;\n b=D05ucExsDpkm1n7mXnJIj6Z6E1BsWsJyA27UC2fUCQt77ciHEzw9JpKnGzjxid0sbV+ClT\n vtJMGc4HElpA0wqKd6JMEAVh57fxV2tIQtL2TxMO4zyA7zEs4mR+wo4U4KwdOoL3ukUim7\n aFus0bqxdA4nPfKc4DbGw3TgxF0M+oo=", "X-MC-Unique": "_pFRmqsgM9mVBzKZtT6RjA-1", "From": "David Marchand <david.marchand@redhat.com>", "To": "dev@dpdk.org", "Cc": "thomas@monjalon.net, ferruh.yigit@amd.com, bruce.richardson@intel.com,\n stephen@networkplumber.org", "Subject": "[RFC 0/3] Detect superfluous newline in logs", "Date": "Fri, 17 Nov 2023 14:18:21 +0100", "Message-ID": "<20231117131824.1977792-1-david.marchand@redhat.com>", "MIME-Version": "1.0", "X-Scanned-By": "MIMEDefang 3.4.1 on 10.11.54.3", "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": "Getting readable and consistent logs is important when running a DPDK\napplication, especially when troubleshooting.\nA common issue with logs is when a DPDK change do not add (or on the\ncontrary add too many \\n) in the format string.\n\nThis issue would only get noticed when actually hitting this log (which\nmay be something difficult to do).\n\nThis series proposes to introduce a new RTE_LOG helper that is\nresponsible for logging a one line message and spews a build error (with\ngcc) if any \\n is part of the format string.\n\n\nNote:\n- the first patch is intentionnally sent as a single block: splitting it\n into per library commits with correct Fixes: tags is a tedious work.\n I would split it for a non RFC series. For now, it is enough to show\n case the idea.\n- the last patch shows how an existing log macro is converted," }