Re: x86 BIOS interface for partitioning and system serial numberon SGI UV

[H. Peter Anvin]

Russ Anderson wrote:
> /proc/sgi_uv already exists, similar to /proc/sgi_sn on Itanium systems.

Both those to be really awful.

> Would it be /sys/class/firmware?  Reading Documentation/sysfs-rules.txt
> seems to indicate somewhere under /sys/devices, such as /sys/devices/system/,
> but I may be wrong.
>
> > > +# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7)		\
> > > +	do {								\
> > > +		/* XXX - the real call goes here */			\
> > > +		result.status = BIOS_STATUS_UNIMPLEMENTED;		\
> > > +		result.v0 = 0;						\
> > > +		result.v1 = 0;						\
> > > +	} while (0)
> > I have more than a little problem with submitting patches like this.  We 
> > have no way to judge the suitability of the coding or the interface with 
> > the "meat" of the driver stubbed out!
>
> This is also code for hardware that does not exist.  In order for the
> code to be in distro releases in time for the hardware to ship, we
> must push it _before_ we have hardware.  The main concern is getting in
> interfaces now, because the interfaces cannot change in a minor release.
> Providing more "meat" is on the ToDo list.  

You could still document the intended interface so we know how it is 
supposed to work.  At this point, you're basically saying "trust us", 
which really isn't how the Linux community is supposed to work.

	-hpa
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/