public inbox for linux-fsdevel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jon DeVree <nuxi@vault24.org>
To: Christian Brauner <brauner@kernel.org>
Cc: linux-fsdevel@vger.kernel.org
Subject: Re: [BUG] 2038 warning is not printed with new mount API
Date: Mon, 2 Dec 2024 09:46:21 -0500	[thread overview]
Message-ID: <Z03IPaweJtozuFvV@feynman.vault24.org> (raw)
In-Reply-To: <20241202-natur-davor-864eb423be9c@brauner>

On Mon, Dec 02, 2024 at 12:55:13 +0100, Christian Brauner wrote:
> IMHO, the best place to log a warning is either at fsmount() time or at
> superblock creation time but then the format of the warning will have to
> be slightly, changed. We could change it to:
> 
> [11526.621600] xfs filesystem supports timestamps until 2038 (0x7ffffff
> 
> libmount will log additional information about the mount to figure out
> what filesystem caused this warning to be logged.
>

Maybe it could log the device node instead of the mount point? Then the
messages from both APIs could remain consistent. One is probably as good
as the other and the other mount related log messages all seem to be
using the device node:

[   39.379071] EXT4-fs (sda3): mounted filesystem with ordered data mode. Quota mode: none.
[   39.574146] XFS (sda4): Mounting V5 Filesystem

I assume that info is available at this point because the filesystems
are able to use it in their log messages, but I could be wrong.

>
> Would that work for you?
>

I don't really have a preference for how its fixed. I reported it
because I happened to notice it go missing after a system upgrade.

I did my best to make a small suggestion despite a lack of expertise in
this area.

-- 
Jon
Doge Wrangler
X(7): A program for managing terminal windows. See also screen(1) and tmux(1).

  reply	other threads:[~2024-12-02 14:46 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-02  3:57 [BUG] 2038 warning is not printed with new mount API Jon DeVree
2024-12-02 11:55 ` Christian Brauner
2024-12-02 14:46   ` Jon DeVree [this message]
2024-12-03  5:12   ` Dave Chinner
2024-12-03  9:19     ` Christian Brauner
2024-12-03 11:06   ` Karel Zak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z03IPaweJtozuFvV@feynman.vault24.org \
    --to=nuxi@vault24.org \
    --cc=brauner@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox