http://bugzilla.kernel.org/show_bug.cgi?id=8798


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|CODE_FIX                    |




------- Comment #11 from [EMAIL PROTECTED]  2007-08-02 08:18 -------
re: patch in comment #4 to error-out in generic procfs code
on registration of files with duplicate names.
Maybe this is correct, but as we are abandoning procfs
and as it doesn't really fix the issue at hand where we presumably
need two different files, so lets not apply this one.

re: patch in comment #5 to change the names in all of /proc/acpi/*/*
Again, probably technically correct, and this one actually should
fix the problem at hand, but as we want /proc/acpi/
to become legacy, lets not stir the pot by changing the strings
on everybody.  It will probably just upset somebody
who will have to make a userspace change.  Instead, lets save
that pain for their migration to sysfs.  Also, per comment #9,
this patch causes sbs to fail because it was counting on the
fixed size of these strings:-(


Re: comment #8 why AGP_
Chris, these strings are completely arbitrary, it could just
as well be called ABCD -- it is up to the BIOS developer to
choose (or in this case, probably inherit) whatever strings
they want.  Part of the migration from procfs to sysfs is to
stop exposing (and confusing) user-space with these internal
AML variable names -- they make for a really poor user/kernel API.

re: patch in comment #10 to change all /proc/acpi/*/*
Again, too disruptive for a legacy interface.

re: what to do
The problem at hand is two /proc/acpi/video/VID files
We should make the late-comer /proc/acpi/video/VID:1 or something
to make this "instance" of it unique.  It would be ideal if the
solution applies to all of /proc/acpi/*/* -- but I'd settle
for a modification to the video.c file that solves just
this instance of perhaps a more general problem -- since we've
not seen the problem elsewhere, and we want to leave procfs
behind with as few changes to it as possible.


-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to