[PATCH] Xorg modalias dump

[Nathaniel McCallum]

Nathaniel McCallum wrote:
> Nathaniel McCallum wrote:
>> Adam Jackson wrote:
>>> On Sun, 2008-09-07 at 02:13 -0400, Nathaniel McCallum wrote:
>>>
>>>  
>>>> There are still some problems though (and I'd appreciate any advise 
>>>> on how to solve them):
>>>> 1. It crashes when the ati "umbrella" driver is installed, I'm not 
>>>> sure why...
>>>>     
>>>
>>> Adding this hunk makes it work for me:
>>>
>>> --- a/hw/xfree86/common/xf86Helper.c
>>> +++ b/hw/xfree86/common/xf86Helper.c
>>> @@ -1486,6 +1486,8 @@ xf86MatchDevice(const char *drivername, 
>>> GDevPtr **sectlist)
>>>      if (sectlist)
>>>         *sectlist = NULL;
>>>
>>> +    if (xf86DoModalias) return 0;
>>> +
>>>      if (xf86DoProbe) return 1;
>>>
>>>      if (xf86DoConfigure && xf86DoConfigurePass1) return 1;
>>>
>>>  
>>>> 2. It seems to want to initialize the display (or something).  
>>>> Could someone help me move this option to earlier in the Xorg 
>>>> start-up process so we don't have to mess with peoples' displays?
>>>>     
>>>
>>> Admittedly I tested this on a headless machine, so I'm not sure what
>>> you're talking about here.  Possibly the initial vt switch?
>>>   
>> This patch turns off the Xorg header output if -modalias is specified 
>> (as per our discussion on IRC).
>>
>> Nathaniel
> Can someone please review this?  I'd really like to see it merged.
This patch still needs a review (and is trivial).  Can someone please 
review?

Nathaniel