[PATCH] Xorg modalias dump

[Nathaniel McCallum]

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.

Nathaniel