Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

upgrade libdrm-2.4.75 to libdrm-2.4.109 #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

klausz65
Copy link

And add initial SPARC support for drm module

@klausz65 klausz65 force-pushed the update-to-2.4.109 branch 3 times, most recently from 6c80f5e to d009fb7 Compare December 30, 2021 17:38
@randyfishel
Copy link

To the best of my knowledge, there is no gfx_private for Sparc, so the LDFLAGS on line 50 would at best cause a load failure (would be a pain to port given gfx_private's intensive use of x86-specific facilities). And I am unaware of any Sparc frame buffer driver that is still around that uses DRM (could possibly be one in the future, but wouldn't it be better to cross that bridge then?).

@klausz65 klausz65 force-pushed the update-to-2.4.109 branch 3 times, most recently from 8558cca to c089c8f Compare January 1, 2022 15:36
@gwr
Copy link
Member

gwr commented Jan 3, 2022

There's a lot here. Let me look at this later. Thanks.

Copy link
Member

@gwr gwr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For the most part, this looks good. Thanks for giving this some attention.
I added a few minor improvements and put them in another PR
#26
Feel free to incorporate those as you like.


2. Implement dynamic creation of the new header file:

- generated_static_table_fourcc.h
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I generally causes less problems for people if you go ahead and check in the generated file,
and provide a make rule to (optionally) verify that regenerating the file produces the same.
This is the same approach described in UPDATE_LIB.
Oh, looks like you did "git add" that file, right?
Is there a rule to verify if matches what gets generated?

@@ -179,7 +179,7 @@ ELFEDIT= /usr/bin/elfedit
ELFSIGN= /usr/bin/elfsign
DTRACE= /usr/sbin/dtrace -xnolibs
UNIQ= /usr/bin/uniq
TAR= /usr/bin/tar
TAR= /usr/bin/gtar
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was there a problem with plain old tar?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, we need to unpack an xz file now.

+ * Don't have DRM_CLONEID_NBITS here.
+ */
+ d &= ~0x3fe00;
+#endif
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a bit worried about the removal of these two changes.
When I latest tested this, the library would not correctly determine the type of device was open on some FD because the "clone open" bits cause the minor number to (usually) not match.

I forget which test demonstrates this. Do you need help tracking that down?
I think it was the "drmdevice" test (which we build under cmd/misc1)

+ err = _sun_drm_find_device(min, &path);
+ if (err == 0)
+ return (path);
+ /* else ... (ret NULL at endif) */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the 2nd part I referred to above.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added both those parts back for you in #26

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants