From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id DD7264601A; Mon, 20 Jan 2025 19:42:58 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CDF43402AC; Mon, 20 Jan 2025 19:42:58 +0100 (CET) Received: from mail-pj1-f45.google.com (mail-pj1-f45.google.com [209.85.216.45]) by mails.dpdk.org (Postfix) with ESMTP id 1B2154027A for ; Mon, 20 Jan 2025 19:42:56 +0100 (CET) Received: by mail-pj1-f45.google.com with SMTP id 98e67ed59e1d1-2eed82ca5b4so8159143a91.2 for ; Mon, 20 Jan 2025 10:42:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1737398576; x=1738003376; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=WfaRDmJ03Qm0kLb2CRHIFqfCIH5uL3ozSF74P5nyvLc=; b=UPfo8lc2iMfj2TLWpKKV6fHCfqmOtuqTI7HyU6nXoBu1msGkTVgZ3wLzhir6y1I9ko Jn8QKo7F7Y9+EzSRhs60dUhmUTpH/Kk5UfeM7yfwA8GtuyZuFPSGeekJ+OMUDcjoW+ST h4SnZXCRvIbycKpDquUeQGfEtidJvR/ItbR8ibXjnc0eJgcYLACDhY3V/05TbBRwft2H pyevahBH8tEmTc66S7D20aI0JhTT3bjEXKq128VwAEL5lvKDP05VXF4xBFZqVmHUtn7U 6eE8T5G5T4kzjv29vMhSE+zea0uENVOTq8G1XFh67z6vTBcvvl+W9lMG09E6L/6Rd/T2 OVQA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737398576; x=1738003376; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=WfaRDmJ03Qm0kLb2CRHIFqfCIH5uL3ozSF74P5nyvLc=; b=EMZM4YM+l74AQROIr1uy8SykbHte9oywkDACKExXY514lagUexxwf4BKTW6nPpLP3v 4B7f+u/uD4s/BQDVyM2HcBJKgD6ZIVjHrJkhyXR53hJDATMHDzonKZQUolq7Q9Y1HYSz T3ewsVv8BXvNr4nahlgLlRLTqegiTWj02dB9CUjqABA/oZNfRBnazLOe38sRsrvNHdB2 AMIdDLGjj1cx9ukDHDu8R7arVtTI0BHGthZv8fQYQlyUc7RGeiCogr2mK2MwI9baUJL6 HEb7+9Art/J8JC/HuRl8UHRfdMTtaEM27z1ct4CP4W9AR/6EnMmblQn7XrFwXTFpb2wP BN8g== X-Gm-Message-State: AOJu0YzQyB1yLw6VQMf7oWGLWB0jqWW3MYnAkVmrqGA2gbDlZLz06ooR nZuVealx8kw73HDsE4lxTK7vCB+sZKR2cOtHuqC3LKuvvMMn7Aq5HRqtkmtcAx8= X-Gm-Gg: ASbGncuvX3Icdkr/iVAtDF6IeOsY49zw75z5Tc2til/ODubwjzZstLfs6OpHzxMtdHO zTaujZPEq/JazP9pPXW4YSsbOTvgX7vo3C13H97qINIq/hx1T2AFLAHQQ4VAMFNWaJFUdnxVcAT FiliQgYj9oJmvzRSOAFWIcrPK8Iy/4+OeT27C5S97myrwnsaaLbK7FCzGVl9qR0N2/riKLR8eaF chHO22rR6gXhT9SYdPaGHosM+OAOipsIv0q7R9rlmqfnVvu65ia5e/BSTafOqFlz0MX//zKc+nI G22TT21QW/Jlc5+KfY2pYolzn8QHOIK0JRyf3vx9cDlpGjA= X-Google-Smtp-Source: AGHT+IHNrqbcuIiBRUvRVP9cSOVFbhWVRIATyUyD14thPoPyuS47pBXdBhGvzI0pv9085t2J+qaTJg== X-Received: by 2002:a05:6a00:10d0:b0:72d:9cbc:730d with SMTP id d2e1a72fcca58-72dafa44d90mr21110056b3a.11.1737398576093; Mon, 20 Jan 2025 10:42:56 -0800 (PST) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-72dab9c8d2bsm7699022b3a.92.2025.01.20.10.42.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Jan 2025 10:42:55 -0800 (PST) Date: Mon, 20 Jan 2025 10:42:54 -0800 From: Stephen Hemminger To: Bruce Richardson Cc: Subject: Re: [PATCH 00/15] Add attributes to allocation functions Message-ID: <20250120104254.74828cb1@hermes.local> In-Reply-To: References: <20250120180550.198121-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Mon, 20 Jan 2025 18:40:53 +0000 Bruce Richardson wrote: > On Mon, Jan 20, 2025 at 10:03:18AM -0800, Stephen Hemminger wrote: > > This patch series builds on the allocation function attributes > > added in 24.11 release. These annotations will allow for compiler > > to flag cases where a pointer is allocated with one function > > but incorrectly passed to a different free function. > > > > Checkpatch perl script falsely complains in a couple patches > > because it doesn't really understand C syntax for attributes. > > > > Looking at the patchset, I see a number of functions moved in files. Is > there a restriction on the placement of the alloc and free functions where > free has to be defined first? > > /Bruce Yes the function prototype must be defined before it can be used.