From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IwOVK-0003nm-4p for qemu-devel@nongnu.org; Sun, 25 Nov 2007 15:55:02 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IwOVH-0003la-Ug for qemu-devel@nongnu.org; Sun, 25 Nov 2007 15:55:01 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IwOVH-0003lX-OX for qemu-devel@nongnu.org; Sun, 25 Nov 2007 15:54:59 -0500 Received: from nf-out-0910.google.com ([64.233.182.185]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1IwOVH-0008HD-9u for qemu-devel@nongnu.org; Sun, 25 Nov 2007 15:54:59 -0500 Received: by nf-out-0910.google.com with SMTP id 30so497811nfu for ; Sun, 25 Nov 2007 12:54:58 -0800 (PST) Message-ID: Date: Sun, 25 Nov 2007 22:54:57 +0200 From: "Blue Swirl" Subject: Re: [Qemu-devel] [PATCH][UPDATE3] Add -drive parameter In-Reply-To: <11954606662783@bull.net> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <11954606662783@bull.net> Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org On 11/19/07, Laurent Vivier wrote: > > This is an update with Qemu CVS tree and comments from Blue Swirl. > > This patch introduces a new parameter allowing to define drives. > > The new parameter is "-drive": > > -drive [file=file][,if=type][,bus=n][,unit=m][,media=d][index=i][,cyls=c,heads=h,secs=s[,trans=t]][snapshot=on|off] > > where: > > file is the disk image > type is the interface type (ide, scsi, sd, mtd, floppy, pflash) > n is the bus number of the given type > m is the unit number on the given bus > d is the type of the media (disk, cdrom) > c,h,s,t are the parameters usually given by -hdachs > snapshot allows to enable or not the snapshot for this disk > i is an index number within the given type (replace bus,unit) > > "-cdrom file" is an alias for "-drive file=file,index=2,media=cdrom" > "-hda file" is an alias for "-drive file=file,index=0,media=disk" > "-hdb file" is an alias for "-drive file=file,index=1,media=disk" > "-hdc file" is an alias for "-drive file=file,index=2,media=disk" > "-hdd file" is an alias for "-drive file=file,index=3,media=disk" > "-hda file -hdachs a,b,c" is an alias for > "-drive file=file,index=0,cyls=a,heads=b,secs=c" > > You can also define a cdrom on the slace of ide0 with: > "-drive file=file,if=ide,index=1,media=cdrom" > > You can define an empty cdrom: > "-drive if=ide,index=1,media=cdrom" > > "-drive file=file,if=scsi,bus=0,unit=6" allows to connect the disk image file > to the scsi bus 0 with the unit id 6. > > if there is no SCSI disk, the SCSI interface is not created. > > It also defines the default interface type to "scsi" for targets > "realview", "SS-5", "SS-10", "versatilepb", "versatileab" > to keep old behavior, where "-hda" is a SCSI disk. > > "-fda file" is an alias for "-drive file=file,index=0,if=floppy" > "-fdb file" is an alias for "-drive file=file,index=1,if=floppy" > "-pflash file" is an alias for "-drive file=file,if=pflash" > "-mtdblock file" is an alias for "-drive file=file,if=mtd" > "-sd file" becomes the alias of "-drive file=file,if=sd" > > "-drive file=a -drive file=b" will be interpreted like "-hda a -hdb b" Any objections to committing this patch? To me it looks fine, but I have tested only Sparc.