Messages in this thread | | | Date | Tue, 09 Apr 2002 17:22:05 +0200 | From | Martin Dalecki <> | Subject | Re: [PATCH][CFT] IDE TCQ #2 |
| |
Andries.Brouwer@cwi.nl wrote: > On Tue, Apr 09 2002, Martin Dalecki wrote: > > > echo "using_tcq:0" > /proc/ide/hdX/setting > > > > > > will disable TCQ and revert to DMA, > > > > > > echo "using_tcq:32" > /proc/ide/hdX/setting > > > > > > will set queue depth to 32, any value in between the two are of course > > > also allowed. The driver will print enable/disable info to the kernel > > > log. > > > > Well this belongs to an ioctl or sysctl... However most > > of the /proc/ide stuff if not all will go to the sysctl quite soon. > > Put it wherever you want it, I'm just making it easier for myself not > having to pass patches to hdparm around as well for people to enable > taggged queueing. > > Yes, for IDE purposes it does not matter much what one does. > > But one needs communication between user or user space > and kernel to interact with hundreds of drivers, in a rather > messy way. > > In my opinion sysctl() is worthless. It uses an array of numbers > where ioctl() uses a single number. Especially since the names are > already in the kernel it is much clearer and cleaner to use a > pathname. I wouldn't mind if sysctl() disappeared entirely.
Please have a look at /proc/sys/ OK?
> > Also ioctl() has its problems. First of all, nobody knows what the > prototype is. Secondly, it is too rigid - the moment one needs a > larger structure one needs a different ioctl. > > A text based interface is much more flexible. If the number of > cylinders of a disk no longer fits in a short, well doesn't matter, > then the number of digits may increase but the interface remains > unchanged. Of course the price is that one has to parse, but > typically that is not a problem.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |