On Fri, Dec 26, 2008 at 01:26, Karel Zak <kzak@xxxxxxxxxx> wrote:
> On Fri, Dec 26, 2008 at 12:39:16AM +0100, Kay Sievers wrote:
>> On Fri, Dec 26, 2008 at 00:26, Karel Zak <kzak@xxxxxxxxxx> wrote:
>> > On Fri, Dec 19, 2008 at 05:03:52PM +0100, Kay Sievers wrote:
>> >> > names
>> >> >
>> >> > you rename some devices and disagree with devices.txt
>> >> >
>> >> > rawctl -> raw/rawctl
>> >>
>> >> The tools use that, if I remember. Someone maintaining the tools should
>> >> tell, devices.txt is pretty unreliable sometimes.
>> >
>> > The upstream raw(8) command supports /dev/rawctl and also
>> > /dev/raw/rawctl. I think it makes more sense to use raw/rawctl when
>> > you have all your raw devices in raw/ subdirectory (e.g. /dev/raw/raw<N>).
>>
>> The raw tool looks for /dev/rawctl first and the fallback to
>> /dev/raw/rawctl is named DEVFS_*. Should we turn that order around and
>> remove the devfs notion from the raw tool and let udev create a
>> dev/raw/rawctl node?
>
> Yeah. Fixed, committed and pushed.
>
> $ strace -e open ./raw
> open("/dev/raw/rawctl", O_RDWR) = -1 ENOENT (No such file or directory)
> open("/dev/rawctl", O_RDWR) = -1 ENOENT (No such file or directory)
>
> I have also removed the #ifdef OLD_RAW_DEVS (/dev/raw<N>) junk.
Pushed out /dev/raw/rawctl.
> BTW, it would be nice to refresh devices.txt according to your latest
> changes in udev.
Yeah, we should do that, also the /dev/scd* stuff should be removed there.
Thanks,
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
