All of lore.kernel.org
 help / color / mirror / Atom feed
From: Francois Tigeot <ftigeot@wolfpond.org>
To: randyf@sibernet.com
Cc: Emil Velikov <emil.l.velikov@gmail.com>,
	ML dri-devel <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH] drm.h: Handle DragonFly like Linux
Date: Tue, 17 May 2016 09:15:44 +0200	[thread overview]
Message-ID: <573AC520.7090209@wolfpond.org> (raw)
In-Reply-To: <alpine.SOC.2.00.1605162302020.14612@sibernet.com>

Hi Randy,

randyf@sibernet.com wrote:
>
>    If you are interested in the primary Solaris source, you will really
> want to looking at:
>
> https://java.net/projects/solaris-x11/sources/x-s12-clone/show/open-src/kernel

Thanks. This doesn't look like a git repository though.
How can I clone it ?

>> They are divergent by design :-/
>> Making the Linux headers public and removing the #else path in libdrm's
>> drm.h could be the right thing to do. I'll keep thinking about it...
>
>    Removing the #else path will cause Solaris compiles to fail as it
> does consume that side of the conditional (though, as we patch this file
> and don't use it as is, it would be trivial to add it or any other
> Solaris-specific requirements back in).

I was only thinking about patching our libdrm package here, not changing 
the upstream version.

>    From a Solaris perspective, I don't see how this file can ever -not-
> have conditional compile statements or be patched to support Solaris, as
> there are sufficient differences requiring at least minor changes.

I'd still like to only use the Linux version by default, both in kernel 
and userland.
I guess at this point the only way to do that without breaking Solaris 
is to either use local patches or change the logic of the #if directives 
to replace the Linux check with a !Solaris one.

>> I'm afraid there won't be many *BSD people around. This year, the XDC and
>> EuroBSDCon conferences fall on the same week-end :-(
>> I'm not sure which one I will be attending to at this point.
>
>    Not sure yet if I will be attending either.  But would happily
> provide any input on this either in person (if I am there), or via email.

Dri-devel seems to still be the best place for now :)

Cheers,

-- 
Francois Tigeot
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2016-05-17  7:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-14  7:13 [PATCH] drm.h: Handle DragonFly like Linux François Tigeot
2016-05-14 14:51 ` Emil Velikov
2016-05-16 16:24   ` Francois Tigeot
2016-05-16 22:02     ` Emil Velikov
2016-05-17  5:26       ` Francois Tigeot
2016-05-17  6:25         ` randyf
2016-05-17  7:15           ` Francois Tigeot [this message]
2016-05-17  7:48             ` randyf
2016-05-17 13:58         ` Ed Maste

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=573AC520.7090209@wolfpond.org \
    --to=ftigeot@wolfpond.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=emil.l.velikov@gmail.com \
    --cc=randyf@sibernet.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.