From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from bregans-0.gladserv.net (bregans-0.gladserv.net [185.128.210.58]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 27DA42EB845; Tue, 2 Sep 2025 09:33:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.128.210.58 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756805622; cv=none; b=MF4mG78avpPn9G5XiYhQ7fq4UheOKPz3dEXIPO03GR9uFK4eoPcwx7jCnTJZERF4Q99R6p60Wy7fVnVeyR3i4WsHrmtmvRVkwljtCY7L1gB097EqhhTwgWJGMSDR3H9WI+xgdbyOTjcdlYHFhRI2wCP/0iWsHDdDgIF+ZaaJJBA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756805622; c=relaxed/simple; bh=bbIWtaRapTz4x6+Wf9RWlPmzpKW3MH8iyuTakHp1rQ8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=NAIdGwcgMVedABNyhFRD7maVUvly5WUF7KxLVhJ6C9yTgcxRUIFffP7WBHZ34IB3FzBhjdf070mbo//yNSl4I7VDkagOQzLb4sjrKIDVM3X1aQItbEXJeuPuBD54uOh5xwTdBnfe9EQpoqxkpYLnkQbv5PqkaAOIK1LDqFizLTM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=librecast.net; spf=pass smtp.mailfrom=librecast.net; arc=none smtp.client-ip=185.128.210.58 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=librecast.net Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=librecast.net Date: Tue, 2 Sep 2025 11:33:22 +0200 From: Brett Sheffield To: Paolo Abeni Cc: Oscar Maes , netdev@vger.kernel.org, bacs@librecast.net, kuba@kernel.org, davem@davemloft.net, dsahern@kernel.org, stable@vger.kernel.org Subject: Re: [PATCH net v4] selftests: net: add test for destination in broadcast packets Message-ID: References: <20250828114242.6433-1-oscmaes92@gmail.com> <03991134-4007-422b-b25a-003a85c1edb0@redhat.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <03991134-4007-422b-b25a-003a85c1edb0@redhat.com> On 2025-09-02 10:49, Paolo Abeni wrote: > On 8/28/25 1:42 PM, Oscar Maes wrote: > > Add test to check the broadcast ethernet destination field is set > > correctly. > > > > This test sends a broadcast ping, captures it using tcpdump and > > ensures that all bits of the 6 octet ethernet destination address > > are correctly set by examining the output capture file. > > > > Signed-off-by: Oscar Maes > > Co-authored-by: Brett A C Sheffield > > I'm sorry for nit-picking, but the sob/tag-chain is wrong, please have a > look at: > > https://elixir.bootlin.com/linux/v6.16.4/source/Documentation/process/submitting-patches.rst#L516 Thanks Paolo. So, something like: Co-developed-by: Brett A C Sheffield Signed-off-by: Brett A C Sheffield Co-developed-by: Oscar Maes Signed-off-by: Oscar Maes with the last sign-off by Oscar because he is submitting? Brett