Paths

Table of Contentst

Differential D36961

linuxkpi: Add <linux/page-flags.h>
ClosedPublic
Actions

Authored by dumbbell on Oct 12 2022, 7:18 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Sep 21, 4:08 PM

	Unknown Object (File)
	Wed, Sep 18, 6:05 AM

	Unknown Object (File)
	Sat, Sep 7, 1:37 AM

	Unknown Object (File)
	Thu, Sep 5, 9:16 AM

	Unknown Object (File)
	Mon, Sep 2, 1:38 PM

	Unknown Object (File)
	Sun, Sep 1, 12:29 PM

	Unknown Object (File)
	Sun, Sep 1, 7:49 AM

	Unknown Object (File)
	Aug 20 2024, 4:26 PM

View All 26 Files

Subscribers

Details

Reviewers

Commits

rGcec94450c299: linuxkpi: Add <linux/page-flags.h>
rG708a570255a4: linuxkpi: Add <linux/page-flags.h>

Summary

It just provides a PageHighMem() macro stub.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dumbbell created this revision.Oct 12 2022, 7:18 PM

Herald added subscribers: linuxkpi, emaste, imp. · View Herald TranscriptOct 12 2022, 7:18 PM

dumbbell requested review of this revision.Oct 12 2022, 7:18 PM

Harbormaster completed remote builds in B47809: Diff 111731.Oct 12 2022, 7:18 PM

dumbbell mentioned this in D36869: This patch contains [[ https://github.com/dumbbell/freebsd-src/tree/linuxkpi-changes-for-drm-from-5.11 | a series of commits to linuxkpi ]] in order to support an update of the DRM drivers to match Linux 5.11:.Oct 12 2022, 7:32 PM

manu added inline comments.Oct 14 2022, 8:38 AM

sys/compat/linuxkpi/common/include/linux/page-flags.h
8	Are those copyrights correct ?

Remove copy-pasted copyright statements.

dumbbell marked an inline comment as done.Nov 5 2022, 2:02 PM

manu added inline comments.Nov 5 2022, 5:18 PM

sys/compat/linuxkpi/common/include/linux/page-flags.h
2	I think that this SPDX isn't correct even if it's in the template, @imp ? Also you might want to add yourself as the copyright holder or put the file in public domain, I don't think that BSD-2-Clause without ownership make sense.

manu mentioned this in D36970: linuxkpi: Add <linux/mman.h> which includes <linux/smp.h>.Nov 5 2022, 5:19 PM

emaste added inline comments.Nov 5 2022, 8:45 PM

sys/compat/linuxkpi/common/include/linux/page-flags.h
2	Right, just BSD-2-Clause

imp added inline comments.Nov 5 2022, 9:23 PM

sys/compat/linuxkpi/common/include/linux/page-flags.h
2	I just updated the licensing files. BSD-2-Clause-FreeBSD was approved and then deprecated by SPDX for reasons too long to go into here. The project's current use of it is largely in error, but the resources to correct this error haven't been found. This applies to all the different places manu@ has flagged. You also have the option to omit the actual license text now.

Fix SPDX identifier
Add copyright line

dumbbell marked 3 inline comments as done.Nov 11 2022, 6:10 PM

manu accepted this revision.Nov 11 2022, 7:48 PM

This revision is now accepted and ready to land.Nov 11 2022, 7:48 PM

Closed by commit rG708a570255a4: linuxkpi: Add <linux/page-flags.h> (authored by dumbbell). · Explain WhyNov 11 2022, 8:05 PM

This revision was automatically updated to reflect the committed changes.

dumbbell added a commit: rG708a570255a4: linuxkpi: Add <linux/page-flags.h>.

manu added a commit: rGcec94450c299: linuxkpi: Add <linux/page-flags.h>.Jan 24 2023, 8:56 AM

Revision Contents
Changeset List

Path

Size

sys/

compat/

linuxkpi/

common/

include/

linux/

32 lines

Diff 112655

sys/compat/linuxkpi/common/include/linux/page-flags.h

Loading...