Paths

Table of Contentst

Differential D58758

linuxkpi: Add `timer64_to_tm()`
ClosedPublic
Actions

Authored by dumbbell on Mon, Aug 10, 5:27 PM.

Tags

Referenced Files

	Unknown Object (File)
	Sun, Sep 6, 5:25 AM

	Unknown Object (File)
	Fri, Sep 4, 11:57 AM

	Unknown Object (File)
	Fri, Sep 4, 8:09 AM

	Unknown Object (File)
	Thu, Sep 3, 4:18 PM

	Unknown Object (File)
	Wed, Sep 2, 8:00 AM

	Unknown Object (File)
	Tue, Sep 1, 10:41 PM

	Unknown Object (File)
	Thu, Aug 27, 10:01 PM

	Unknown Object (File)
	Thu, Aug 27, 10:00 PM

View All 30 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rG5fad251f0563: linuxkpi: Add `timer64_to_tm()`

Summary

The function converts a number of seconds since Epoch to a struct tm.

The implementation was copied from contrib/tzcode/localtime.c.

The amdgpu DRM driver started to use it in Linux 6.15.

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dumbbell created this revision.Mon, Aug 10, 5:27 PM

Herald added subscribers: emaste, imp. · View Herald TranscriptMon, Aug 10, 5:27 PM

dumbbell requested review of this revision.Mon, Aug 10, 5:27 PM

dumbbell added a parent revision: D58757: linuxkpi: Add `hrtimer_setup()`.Mon, Aug 10, 5:34 PM

dumbbell added a child revision: D58759: linuxkpi: Add the `*_new` fields to `struct bin_attribute`.

emaste added a project: linuxkpi.Mon, Aug 17, 4:58 PM

bz added a subscriber: bz.Sat, Sep 5, 9:42 AM

bz added inline comments.

sys/compat/linuxkpi/common/src/linux_time.c
11 ↗	(On Diff #183793)	Which is public domain. What does that make this file then?

Fix new file license as suggested by @bz.

dumbbell marked an inline comment as done.Mon, Sep 7, 7:34 PM

dumbbell added inline comments.

sys/compat/linuxkpi/common/src/linux_time.c
11 ↗	(On Diff #183793)	I renamed the file to suggest that it only contains `timesub()` and I fixed the license header. Does this match what you had in mind?

I assume the copied file contents and logic are fine, so I haven't reviewed those.

This revision is now accepted and ready to land.Mon, Sep 7, 7:41 PM

Closed by commit rG5fad251f0563: linuxkpi: Add `timer64_to_tm()` (authored by dumbbell). · Explain WhyMon, Sep 7, 8:32 PM

This revision was automatically updated to reflect the committed changes.

dumbbell marked an inline comment as done.

dumbbell added a commit: rG5fad251f0563: linuxkpi: Add `timer64_to_tm()`.

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

19 lines

src/

linux_timesub.c

183 lines

conf/

2 lines

modules/

linuxkpi/

1 line

Diff 186175

sys/compat/linuxkpi/common/include/linux/time.h

Loading...

sys/compat/linuxkpi/common/src/linux_timesub.c

Loading...

sys/conf/files

Loading...

sys/modules/linuxkpi/Makefile

Loading...