Received: by 2002:ab2:6991:0:b0:1f7:f6c3:9cb1 with SMTP id v17csp332518lqo; Wed, 8 May 2024 00:39:24 -0700 (PDT) X-Forwarded-Encrypted: i=3; AJvYcCXRAB8PYOs66gouGftiJRbbrI8i7f/keCbWc7RQoyTSjH1AVSnwg+SC5DhQTfnaumZMevtI8TLxjSA3QsS51Lmfi+zgit70B2pqVYCHIw== X-Google-Smtp-Source: AGHT+IG4SHVROrtfZGxYg3631Tye1QKirRM8V6etzX+cClT/j5MyawF5x/fqFoDClac5LMM9izUP X-Received: by 2002:a05:6a21:4981:b0:1aa:6a28:cf6e with SMTP id adf61e73a8af0-1afc8dc6e23mr1801778637.48.1715153964724; Wed, 08 May 2024 00:39:24 -0700 (PDT) ARC-Seal: i=2; a=rsa-sha256; t=1715153964; cv=pass; d=google.com; s=arc-20160816; b=vCwrBAE85lFcVo6RIyEsJenrdLHAmyEs4tOz5eUFLVGHahlMge0kJk1fwTAYoBMStP Rse4Rtg+KDQEz6jBk4wDOwOOcYIsBeolqjeUNMvXZI54iqsEGaKECgpKm90o52eLTuGr guKzlXuB8LwHWuvIkWGowpxSeQICusF7P6AcKORMOHazH7Q2PdK+w/4Rvg0Lj+R+bMYu KVuFQ8ustt7LaivRNEGhgkZv633lGeno5jEzzvmIF2UwqL+VBspHh8yU3NrWW9rE7eHn 9dribwHiSiHquWf9VZQN05HCdCZMf6PRZrl+CQDbnIXc98IU0h8IkMHFet8w/2FZ8e0B P5JQ== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:list-unsubscribe:list-subscribe:list-id:precedence :message-id:date:references:in-reply-to:subject:cc:to:dkim-signature :dkim-signature:from; bh=H6hqXb9MCRTxs635sBkjBFmdNV/lsU4W+VAlI6NfWPc=; fh=XUWFeaneiYxOtN7LnSf5iGP+F2DtNZ3PzSE1jplUFww=; b=QkxCA17M02BsYSRgkNJv+3fwp5MYJGL6N3CLtE8qPAvglyrlB9m/0SUwVkbRtnD9HX BP7bPWDhJmzAv4+AtLDTPYsmRm9+5rTNsIIpapPfb5P/0bqqkMQGdMFaiI1muM6Xz0LI hLUHmosq6XvBFivV+GqgZWowN0JsuW6PAawwHDlziuxGR3Vlc8HJYcw0hZkDCOZ+trIA Fm7TMwEWUhT8HachFihZwBQtaZVVhHYjRB2IveYMNR+MeK0HcH5ZLhp0fcR6tIUmksmU gciRAN+2odMNuSCGgkZsg+X6nCV5eGi7HWhz7c6mgBrxDMf1zXwNT/aY6WxNYpAWvUGE GaNQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=1nuvvsDk; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e; arc=pass (i=1 spf=pass spfdomain=linutronix.de dkim=pass dkdomain=linutronix.de dmarc=pass fromdomain=linutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-172824-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-172824-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id cd23-20020a056a00421700b006f409781aa3si5145351pfb.230.2024.05.08.00.39.24 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 08 May 2024 00:39:24 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel+bounces-172824-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=1nuvvsDk; dkim=neutral (no key) header.i=@linutronix.de header.s=2020e; arc=pass (i=1 spf=pass spfdomain=linutronix.de dkim=pass dkdomain=linutronix.de dmarc=pass fromdomain=linutronix.de); spf=pass (google.com: domain of linux-kernel+bounces-172824-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-172824-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id B17CC281BBC for ; Wed, 8 May 2024 07:39:19 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id D880239FFD; Wed, 8 May 2024 07:39:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="1nuvvsDk"; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="VhcHvm6K" Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) (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 7E5A73A1DA; Wed, 8 May 2024 07:39:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=193.142.43.55 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715153942; cv=none; b=qSJ9hzS/inqPb9UlrM7genso6Th88gUw0TB5n5OWcdsOCEgJ3zZgKy44CqgyKhBfrtcXz6NDsBJgnC8BZjCgXPWOHTTma/rUFAKfS8dt7zPscr2+T0ur4yHXosPTU2HZrshXB8ljIZXiSYwRWxZ8Q5vYQDI79Vd3ofUaymDkq30= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715153942; c=relaxed/simple; bh=GRBM6Q9ulEa6b1C1AIlEe8Wdt4+S7xROw+0i92+JaGM=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=Vl2oIYOqyOkitW77ksIncoyWS3WdSqPuvnWaTgkGHmYgFKNyjqGKE8rB0z05nOpE2SOvzKqYi5uwUohh3wM5duRx8irqZOsI3mfzRYGi5zy0uFtGW//D7zEsFFUgK+WXC7dGsncrbZiKGy7fxj0JXMqu63aMEkDZhUEl+9Crtrs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de; spf=pass smtp.mailfrom=linutronix.de; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=1nuvvsDk; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=VhcHvm6K; arc=none smtp.client-ip=193.142.43.55 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linutronix.de From: Thomas Gleixner DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1715153938; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=H6hqXb9MCRTxs635sBkjBFmdNV/lsU4W+VAlI6NfWPc=; b=1nuvvsDkJk308I59qyuDDAY7raMXe58IJXHnvzks1l63fWs+dsBnkjDhXWyX+RkV5iwtA8 T3RA3EpWWIdkiKp1cax73+1KK2ox+7wLrbrvMIvRG2TELc+w2JbgQjKEZq+5ZfvRrJD/8F F8jpJogjcxUpCQMQFrJMErIyKbaTyAenawxMeeWOe14p9gCqzdgaEKrp44L/1IWM1lalyL yw6OMkBZatjNT8rJ8aNZ4It2sMoGGoLVipfSUaTMOZCrVC869f7TDfA9R9MnAokrpPpt/B OJYBR/wOvl0N9uAd1+YZM4U0seXYiQfQ+tYgjvnzs9psZGw2RDjWoVeZPgz0Xw== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1715153938; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=H6hqXb9MCRTxs635sBkjBFmdNV/lsU4W+VAlI6NfWPc=; b=VhcHvm6KLaVB6rBreMYUoEE64Vi598JqUzdOvrN74UVu4Cjh8nJs+xoFXBzFMTwf1LmnpH S/C3f/eYNLCT5VAA== To: Richard Cochran , Mahesh Bandewar Cc: Netdev , Linux , David Miller , Jakub Kicinski , Eric Dumazet , Paolo Abeni , Arnd Bergmann , Sagi Maimon , Jonathan Corbet , John Stultz , Mahesh Bandewar Subject: Re: [PATCHv4 net-next] ptp/ioctl: support MONOTONIC_RAW timestamps for PTP_SYS_OFFSET_EXTENDED In-Reply-To: References: <20240502211047.2240237-1-maheshb@google.com> Date: Wed, 08 May 2024 09:38:58 +0200 Message-ID: <87cypwpxbh.ffs@tglx> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On Tue, May 07 2024 at 21:44, Richard Cochran wrote: > On Thu, May 02, 2024 at 02:10:47PM -0700, Mahesh Bandewar wrote: >> +/* >> + * ptp_sys_offset_extended - data structure for IOCTL operation >> + * PTP_SYS_OFFSET_EXTENDED >> + * >> + * @n_samples: Desired number of measurements. >> + * @clockid: clockid of a clock-base used for pre/post timestamps. >> + * @rsv: Reserved for future use. >> + * @ts: Array of samples in the form [pre-TS, PHC, post-TS]. The >> + * kernel provides @n_samples. >> + * >> + * History: >> + * v1: Initial implementation. >> + * >> + * v2: Use the first word of the reserved-field for @clockid. That's >> + * backward compatible since v1 expects all three reserved words >> + * (@rsv[3]) to be 0 while the clockid (first word in v2) for >> + * CLOCK_REALTIME is '0'. > > This is not really appropriate for a source code comment. The > un-merged patch series iterations are preserved at lore.kernel in case > someone needs that. > > The "backward compatible" information really wants to be in the commit > message. I agree that it wants to be in the commit message, but having the version information in the kernel-doc which describes the UAPI is sensible and useful. That's where I'd look first and asking a user to dig up this information on lore is not really helpful. Thanks, tglx