)]}' { "log": [ { "commit": "7ab358c23cbf15cea08129cd722d1ce77433a94d", "tree": "713d38823efda30222230314bcae26b919782f16", "parents": [ "4423fe40b03f32b11e72ecfa03077e702e55d5a9" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Fri May 20 02:11:14 2011 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon May 30 11:14:16 2011 +0930" }, "message": "virtio: add api for delayed callbacks\n\nAdd an API that tells the other side that callbacks\nshould be delayed until a lot of work has been done.\nImplement using the new event_idx feature.\n\nNote: it might seem advantageous to let the drivers\nask for a callback after a specific capacity has\nbeen reached. However, as a single head can\nfree many entries in the descriptor table,\nwe don\u0027t really have a clue about capacity\nuntil get_buf is called. The API is the simplest\nto implement at the moment, we\u0027ll see what kind of\nhints drivers can pass when there\u0027s more than one\nuser of the feature.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "a5c262c5fd83ece01bd649fb08416c501d4c59d7", "tree": "295ddc61d59236b6521f5587e2e78ccb8da5d97a", "parents": [ "bf7035bf20563a6cadcb9e870406e7b21daf5e30" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Fri May 20 02:10:44 2011 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon May 30 11:14:15 2011 +0930" }, "message": "virtio_ring: support event idx feature\n\nSupport for the new event idx feature:\n1. When enabling interrupts, publish the current avail index\n value to the host to get interrupts on the next update.\n2. Use the new avail_event feature to reduce the number\n of exits from the guest.\n\nSimple test with the simulator:\n\n[virtio]# time ./virtio_test\nspurious wakeus: 0x7\n\nreal 0m0.169s\nuser 0m0.140s\nsys 0m0.019s\n[virtio]# time ./virtio_test --no-event-idx\nspurious wakeus: 0x11\n\nreal 0m0.649s\nuser 0m0.295s\nsys 0m0.335s\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "bf50e69f63d21091e525185c3ae761412be0ba72", "tree": "fc88d1e774ae03e680b2fddc265404ee628b8545", "parents": [ "177dbd95637a52b9118aca757d5856ec3995d3e7" ], "author": { "name": "Dave Hansen", "email": "dave@linux.vnet.ibm.com", "time": "Thu Apr 07 10:43:25 2011 -0700" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon May 30 11:14:13 2011 +0930" }, "message": "virtio balloon: kill tell-host-first logic\n\nThe virtio balloon driver has a VIRTIO_BALLOON_F_MUST_TELL_HOST\nfeature bit. Whenever the bit is set, the guest kernel must\nalways tell the host before we free pages back to the allocator.\nWithout this feature, we might free a page (and have another\nuser touch it) while the hypervisor is unprepared for it.\n\nBut, if the bit is _not_ set, we are under no obligation to\nreverse the order; we\u0027re under no obligation to do _anything_.\nAs of now, qemu-kvm defines the bit, but doesn\u0027t set it.\n\nThis patch makes the \"tell host first\" logic the only case. This\nshould make everybody happy, and reduce the amount of untested or\nuntestable code in the kernel.\n\nThis _also_ means that we don\u0027t have to preserve a pfn list\nafter the pages are freed, which should let us get rid of some\ntemporary storage (vb-\u003epfns) eventually.\n\nSigned-off-by: Dave Hansen \u003cdave@linux.vnet.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "31a3ddda166cda86d2b5111e09ba4bda5239fae6", "tree": "9cd6af0a5a7b59b4452db2fa84545ce782c4006b", "parents": [ "b3258ff1d6086bd2b9eeb556844a868ad7d49bc8" ], "author": { "name": "Amit Shah", "email": "amit.shah@redhat.com", "time": "Mon Mar 14 17:45:02 2011 +0530" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Apr 21 22:57:00 2011 +0930" }, "message": "virtio_pci: Prevent double-free of pci regions after device hot-unplug\n\nIn the case where a virtio-console port is in use (opened by a program)\nand a virtio-console device is removed, the port is kept around but all\nthe virtio-related state is assumed to be gone.\n\nWhen the port is finally released (close() called), we call\ndevice_destroy() on the port\u0027s device. This results in the parent\ndevice\u0027s structures to be freed as well. This includes the PCI regions\nfor the virtio-console PCI device.\n\nOnce this is done, however, virtio_pci_release_dev() kicks in, as the\nlast ref to the virtio device is now gone, and attempts to do\n\n pci_iounmap(pci_dev, vp_dev-\u003eioaddr);\n pci_release_regions(pci_dev);\n pci_disable_device(pci_dev);\n\nwhich results in a double-free warning.\n\nMove the code that releases regions, etc., to the virtio_pci_remove()\nfunction, and all that\u0027s now left in release_dev is the final freeing of\nthe vp_dev.\n\nSigned-off-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "b3258ff1d6086bd2b9eeb556844a868ad7d49bc8", "tree": "2c9dde005804227038c997a983ded7e46951ca64", "parents": [ "584f79046780e10cb24367a691f8c28398a00e84" ], "author": { "name": "Amit Shah", "email": "amit.shah@redhat.com", "time": "Wed Mar 16 19:12:10 2011 +0530" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Apr 21 22:57:00 2011 +0930" }, "message": "virtio: Decrement avail idx on buffer detach\n\nWhen detaching a buffer from a vq, the avail.idx value should be\ndecremented as well.\n\nThis was noticed by hot-unplugging a virtio console port and then\nplugging in a new one on the same number (re-using the vqs which were\njust \u0027disowned\u0027). qemu reported\n\n \u0027Guest moved used index from 0 to 256\u0027\n\nwhen any IO was attempted on the new port.\n\nCC: stable@kernel.org\nReported-by: juzhang \u003cjuzhang@redhat.com\u003e\nSigned-off-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "8b3bb3ecf1934ac4a7005ad9017de1127e2fbd2f", "tree": "62ccde0eb73a77801b329391aff75be4c96da8eb", "parents": [ "2b8216e6354e7666a2718d4b891c8e8d7fcded27" ], "author": { "name": "Milton Miller", "email": "miltonm@bga.com", "time": "Fri Jan 07 02:55:06 2011 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Jan 20 21:37:30 2011 +1030" }, "message": "virtio: remove virtio-pci root device\n\nWe sometimes need to map between the virtio device and\nthe given pci device. One such use is OS installer that\ngets the boot pci device from BIOS and needs to\nfind the relevant block device. Since it can\u0027t,\ninstallation fails.\n\nInstead of creating a top-level devices/virtio-pci\ndirectory, create each device under the corresponding\npci device node. Symlinks to all virtio-pci\ndevices can be found under the pci driver link in\nbus/pci/drivers/virtio-pci/devices, and all virtio\ndevices under drivers/bus/virtio/devices.\n\nSigned-off-by: Milton Miller \u003cmiltonm@bga.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nAcked-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nTested-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nAcked-by: Gleb Natapov \u003cgleb@redhat.com\u003e\nTested-by: \"Daniel P. Berrange\" \u003cberrange@redhat.com\u003e\nCc: stable@kernel.org\n" }, { "commit": "be6528b2e58d92ef2ffed6f130e2d42bb85dbf29", "tree": "b16de122d65f8f2891748e76f94bc117dfaaa396", "parents": [ "22e132ff2645aab1e1a25c45e9544a39ae1dc106" ], "author": { "name": "Stephen Hemminger", "email": "shemminger@vyatta.com", "time": "Tue Nov 09 22:20:29 2010 -0800" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Nov 24 15:21:12 2010 +1030" }, "message": "virtio: fix format of sysfs driver/vendor files\n\nThe sysfs files for virtio produce the wrong format and are missing\nthe required newline. The output for virtio bus vendor/device should\nhave the same format as the corresponding entries for PCI devices.\n\nAlthough this technically changes the ABI for sysfs, these files were\nbroken to start with!\n\nSigned-off-by: Stephen Hemminger \u003cshemminger@vyatta.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "7ae4b866f86f9ab7b99484b56dd303a860ad1cc9", "tree": "247cd281bdf2e892180ff3407377d0a212c30016", "parents": [ "dfd62d1d84d83f9421792c78bcf72de9bc2bb603" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Nov 04 14:24:24 2010 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Nov 24 15:21:11 2010 +1030" }, "message": "virtio: return correct capacity to users\n\nWe can\u0027t rely on indirect buffers for capacity\ncalculations because they need a memory allocation\nwhich might fail. In particular, virtio_net can get\ninto this situation under stress, and it drops packets\nand performs badly.\n\nSo return the number of buffers we can guarantee users.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nReported-By: Krishna Kumar2 \u003ckrkumar2@in.ibm.com\u003e\n" }, { "commit": "1fe9b6fef11771461e69ecd1bc8935a1c7c90cb5", "tree": "3ecc82deed99dc11c67d64bfea2737fc68d769d3", "parents": [ "2e65a2075cc740b485ab203430bdf3459d5551b6" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Mon Jul 26 16:55:30 2010 +0930" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 26 08:05:31 2010 -0700" }, "message": "virtio: fix oops on OOM\n\nvirtio ring was changed to return an error code on OOM,\nbut one caller was missed and still checks for vq-\u003evring.num.\nThe fix is just to check for \u003c0 error code.\n\nLong term it might make sense to change goto add_head to\njust return an error on oom instead, but let\u0027s apply\na minimal fix for 2.6.35.\n\nReported-by: Chris Mason \u003cchris.mason@oracle.com\u003e\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nTested-by: Chris Mason \u003cchris.mason@oracle.com\u003e\nCc: stable@kernel.org # .34.x\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b03214d559471359e2a85ae256686381d0672f29", "tree": "b2c5d617304bffc8dd4917ecef251ec109eb3b8b", "parents": [ "686d363786a53ed28ee875b84ef24e6d5126ef6f" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Wed Jun 23 22:49:06 2010 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Jun 23 22:49:07 2010 +0930" }, "message": "virtio-pci: disable msi at startup\n\nvirtio-pci resets the device at startup by writing to the status\nregister, but this does not clear the pci config space,\nspecifically msi enable status which affects register\nlayout.\n\nThis breaks things like kdump when they try to use e.g. virtio-blk.\n\nFix by forcing msi off at startup. Since pci.c already has\na routine to do this, we export and use it instead of duplicating code.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nTested-by: Vivek Goyal \u003cvgoyal@redhat.com\u003e\nAcked-by: Jesse Barnes \u003cjbarnes@virtuousgeek.org\u003e\nCc: linux-pci@vger.kernel.org\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: stable@kernel.org\n" }, { "commit": "686d363786a53ed28ee875b84ef24e6d5126ef6f", "tree": "c97b5c5961df1f051e42a59dcb9ee1884d8f3086", "parents": [ "7e27d6e778cd87b6f2415515d7127eba53fe5d02" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Jun 10 18:16:11 2010 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Jun 23 22:49:06 2010 +0930" }, "message": "virtio: return ENOMEM on out of memory\n\nadd_buf returns ring size on out of memory,\nthis is not what devices expect.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nAcked-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: stable@kernel.org # .34.x\n" }, { "commit": "1756ac3d3c41341297ea25b818b7fce505bb2a9a", "tree": "96382220afbb82fd5c576c4c08b3c3e13282851f", "parents": [ "98edb6ca4174f17a64890a02f44c211c8b44fb3c", "0643e4c6e4fd67778fa886a89e6ec2320e0ff4d3" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 21 17:22:52 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 21 17:22:52 2010 -0700" }, "message": "Merge branch \u0027virtio\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux-2.6-for-linus\n\n* \u0027virtio\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux-2.6-for-linus: (27 commits)\n drivers/char: Eliminate use after free\n virtio: console: Accept console size along with resize control message\n virtio: console: Store each console\u0027s size in the console structure\n virtio: console: Resize console port 0 on config intr only if multiport is off\n virtio: console: Add support for nonblocking write()s\n virtio: console: Rename wait_is_over() to will_read_block()\n virtio: console: Don\u0027t always create a port 0 if using multiport\n virtio: console: Use a control message to add ports\n virtio: console: Move code around for future patches\n virtio: console: Remove config work handler\n virtio: console: Don\u0027t call hvc_remove() on unplugging console ports\n virtio: console: Return -EPIPE to hvc_console if we lost the connection\n virtio: console: Let host know of port or device add failures\n virtio: console: Add a __send_control_msg() that can send messages without a valid port\n virtio: Revert \"virtio: disable multiport console support.\"\n virtio: add_buf_gfp\n trans_virtio: use virtqueue_xxx wrappers\n virtio-rng: use virtqueue_xxx wrappers\n virtio_ring: remove a level of indirection\n virtio_net: use virtqueue_xxx wrappers\n ...\n\nFix up conflicts in drivers/net/virtio_net.c due to new virtqueue_xxx\nwrappers changes conflicting with some other cleanups.\n" }, { "commit": "bbd603efb4238cf78083c00f0a81adfa8994aa33", "tree": "3980961f9dbf08e748b3c2de56caf43dec21aa76", "parents": [ "dc3f5e68f846eec38fb31d78f0b6e83633ad375e" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Apr 29 17:26:37 2010 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed May 19 22:15:46 2010 +0930" }, "message": "virtio: add_buf_gfp\n\nAdd an add_buf variant that gets gfp parameter. Use that\nto allocate indirect buffers.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "7c5e9ed0c84e7d70d887878574590638d5572659", "tree": "c929c367c6854f021b787fa99fc56d37f64d9bc0", "parents": [ "1915a712f210f0b63d10bc4f875e8e66aac7a2c4" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Mon Apr 12 16:19:07 2010 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed May 19 22:15:43 2010 +0930" }, "message": "virtio_ring: remove a level of indirection\n\nWe have a single virtqueue_ops implementation,\nand it seems unlikely we\u0027ll get another one\nat this point. So let\u0027s remove an unnecessary\nlevel of indirection: it would be very easy to\nre-add it if another implementation surfaces.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "946cfe0e05664543b22ed674fff3764f41a372c7", "tree": "ecab9338ebd5eccf3befbf1325b3705cbf3bbc75", "parents": [ "316f25f5b3d547ce892d370f99f08a27276896ff" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Mon Apr 12 16:18:28 2010 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed May 19 22:15:41 2010 +0930" }, "message": "virtio_balloon: use virtqueue_xxx wrappers\n\nSwitch virtio_balloon to new virtqueue_xxx wrappers.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "6c9468e9eb1252eaefd94ce7f06e1be9b0b641b1", "tree": "797676a336b050bfa1ef879377c07e541b9075d6", "parents": [ "4cb3ca7cd7e2cae8d1daf5345ec99a1e8502cf3f", "c81eddb0e3728661d1585fbc564449c94165cc36" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Apr 23 02:08:44 2010 +0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Apr 23 02:08:44 2010 +0200" }, "message": "Merge branch \u0027master\u0027 into for-next\n" }, { "commit": "61fb06cc8e68906fb29d67bd5b6c2d50a95fc9e5", "tree": "0019a4794e8060f9c71a4cf6d6a712e204f3e5a7", "parents": [ "b7b6cf005b59ff5a4dba10338954c3dcfc75ef7b" ], "author": { "name": "Balbir Singh", "email": "balbir@linux.vnet.ibm.com", "time": "Thu Apr 22 12:22:34 2010 +0930" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Apr 22 07:34:05 2010 -0700" }, "message": "virtio: Fix GFP flags passed from the virtio balloon driver\n\nThe virtio balloon driver can dig into the reservation pools of the OS\nto satisfy a balloon request. This is not advisable and other balloon\ndrivers (drivers/xen/balloon.c) avoid this as well.\n\nThe patch also adds changes to avoid printing a warning if allocation\nfails, since we retry after sometime anyway.\n\nSigned-off-by: Balbir Singh \u003cbalbir@linux.vnet.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: kvm \u003ckvm@vger.kernel.org\u003e\nCc: stable@kernel.org\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5a0e3ad6af8660be21ca98a971cd00f331318c05", "tree": "5bfb7be11a03176a87296a43ac6647975c00a1d1", "parents": [ "ed391f4ebf8f701d3566423ce8f17e614cde9806" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 24 17:04:11 2010 +0900" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Mar 30 22:02:32 2010 +0900" }, "message": "include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit slab.h inclusion from percpu.h\n\npercpu.h is included by sched.h and module.h and thus ends up being\nincluded when building most .c files. percpu.h includes slab.h which\nin turn includes gfp.h making everything defined by the two files\nuniversally available and complicating inclusion dependencies.\n\npercpu.h -\u003e slab.h dependency is about to be removed. Prepare for\nthis change by updating users of gfp and slab facilities include those\nheaders directly instead of assuming availability. As this conversion\nneeds to touch large number of source files, the following script is\nused as the basis of conversion.\n\n http://userweb.kernel.org/~tj/misc/slabh-sweep.py\n\nThe script does the followings.\n\n* Scan files for gfp and slab usages and update includes such that\n only the necessary includes are there. ie. if only gfp is used,\n gfp.h, if slab is used, slab.h.\n\n* When the script inserts a new include, it looks at the include\n blocks and try to put the new include such that its order conforms\n to its surrounding. It\u0027s put in the include block which contains\n core kernel includes, in the same order that the rest are ordered -\n alphabetical, Christmas tree, rev-Xmas-tree or at the end if there\n doesn\u0027t seem to be any matching order.\n\n* If the script can\u0027t find a place to put a new include (mostly\n because the file doesn\u0027t have fitting include block), it prints out\n an error message indicating which .h file needs to be added to the\n file.\n\nThe conversion was done in the following steps.\n\n1. The initial automatic conversion of all .c files updated slightly\n over 4000 files, deleting around 700 includes and adding ~480 gfp.h\n and ~3000 slab.h inclusions. The script emitted errors for ~400\n files.\n\n2. Each error was manually checked. Some didn\u0027t need the inclusion,\n some needed manual addition while adding it to implementation .h or\n embedding .c file was more appropriate for others. This step added\n inclusions to around 150 files.\n\n3. The script was run again and the output was compared to the edits\n from #2 to make sure no file was left behind.\n\n4. Several build tests were done and a couple of problems were fixed.\n e.g. lib/decompress_*.c used malloc/free() wrappers around slab\n APIs requiring slab.h to be added manually.\n\n5. The script was run on all .h files but without automatically\n editing them as sprinkling gfp.h and slab.h inclusions around .h\n files could easily lead to inclusion dependency hell. Most gfp.h\n inclusion directives were ignored as stuff from gfp.h was usually\n wildly available and often used in preprocessor macros. Each\n slab.h inclusion directive was examined and added manually as\n necessary.\n\n6. percpu.h was updated not to include slab.h.\n\n7. Build test were done on the following configurations and failures\n were fixed. CONFIG_GCOV_KERNEL was turned off for all tests (as my\n distributed build env didn\u0027t work with gcov compiles) and a few\n more options had to be turned off depending on archs to make things\n build (like ipr on powerpc/64 which failed due to missing writeq).\n\n * x86 and x86_64 UP and SMP allmodconfig and a custom test config.\n * powerpc and powerpc64 SMP allmodconfig\n * sparc and sparc64 SMP allmodconfig\n * ia64 SMP allmodconfig\n * s390 SMP allmodconfig\n * alpha SMP allmodconfig\n * um on x86_64 SMP allmodconfig\n\n8. percpu.h modifications were reverted so that it could be applied as\n a separate patch and serve as bisection point.\n\nGiven the fact that I had only a couple of failures from tests on step\n6, I\u0027m fairly confident about the coverage of this conversion patch.\nIf there is a breakage, it\u0027s likely to be something in one of the arch\nheaders which should be easily discoverable easily on most builds of\nthe specific arch.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nGuess-its-ok-by: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nCc: Lee Schermerhorn \u003cLee.Schermerhorn@hp.com\u003e\n" }, { "commit": "88393161210493e317ae391696ee8ef463cb3c23", "tree": "6ec81a50d0e8174b415d83948b48cbabd7e54ddb", "parents": [ "932fb06b0898f5883200f1da2e00075f0d70ba9c" ], "author": { "name": "Thomas Weber", "email": "swirl@gmx.li", "time": "Tue Mar 16 11:47:56 2010 +0100" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Tue Mar 16 11:47:56 2010 +0100" }, "message": "Fix typos in comments\n\n[Ss]ytem \u003d\u003e [Ss]ystem\nudpate \u003d\u003e update\nparamters \u003d\u003e parameters\norginal \u003d\u003e original\n\nSigned-off-by: Thomas Weber \u003cswirl@gmx.li\u003e\nAcked-by: Randy Dunlap \u003crdunlap@xenotime.net\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "bc505f373979692d51a86d40925f77a8b09d17b9", "tree": "9a40aee54a4010af3eb0209db49f1193d4e45511", "parents": [ "3119815912a220bdac943dfbdfee640414c0c611" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Sun Nov 29 17:52:00 2009 +0200" }, "committer": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Tue Mar 02 13:41:14 2010 +0200" }, "message": "virtio: set pci bus master enable bit\n\nAs all virtio devices perform DMA, we\nmust enable bus mastering for them to be\nspec compliant.\n\nThis patch fixes hotplug of virtio devices\nwith Linux guests and qemu 0.11-0.12.\n\nTested-by: Alexander Graf \u003cagraf@suse.de\u003e\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\n" }, { "commit": "3119815912a220bdac943dfbdfee640414c0c611", "tree": "e8a6923ba0da89d7bb3cc9fd698f8c32a4a2f8ae", "parents": [ "847f9c606cad121cebf984639e3eeee1c4db82f8" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Feb 25 19:08:55 2010 +0200" }, "committer": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Sun Feb 28 20:39:11 2010 +0200" }, "message": "virtio: fix out of range array access\n\nI have observed the following error on virtio-net module unload:\n\n------------[ cut here ]------------\nWARNING: at kernel/irq/manage.c:858 __free_irq+0xa0/0x14c()\nHardware name: Bochs\nTrying to free already-free IRQ 0\nModules linked in: virtio_net(-) virtio_blk virtio_pci virtio_ring\nvirtio af_packet e1000 shpchp aacraid uhci_hcd ohci_hcd ehci_hcd [last\nunloaded: scsi_wait_scan]\nPid: 1957, comm: rmmod Not tainted 2.6.33-rc8-vhost #24\nCall Trace:\n [\u003cffffffff8103e195\u003e] warn_slowpath_common+0x7c/0x94\n [\u003cffffffff8103e204\u003e] warn_slowpath_fmt+0x41/0x43\n [\u003cffffffff810a7a36\u003e] ? __free_pages+0x5a/0x70\n [\u003cffffffff8107cc00\u003e] __free_irq+0xa0/0x14c\n [\u003cffffffff8107cceb\u003e] free_irq+0x3f/0x65\n [\u003cffffffffa0081424\u003e] vp_del_vqs+0x81/0xb1 [virtio_pci]\n [\u003cffffffffa0091d29\u003e] virtnet_remove+0xda/0x10b [virtio_net]\n [\u003cffffffffa0075200\u003e] virtio_dev_remove+0x22/0x4a [virtio]\n [\u003cffffffff812709ee\u003e] __device_release_driver+0x66/0xac\n [\u003cffffffff81270ab7\u003e] driver_detach+0x83/0xa9\n [\u003cffffffff8126fc66\u003e] bus_remove_driver+0x91/0xb4\n [\u003cffffffff81270fcf\u003e] driver_unregister+0x6c/0x74\n [\u003cffffffffa0075418\u003e] unregister_virtio_driver+0xe/0x10 [virtio]\n [\u003cffffffffa0091c4d\u003e] fini+0x15/0x17 [virtio_net]\n [\u003cffffffff8106997b\u003e] sys_delete_module+0x1c3/0x230\n [\u003cffffffff81007465\u003e] ? old_ich_force_enable_hpet+0x117/0x164\n [\u003cffffffff813bb720\u003e] ? do_page_fault+0x29c/0x2cc\n [\u003cffffffff81028e58\u003e] sysenter_dispatch+0x7/0x27\n---[ end trace 15e88e4c576cc62b ]---\n\nThe bug is in virtio-pci: we use msix_vector as array index to get irq\nentry, but some vqs do not have a dedicated vector so this causes an out\nof bounds access. By chance, we seem to often get 0 value, which\nresults in this error.\n\nFix by verifying that vector is legal before using it as index.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nAcked-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nAcked-by: Shirley Ma \u003cxma@us.ibm.com\u003e\nAcked-by: Amit Shah \u003camit.shah@redhat.com\u003e\n" }, { "commit": "3b8706240ee6084ccb46e53cd3a554356b7eeec8", "tree": "213051dfce6ddf5eb09d89ead56214372d200338", "parents": [ "c021eac4148c16bf53baa0dd14e8ebee6f39dab5" ], "author": { "name": "Amit Shah", "email": "amit.shah@redhat.com", "time": "Fri Feb 12 10:32:14 2010 +0530" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:29 2010 +1030" }, "message": "virtio: Initialize vq-\u003edata entries to NULL\n\nvq operations depend on vq-\u003edata[i] being NULL to figure out if the vq\nentry is in use (since the previous patch).\n\nWe have to initialize them to NULL to ensure we don\u0027t work with junk\ndata and trigger false BUG_ONs.\n\nSigned-off-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Shirley Ma \u003cxma@us.ibm.com\u003e\n" }, { "commit": "c021eac4148c16bf53baa0dd14e8ebee6f39dab5", "tree": "558db2ede74488606614d6e7e277f9914e38fbc1", "parents": [ "69740c8ba878f58bc3c71f74618fc2cd1da990da" ], "author": { "name": "Shirley Ma", "email": "mashirle@us.ibm.com", "time": "Mon Jan 18 19:15:23 2010 +0530" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:27 2010 +1030" }, "message": "virtio: Add ability to detach unused buffers from vrings\n\nThere\u0027s currently no way for a virtio driver to ask for unused\nbuffers, so it has to keep a list itself to reclaim them at shutdown.\nThis is redundant, since virtio_ring stores that information. So\nadd a new hook to do this.\n\nSigned-off-by: Shirley Ma \u003cxma@us.ibm.com\u003e\nSigned-off-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "d57ed95da483418e8b0433da693c9168dd0a2df6", "tree": "0eb215813f3066bf634a395d02cf4aab3ba94fc6", "parents": [ "97a545ab6ce922a0f868d192718a48a0091ebc5e" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Jan 28 00:42:23 2010 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:25 2010 +1030" }, "message": "virtio: use smp_XX barriers on SMP\n\nvirtio is communicating with a virtual \"device\" that actually runs on\nanother host processor. Thus SMP barriers can be used to control\nmemory access ordering.\n\nWhere possible, we should use SMP barriers which are more lightweight than\nmandatory barriers, because mandatory barriers also control MMIO effects on\naccesses through relaxed memory I/O windows (which virtio does not use)\n(compare specifically smp_rmb and rmb on x86_64).\n\nWe can\u0027t just use smp_mb and friends though, because\nwe must force memory ordering even if guest is UP since host could be\nrunning on another CPU, but SMP barriers are defined to barrier() in\nthat configuration. So, for UP fall back to mandatory barriers instead.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "97a545ab6ce922a0f868d192718a48a0091ebc5e", "tree": "1593d2a0f0ff19c4deb731d6079997e79989c8b1", "parents": [ "28ff4ef71795fe6f6c0cc06a04911c9ca09c2c11" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:22 2010 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:24 2010 +1030" }, "message": "virtio: remove bogus barriers from DEBUG version of virtio_ring.c\n\nWith DEBUG defined, we add an -\u003ein_use flag to detect if the caller\ninvokes two virtio methods in parallel. The barriers attempt to ensure\ntimely update of the -\u003ein_use flag.\n\nBut they\u0027re voodoo: if we need these barriers it implies that the\ncalling code doesn\u0027t have sufficient synchronization to ensure the\ncode paths aren\u0027t invoked at the same time anyway, and we want to\ndetect it.\n\nAlso, adding barriers changes timing, so turning on debug has more\nchance of hiding real problems.\n\nThanks to MST for drawing my attention to this code...\n\nCC: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "169c246a30808588436794e96a97c61a01af9bed", "tree": "cce7c1e1ca52687f711a18f46ea741841f43ea34", "parents": [ "1f34c71afe5115e77a49c4e67720a66e27053e54" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:14 2010 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:17 2010 +1030" }, "message": "virtio: fix balloon without VIRTIO_BALLOON_F_STATS_VQ\n\nWhen running under qemu-kvm-0.11.0:\n\n\tBUG: unable to handle kernel paging request at 56e58955\n\t...\n\tProcess vballoon (pid: 1297, ti\u003dc7976000 task\u003dc70a6ca0 task.ti\u003dc7\n\t...\n\tCall Trace:\n\t [\u003cc88253a3\u003e] ? balloon+0x1b3/0x440 [virtio_balloon]\n\t [\u003cc041c2d7\u003e] ? schedule+0x327/0x9d0\n\t [\u003cc88251f0\u003e] ? balloon+0x0/0x440 [virtio_balloon]\n\t [\u003cc014a2d4\u003e] ? kthread+0x74/0x80\n\t [\u003cc014a260\u003e] ? kthread+0x0/0x80\n\t [\u003cc0103b36\u003e] ? kernel_thread_helper+0x6/0x30\n\nneed_stats_update should be zero-initialized.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nAcked-by: Adam Litke \u003cagl@us.ibm.com\u003e\n" }, { "commit": "1f34c71afe5115e77a49c4e67720a66e27053e54", "tree": "f6481c9a5fbafffd2bd8cbf17274fdc75ffc94b7", "parents": [ "9564e138b1f6eb137f7149772438d3f3fb3277dd" ], "author": { "name": "Adam Litke", "email": "agl@us.ibm.com", "time": "Thu Dec 10 16:35:15 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:14 2010 +1030" }, "message": "virtio: Fix scheduling while atomic in virtio_balloon stats\n\nThis is a fix for my earlier patch: \"virtio: Add memory statistics reporting to\nthe balloon driver (V4)\".\n\nI discovered that all_vm_events() can sleep and therefore stats collection\ncannot be done in interrupt context. One solution is to handle the interrupt\nby noting that stats need to be collected and waking the existing vballoon\nkthread which will complete the work via stats_handle_request(). Rusty, is\nthis a saner way of doing business?\n\nThere is one issue that I would like a broader opinion on. In stats_request, I\nupdate vb-\u003eneed_stats_update and then wake up the kthread. The kthread uses\nvb-\u003eneed_stats_update as a condition variable. Do I need a memory barrier\nbetween the update and wake_up to ensure that my kthread sees the correct\nvalue? My testing suggests that it is not needed but I would like some\nconfirmation from the experts.\n\nSigned-off-by: Adam Litke \u003cagl@us.ibm.com\u003e\nTo: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Anthony Liguori \u003caliguori@linux.vnet.ibm.com\u003e\nCc: linux-kernel@vger.kernel.org\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "9564e138b1f6eb137f7149772438d3f3fb3277dd", "tree": "2e53562c2b7ab7051d448b8f81766dc61ad524b5", "parents": [ "1f08b833ddbdb1c8e81c4b1053c2ebb7b89cb437" ], "author": { "name": "Adam Litke", "email": "agl@us.ibm.com", "time": "Mon Nov 30 10:14:15 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:08 2010 +1030" }, "message": "virtio: Add memory statistics reporting to the balloon driver (V4)\n\nChanges since V3:\n - Do not do endian conversions as they will be done in the host\n - Report stats that reference a quantity of memory in bytes\n - Minor coding style updates\n\nChanges since V2:\n - Increase stat field size to 64 bits\n - Report all sizes in kb (not pages)\n - Drop anon_pages stat and fix endianness conversion\n\nChanges since V1:\n - Use a virtqueue instead of the device config space\n\nWhen using ballooning to manage overcommitted memory on a host, a system for\nguests to communicate their memory usage to the host can provide information\nthat will minimize the impact of ballooning on the guests. The current method\nemploys a daemon running in each guest that communicates memory statistics to a\nhost daemon at a specified time interval. The host daemon aggregates this\ninformation and inflates and/or deflates balloons according to the level of\nhost memory pressure. This approach is effective but overly complex since a\ndaemon must be installed inside each guest and coordinated to communicate with\nthe host. A simpler approach is to collect memory statistics in the virtio\nballoon driver and communicate them directly to the hypervisor.\n\nThis patch enables the guest-side support by adding stats collection and\nreporting to the virtio balloon driver.\n\nSigned-off-by: Adam Litke \u003cagl@us.ibm.com\u003e\nCc: Anthony Liguori \u003canthony@codemonkey.ws\u003e\nCc: virtualization@lists.linux-foundation.org\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (minor fixes)\n" }, { "commit": "1f08b833ddbdb1c8e81c4b1053c2ebb7b89cb437", "tree": "d6f9a8e468e177ca5ab97c233d827f4698ba8757", "parents": [ "9f3a6284880ceea452903e2043c88d7226736318" ], "author": { "name": "Jamie Lokier", "email": "jamie@shareable.org", "time": "Fri Jan 08 22:01:43 2010 +0000" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Feb 24 14:22:04 2010 +1030" }, "message": "Add __devexit_p around reference to virtio_pci_remove\n\nThis is needed to compile with CONFIG_VIRTIO_PCI\u003dy,\nbecause virtio_pci_remove is marked __devexit.\n\nSigned-off-by: Jamie Lokier \u003cjamie@shareable.org\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "d817cd525589765aa5f6798734e422c867685a58", "tree": "bc291ba32d55f79a7a8d499d90e6b9c41b4e1188", "parents": [ "de3fab39348dff18c69a0cd04efee9c276a02f51" ], "author": { "name": "Jeff Mahoney", "email": "jeffm@suse.com", "time": "Fri Jan 15 17:01:26 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sat Jan 16 12:15:39 2010 -0800" }, "message": "virtio: fix section mismatch warnings\n\nFix fixes the following warnings by renaming the driver structures to be\nsuffixed with _driver.\n\nWARNING: drivers/virtio/virtio_balloon.o(.data+0x88): Section mismatch in reference from the variable virtio_balloon to the function .devexit.text:virtballoon_remove()\n\nWARNING: drivers/char/hw_random/virtio-rng.o(.data+0x88): Section mismatch in reference from the variable virtio_rng to the function .devexit.text:virtrng_remove()\n\nSigned-off-by: Jeff Mahoney \u003cjeffm@suse.com\u003e\nAcked-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "2d61ba95034f1abbdec7729d52c740870a5eddb6", "tree": "ec51d4cba4240af6011f4fabfda06a21532761da", "parents": [ "0b22bd0ba0d500ab40e7673c591c594ec5447342" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Sun Oct 25 15:28:53 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Oct 29 08:50:37 2009 +1030" }, "message": "virtio: order used ring after used index read\n\nOn SMP guests, reads from the ring might bypass used index reads. This\ncauses guest crashes because host writes to used index to signal ring\ndata readiness. Fix this by inserting rmb before used ring reads.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: stable@kernel.org\n" }, { "commit": "0b22bd0ba0d500ab40e7673c591c594ec5447342", "tree": "09146fc159cc456d79f20f5fb66cbc7bfa8c9871", "parents": [ "964fe080d94db82a3268443e9b9ece4c60246414" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Oct 22 15:06:06 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Oct 29 08:50:36 2009 +1030" }, "message": "virtio-pci: fix per-vq MSI-X request logic\n\nCommit f68d24082e22ccee3077d11aeb6dc5354f0ca7f1\nin 2.6.32-rc1 broke requesting IRQs for per-VQ MSI-X vectors:\n- vector number was used instead of the vector itself\n- we try to request an IRQ for VQ which does not\n have a callback handler\n\nThis is a regression that causes warnings in kernel log,\npotentially lower performance as we need to scan vq list,\nand might cause system failure if the interrupt\nrequested is in fact needed by another system.\n\nThis was not noticed earlier because in most cases\nwe were falling back on shared interrupt for all vqs.\n\nThe warnings often look like this:\n\nvirtio-pci 0000:00:03.0: irq 26 for MSI/MSI-X\nvirtio-pci 0000:00:03.0: irq 27 for MSI/MSI-X\nvirtio-pci 0000:00:03.0: irq 28 for MSI/MSI-X\nIRQ handler type mismatch for IRQ 1\ncurrent handler: i8042\nPid: 2400, comm: modprobe Tainted: G W\n2.6.32-rc3-11952-gf3ed8d8-dirty #1\nCall Trace:\n [\u003cffffffff81072aed\u003e] ? __setup_irq+0x299/0x304\n [\u003cffffffff81072ff3\u003e] ? request_threaded_irq+0x144/0x1c1\n [\u003cffffffff813455af\u003e] ? vring_interrupt+0x0/0x30\n [\u003cffffffff81346598\u003e] ? vp_try_to_find_vqs+0x583/0x5c7\n [\u003cffffffffa0015188\u003e] ? skb_recv_done+0x0/0x34 [virtio_net]\n [\u003cffffffff81346609\u003e] ? vp_find_vqs+0x2d/0x83\n [\u003cffffffff81345d00\u003e] ? vp_get+0x3c/0x4e\n [\u003cffffffffa0016373\u003e] ? virtnet_probe+0x2f1/0x428 [virtio_net]\n [\u003cffffffffa0015188\u003e] ? skb_recv_done+0x0/0x34 [virtio_net]\n [\u003cffffffffa00150d8\u003e] ? skb_xmit_done+0x0/0x39 [virtio_net]\n [\u003cffffffff8110ab92\u003e] ? sysfs_do_create_link+0xcb/0x116\n [\u003cffffffff81345cc2\u003e] ? vp_get_status+0x14/0x16\n [\u003cffffffff81345464\u003e] ? virtio_dev_probe+0xa9/0xc8\n [\u003cffffffff8122b11c\u003e] ? driver_probe_device+0x8d/0x128\n [\u003cffffffff8122b206\u003e] ? __driver_attach+0x4f/0x6f\n [\u003cffffffff8122b1b7\u003e] ? __driver_attach+0x0/0x6f\n [\u003cffffffff8122a9f9\u003e] ? bus_for_each_dev+0x43/0x74\n [\u003cffffffff8122a374\u003e] ? bus_add_driver+0xea/0x22d\n [\u003cffffffff8122b4a3\u003e] ? driver_register+0xa7/0x111\n [\u003cffffffffa001a000\u003e] ? init+0x0/0xc [virtio_net]\n [\u003cffffffff81009051\u003e] ? do_one_initcall+0x50/0x148\n [\u003cffffffff8106e117\u003e] ? sys_init_module+0xc5/0x21a\n [\u003cffffffff8100af02\u003e] ? system_call_fastpath+0x16/0x1b\nvirtio-pci 0000:00:03.0: irq 26 for MSI/MSI-X\nvirtio-pci 0000:00:03.0: irq 27 for MSI/MSI-X\n\nReported-by: Marcelo Tosatti \u003cmtosatti@redhat.com\u003e\nReported-by: Shirley Ma \u003cxma@us.ibm.com\u003e\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "1e65175c2c73742495f0e5ca52658539a65825db", "tree": "4e6538fa79a04ff5f6666cde1e8b84f7857192a8", "parents": [ "3225beaba05d4f06087593f5e903ce867b6e118a" ], "author": { "name": "Uwe Kleine-König", "email": "u.kleine-koenig@pengutronix.de", "time": "Thu Oct 01 10:28:33 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Oct 22 16:39:31 2009 +1030" }, "message": "move virtballoon_remove to .devexit.text\n\nThe function virtballoon_remove is used only wrapped by __devexit_p so\ndefine it using __devexit.\n\nSigned-off-by: Uwe Kleine-König \u003cu.kleine-koenig@pengutronix.de\u003e\nAcked-by: Sam Ravnborg \u003csam@ravnborg.org\u003e\nAcked-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "e95646c3ec33c8ec0693992da4332a6b32eb7e31", "tree": "7855767cc4dccdefc4ea64584cc01b64f92176f3", "parents": [ "f8b12e513b953aebf30f8ff7d2de9be7e024dbbe" ], "author": { "name": "Christian Borntraeger", "email": "borntraeger@de.ibm.com", "time": "Wed Sep 30 11:17:21 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Oct 22 16:39:28 2009 +1030" }, "message": "virtio: let header files include virtio_ids.h\n\nRusty,\n\ncommit 3ca4f5ca73057a617f9444a91022d7127041970a\n virtio: add virtio IDs file\nmoved all device IDs into a single file. While the change itself is\na very good one, it can break userspace applications. For example\nif a userspace tool wanted to get the ID of virtio_net it used to\ninclude virtio_net.h. This does no longer work, since virtio_net.h\ndoes not include virtio_ids.h.\nThis patch moves all \"#include \u003clinux/virtio_ids.h\u003e\" from the C\nfiles into the header files, making the header files compatible with\nthe old ones.\n\nIn addition, this patch exports virtio_ids.h to userspace.\n\nCC: Fernando Luis Vazquez Cao \u003cfernando@oss.ntt.co.jp\u003e\nSigned-off-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "3ca4f5ca73057a617f9444a91022d7127041970a", "tree": "382659da29ad0a2e9878df6c7c4204913ab642e7", "parents": [ "3a20210dc26bbfff3bbb48bb22d2846240b71d8f" ], "author": { "name": "Fernando Luis Vazquez Cao", "email": "fernando@oss.ntt.co.jp", "time": "Fri Jul 31 15:25:56 2009 +0900" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Sep 23 22:26:32 2009 +0930" }, "message": "virtio: add virtio IDs file\n\nVirtio IDs are spread all over the tree which makes assigning new IDs\nbothersome. Putting them together should make the process less error-prone.\n\nSigned-off-by: Fernando Luis Vazquez Cao \u003cfernando@oss.ntt.co.jp\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "3c1b27d5043086a485f8526353ae9fe37bfa1065", "tree": "e3b6eda3c66bcd1fc3af6e7fa6e4e3af77459474", "parents": [ "f68d24082e22ccee3077d11aeb6dc5354f0ca7f1" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Sep 23 22:26:31 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Sep 23 22:26:31 2009 +0930" }, "message": "virtio: make add_buf return capacity remaining\n\nThis API change means that virtio_net can tell how much capacity\nremains for buffers. It\u0027s necessarily fuzzy, since\nVIRTIO_RING_F_INDIRECT_DESC means we can fit any number of descriptors\nin one, *if* we can kmalloc.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Dinesh Subhraveti \u003cdineshs@us.ibm.com\u003e\n" }, { "commit": "f68d24082e22ccee3077d11aeb6dc5354f0ca7f1", "tree": "22e2ab313dda8217926b0ddf0c760554391ee092", "parents": [ "ebc79c4f8da0f92efa968e0328f32334a2ce80cf" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Sep 23 22:26:29 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Wed Sep 23 22:26:31 2009 +0930" }, "message": "virtio_pci: minor MSI-X cleanups\n\n1) Rename vp_request_vectors to vp_request_msix_vectors, and take\n non-MSI-X case out to caller.\n2) Comment weird pci_enable_msix API\n3) Rename vp_find_vq to setup_vq.\n4) Fix spaces to tabs\n5) Make nvectors calc internal to vp_try_to_find_vqs()\n6) Rename vector to msix_vector for more clarity.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: \"Michael S. Tsirkin\" \u003cmst@redhat.com\u003e\n" }, { "commit": "e969fed542cae08cb11d666efac4f7c5d624d09f", "tree": "7de7cb45ba6b3a28ef0054df71c7dc7676d4af65", "parents": [ "f6c82507030d61e15928d5cad946d3eac1c4a384" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Sun Jul 26 15:48:08 2009 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Jul 30 16:03:45 2009 +0930" }, "message": "virtio: refactor find_vqs\n\nThis refactors find_vqs, making it more readable and robust, and fixing\ntwo regressions from 2.6.30:\n- double free_irq causing BUG_ON on device removal\n- probe failure when vq can\u0027t be assigned to msi-x vector\n (reported on old host kernels)\n\nTested-by: Amit Shah \u003camit.shah@redhat.com\u003e\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "f6c82507030d61e15928d5cad946d3eac1c4a384", "tree": "5c0dd65a04424dbcf985a4104a8a244085e2f179", "parents": [ "ff52c3fc7188855ede75d87b022271f0da309e5b" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Sun Jul 26 15:48:01 2009 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Jul 30 16:03:44 2009 +0930" }, "message": "virtio: delete vq from list\n\nThis makes delete vq the reverse of find vq.\nThis is required to make it possible to retry find_vqs\nafter a failure, otherwise the list gets corrupted.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "ff52c3fc7188855ede75d87b022271f0da309e5b", "tree": "e10333755598cfbee6676f75dd87168ecdf2d5ce", "parents": [ "8ef562d112c82ec539775698f8b63ac5ec1bd766" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu Jul 23 14:57:37 2009 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Thu Jul 30 16:03:44 2009 +0930" }, "message": "virtio: fix memory leak on device removal\n\nMake vp_free_vectors do the reverse of vq_request_vectors.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "4b892e6582e3a4fe01f623aea386907270d5bf83", "tree": "82214468d537c8cf90b1fc0195bf33c688db3bfd", "parents": [ "d9ecdea7ed7467db32ec160f4eca46c279255606" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Tue Jul 07 08:26:45 2009 +0100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 17 21:47:47 2009 +0930" }, "message": "virtio-pci: correctly unregister root device on error\n\nIf pci_register_driver() fails we\u0027re incorrectly unregistering the root\ndevice with device_unregister() rather than root_device_unregister().\n\nReported-by: Don Zickus \u003cdzickus@redhat.com\u003e\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "e3353853730eb99c56b7b0aed1667d51c0e3699a", "tree": "d0da26c34a8ca12a57662830fd698ca6696f3805", "parents": [ "c89e80168ba1ed37627fe03116b0cf8474dcb7e0" ], "author": { "name": "Christian Borntraeger", "email": "borntraeger@de.ibm.com", "time": "Tue May 26 15:46:10 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:40 2009 +0930" }, "message": "virtio: enhance id_matching for virtio drivers\n\nThis patch allows a virtio driver to use VIRTIO_DEV_ANY_ID for the\ndevice id. This will be used by a test module that can be bound to\nany virtio device.\n\nSigned-off-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "c89e80168ba1ed37627fe03116b0cf8474dcb7e0", "tree": "3314d284cca77dc81913877d1cf58838186e7e85", "parents": [ "594de1dd6449f79c99e1ba4577ea0e4e06e2b405" ], "author": { "name": "Christian Borntraeger", "email": "borntraeger@de.ibm.com", "time": "Tue May 26 15:46:09 2009 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:40 2009 +0930" }, "message": "virtio: fix id_matching for virtio drivers\n\nThis bug never appeared, since all current virtio drivers use\nVIRTIO_DEV_ANY_ID for the vendor field. If a real vendor would be used,\nthe check in virtio_id_match is wrong - it returns 0 if\nid-\u003evendor \u003d\u003d dev-\u003eid.vendor.\n\nSigned-off-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "9fa29b9df32ba4db055f3977933cd0c1b8fe67cd", "tree": "c44177d2d352f74300fe9124decc6462c8804bbe", "parents": [ "ee006b353f1ca8c9a8470b72b462beb011d62e32" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Mon May 11 18:11:45 2009 +0100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:39 2009 +0930" }, "message": "virtio: indirect ring entries (VIRTIO_RING_F_INDIRECT_DESC)\n\nAdd a new feature flag for indirect ring entries. These are ring\nentries which point to a table of buffer descriptors.\n\nThe idea here is to increase the ring capacity by allowing a larger\neffective ring size whereby the ring size dictates the number of\nrequests that may be outstanding, rather than the size of those\nrequests.\n\nThis should be most effective in the case of block I/O where we can\npotentially benefit by concurrently dispatching a large number of\nlarge requests. Even in the simple case of single segment block\nrequests, this results in a threefold increase in ring capacity.\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "a92892825a122a74ddad1d408fa27132e28b05ae", "tree": "c8dad18eb7842a10a33cdf632b5e0a0aa8fdf778", "parents": [ "82af8ce84ed65d2fb6d8c017d3f2bbbf161061fb" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:37 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:38 2009 +0930" }, "message": "virtio: expose features in sysfs\n\nEach device negotiates feature bits; expose these in sysfs to help\ndiagnostics and debugging.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "82af8ce84ed65d2fb6d8c017d3f2bbbf161061fb", "tree": "aed59c86724c7d3f565d1c60b38d25a7bf20985b", "parents": [ "77cf524654a886e0fbbf03b16b44f048deef7b0c" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu May 14 13:55:41 2009 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:37 2009 +0930" }, "message": "virtio_pci: optional MSI-X support\n\nThis implements optional MSI-X support in virtio_pci.\nMSI-X is used whenever the host supports at least 2 MSI-X\nvectors: 1 for configuration changes and 1 for virtqueues.\nPer-virtqueue vectors are allocated if enough vectors\navailable.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nAcked-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (+ whitespace, style)\n" }, { "commit": "77cf524654a886e0fbbf03b16b44f048deef7b0c", "tree": "4b55884ceade70463c53517eae382f0412cc317c", "parents": [ "d2a7ddda9ffb1c8961abff6714b0f1eb925c120f" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Thu May 14 13:55:31 2009 +0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:37 2009 +0930" }, "message": "virtio_pci: split up vp_interrupt\n\nThis reorganizes virtio-pci code in vp_interrupt slightly, so that\nit\u0027s easier to add per-vq MSI support on top.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "d2a7ddda9ffb1c8961abff6714b0f1eb925c120f", "tree": "1090884fd260d042255255467367e4e6b6193e5d", "parents": [ "9499f5e7ed5224c40706f0cec6542a9916bc7606" ], "author": { "name": "Michael S. Tsirkin", "email": "mst@redhat.com", "time": "Fri Jun 12 22:16:36 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:36 2009 +0930" }, "message": "virtio: find_vqs/del_vqs virtio operations\n\nThis replaces find_vq/del_vq with find_vqs/del_vqs virtio operations,\nand updates all drivers. This is needed for MSI support, because MSI\nneeds to know the total number of vectors upfront.\n\nSigned-off-by: Michael S. Tsirkin \u003cmst@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (+ lguest/9p compile fixes)\n" }, { "commit": "9499f5e7ed5224c40706f0cec6542a9916bc7606", "tree": "3e4e1b36d3d549ea356e88e6e44359a887c6ee01", "parents": [ "ef688e151c00e5d529703be9a04fd506df8bc54e" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:35 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:36 2009 +0930" }, "message": "virtio: add names to virtqueue struct, mapping from devices to queues.\n\nAdd a linked list of all virtqueues for a virtio device: this helps for\ndebugging and is also needed for upcoming interface change.\n\nAlso, add a \"name\" field for clearer debug messages.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "ef688e151c00e5d529703be9a04fd506df8bc54e", "tree": "841d0b0bcd8a5f72f8d8bdc455672d9a6e7468ca", "parents": [ "20f77f5654042cf484d8964b618faf9d620f639b" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:35 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jun 12 22:16:35 2009 +0930" }, "message": "virtio: meet virtio spec by finalizing features before using device\n\nVirtio devices are supposed to negotiate features before they start using\nthe device, but the current code doesn\u0027t do this. This is because the\ndriver\u0027s probe() function invariably has to add buffers to a virtqueue,\nor probe the disk (virtio_blk).\n\nThis currently doesn\u0027t matter since no existing backend is strict about\nthe feature negotiation. But it\u0027s possible to imagine a future feature\nwhich completely changes how a device operates: in this case, we\u0027d need\nto acknowledge it before using the device.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "84a139a985300901dfad99bd93c7345d180af860", "tree": "62a8921435f17551db2f26c8f0755f4ab508f537", "parents": [ "a489f0b555b753f9df8ddc24c7e74f657ef7ee7b" ], "author": { "name": "Marcelo Tosatti", "email": "mtosatti@redhat.com", "time": "Thu Apr 16 21:14:04 2009 -0300" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Sun Apr 19 23:14:01 2009 +0930" }, "message": "virtio: fix suspend when using virtio_balloon\n\nBreak out of wait_event_interruptible() if freezing has been requested,\nin the vballoon thread. Without this change vballoon refuses to stop and\nthe system can\u0027t suspend.\n\nSigned-off-by: Marcelo Tosatti \u003cmtosatti@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: stable@kernel.org\n" }, { "commit": "c5f841f1780dad7efb7eca092f60742d47f47d25", "tree": "6a27622d96f5327408c75b914c3a54120dc1d866", "parents": [ "3a35ce7dcefe9e80a00603a195269fbaf6e7d901" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 30 21:55:22 2009 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 30 21:55:23 2009 +1030" }, "message": "virtio: more neatening of virtio_ring macros.\n\nImpact: cleanup\n\nRoel Kluin drew attention to these macros with his patch: here I\nneaten them a little further:\n1) Add a comment on what START_USE and END_USE are checking,\n2) Brackets around _vq in BAD_RING,\n3) Neaten formatting for START_USE so it\u0027s less than 80 cols.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "3a35ce7dcefe9e80a00603a195269fbaf6e7d901", "tree": "23d814e318c36d7a9fe601be2f535da45bceec56", "parents": [ "0d34fb8e93ceba7b6dad0062dbb4a0813bacd75b" ], "author": { "name": "Roel Kluin", "email": "roel.kluin@gmail.com", "time": "Thu Jan 22 16:42:57 2009 +0100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 30 21:55:22 2009 +1030" }, "message": "virtio: fix BAD_RING, START_US and END_USE macros\n\nImpact: cleanup\n\nfix BAD_RING, START_US and END_USE macros\n\nWhen these macros aren\u0027t called with a variable named vq as first\nargument, this would result in a build failure.\n\nSigned-off-by: Roel Kluin \u003croel.kluin@gmail.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "3fff0179e33cd7d0a688dab65700c46ad089e934", "tree": "34c30ceedb3c5ae5cb6a5fb8d1c62aed7c845054", "parents": [ "720eba31f47aeade8ec130ca7f4353223c49170f" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Tue Feb 03 13:33:53 2009 +1030" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Feb 02 19:17:56 2009 -0800" }, "message": "virtio-pci: do not oops on config change if driver not loaded\n\nThe host really shouldn\u0027t be notifying us of config changes\nbefore the device status is VIRTIO_CONFIG_S_DRIVER or\nVIRTIO_CONFIG_S_DRIVER_OK.\n\nHowever, if we do happen to be interrupted while we\u0027re not\nattached to a driver, we really shouldn\u0027t oops. Prevent\nthis simply by checking that device-\u003edriver is non-NULL\nbefore trying to notify the driver of config changes.\n\nProblem observed by doing a \"set_link virtio.0 down\" with\nQEMU before the net driver had been loaded.\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "63d12556703f17817a46e140704360b29b851bad", "tree": "be9741e515a075010e2fbb217ea0c41b85613ca2", "parents": [ "0aa0dc41bfd993491c2344870eee7a3b218551fb" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Mon Dec 15 12:58:27 2008 +0000" }, "committer": { "name": "Greg Kroah-Hartman", "email": "gregkh@suse.de", "time": "Tue Jan 06 10:44:34 2009 -0800" }, "message": "virtio: do not statically allocate root device\n\nWe shouldn\u0027t be statically allocating the root device object,\nso dynamically allocate it using root_device_register()\ninstead.\n\nAlso avoids this warning from \u0027rmmod virtio_pci\u0027:\n\n Device \u0027virtio-pci\u0027 does not have a release() function, it is broken and must be fixed\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nCc: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nAcked-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\n\n" }, { "commit": "29f9f12ec737af62835124e4a8bdb9de631f04dd", "tree": "2323188170fb6462d45f34e4dfd2f34120057a76", "parents": [ "b194aee95622f649311f8e53418a17e210ff6827" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Wed Dec 10 17:45:34 2008 +0000" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:26:10 2008 +1030" }, "message": "virtio: add PCI device release() function\n\nAdd a release() function for virtio_pci devices so as to avoid:\n\n Device \u0027virtio0\u0027 does not have a release() function, it is broken and must be fixed\n\nMove the code to free the resources associated with the device\nfrom virtio_pci_remove() into this new function. virtio_pci_remove()\nnow merely unregisters the device which should cause the final\nref to be dropped and virtio_pci_release_dev() to be called.\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nReported-by: Michael Tokarev \u003cmjt@tls.msk.ru\u003e\nCc: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "1b4aa2faeca1b9922033daf2475b6fc13b0ffea6", "tree": "b94e3f4ec45151e51ad1b3dfecabc5391684c725", "parents": [ "87c7d57c17ade5024d95b6ca0da249da49b0672a" ], "author": { "name": "Hollis Blanchard", "email": "hollisb@us.ibm.com", "time": "Thu Nov 13 15:48:33 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:26:04 2008 +1030" }, "message": "virtio: avoid implicit use of Linux page size in balloon interface\n\nMake the balloon interface always use 4K pages, and convert Linux pfns if\nnecessary. This patch assumes that Linux\u0027s PAGE_SHIFT will never be less than\n12.\n\nSigned-off-by: Hollis Blanchard \u003chollisb@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (modified)\n" }, { "commit": "87c7d57c17ade5024d95b6ca0da249da49b0672a", "tree": "2ffeaae5c2e5ed9a72efec6466678fb797c4cdfa", "parents": [ "db40598863e8cbbd11053ad3c8bae89000f603f9" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:26:03 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:26:03 2008 +1030" }, "message": "virtio: hand virtio ring alignment as argument to vring_new_virtqueue\n\nThis allows each virtio user to hand in the alignment appropriate to\ntheir virtio_ring structures.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nAcked-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\n" }, { "commit": "498af14783935af487d17dbee4ac451783cbc2b7", "tree": "db7a61d88f8eee05e65ac832f571440640299869", "parents": [ "5f0d1d7f2286c8a02dab69f5f0bd51681fab161e" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:57 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:58 2008 +1030" }, "message": "virtio: Don\u0027t use PAGE_SIZE for vring alignment in virtio_pci.\n\nThat doesn\u0027t work for non-4k guests which are now appearing.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "480daab42c4dd74b3c07031ddf9031251c530c77", "tree": "c491b040071be3867aa42942a80435866fbc85cf", "parents": [ "99e0b6c8e3f30602383bcfe3f574537a02ee2bea" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:56 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:57 2008 +1030" }, "message": "virtio: Don\u0027t use PAGE_SIZE in virtio_pci.c\n\nThe virtio PCI devices don\u0027t depend on the guest page size. This matters\nnow PowerPC virtio is gaining ground (they like 64k pages).\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "99e0b6c8e3f30602383bcfe3f574537a02ee2bea", "tree": "83adb7fe6cd869df901c448602872554ed7edbdf", "parents": [ "13b1eb333beed018140be3d0b77bf34000125a34" ], "author": { "name": "Kay Sievers", "email": "kay.sievers@vrfy.org", "time": "Tue Dec 30 09:25:56 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:56 2008 +1030" }, "message": "virtio: struct device - replace bus_id with dev_name(), dev_set_name()\n\nThis patch is part of a larger patch series which will remove\nthe \"char bus_id[20]\" name string from struct device. The device\nname is managed in the kobject anyway, and without any size\nlimitation, and just needlessly copied into \"struct device\".\n\nTo set and read the device name dev_name(dev) and dev_set_name(dev)\nmust be used. If your code uses static kobjects, which it shouldn\u0027t\ndo, \"const char *init_name\" can be used to statically provide the\nname the registered device should have. At registration time, the\ninit_name field is cleared, to enforce the use of dev_name(dev) to\naccess the device name at a later time.\n\nWe need to get rid of all occurrences of bus_id in the entire tree\nto be able to enable the new interface. Please apply this patch,\nand possibly convert any remaining remaining occurrences of bus_id.\n\nWe want to submit a patch to -next, which will remove bus_id from\n\"struct device\", to find the remaining pieces to convert, and finally\nswitch over to the new api, which will remove the 20 bytes array\nand does no longer have a size limitation.\n\nAcked-by: Greg Kroah-Hartman \u003cgregkh@suse.de\u003e\nSigned-off-by: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "13b1eb333beed018140be3d0b77bf34000125a34", "tree": "26d84c1c955ad87687266433e8e54fa6f88bd150", "parents": [ "3c92ec8ae91ecf59d88c798301833d7cf83f2179" ], "author": { "name": "Hollis Blanchard", "email": "hollisb@us.ibm.com", "time": "Tue Dec 02 16:24:40 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Dec 30 09:25:56 2008 +1030" }, "message": "virtio-pci queue allocation not page-aligned\n\nkzalloc() does not guarantee page alignment, and in fact this broke when\nI enabled CONFIG_SLUB_DEBUG_ON.\n\n(Thanks to Anthony Liguori for spotting the missing kfree sub)\n\nSigned-off-by: Hollis Blanchard \u003chollisb@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (fixed kfree)\nTested-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\n" }, { "commit": "532a6086e35fa3b5761e68af36d4e42a550eba15", "tree": "c1d8ab4df9db25e0efd89dab64dda98f84df1fa6", "parents": [ "83097aca8567a0bd593534853b71fe0fa9a75d69" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Mon Aug 18 17:15:31 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Aug 26 00:19:25 2008 +1000" }, "message": "virtio_balloon: fix towards_target when deflating balloon\n\nBoth v and vb-\u003enum_pages are u32 and unsigned int respectively. If v is less\nthan vb-\u003enum_pages (and it is, when deflating the balloon), the result is a\nvery large 32-bit number. Since we\u0027re returning a s64, instead of getting the\nsame negative number we desire, we get a very large positive number.\n\nThis handles the case where v \u003c vb-\u003enum_pages and ensures we get a small,\nnegative, s64 as the result.\n\nRusty: please push this for 2.6.27-rc4. It\u0027s probably appropriate for the\nstable tree too as it will cause an unexpected OOM when ballooning.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (simplified)\n" }, { "commit": "e34f87256794b87e7f4a8f1812538be7b7b5214c", "tree": "b91ffb6df4ac13ace269bef60ae16a1048b1fa73", "parents": [ "c624896e488ba2bff5ae497782cfb265c8b00646" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:13 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:14 2008 +1000" }, "message": "virtio: Add transport feature handling stub for virtio_ring.\n\nTo prepare for virtio_ring transport feature bits, hook in a call in\nall the users to manipulate them. This currently just clears all the\nbits, since it doesn\u0027t understand any features.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "c624896e488ba2bff5ae497782cfb265c8b00646", "tree": "714145903ff8c889e9544e8366b2ee2a62234c0c", "parents": [ "dd7c7bc46211785a1aa7d70feb15830f62682b3c" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:07 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:12 2008 +1000" }, "message": "virtio: Rename set_features to finalize_features\n\nRather than explicitly handing the features to the lower-level, we just\nhand the virtio_device and have it set the features. This make it clear\nthat it has the chance to manipulate the features of the device at this\npoint (and that all feature negotiation is already done).\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "dd7c7bc46211785a1aa7d70feb15830f62682b3c", "tree": "3c615eac390c11339c652947e744d12cb82b4d36", "parents": [ "faeba830b086bc9e58748869054e994cb09693cd" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:07 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:07 2008 +1000" }, "message": "virtio: Formally reserve bits 28-31 to be \u0027transport\u0027 features.\n\nWe assign feature bits as required, but it makes sense to reserve some\nfor the particular transport, rather than the particular device.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "e962fa660d391fc9b90988e6538c94c858c099f9", "tree": "2e498cb10c9a416192c769bec8119455afcb18ba", "parents": [ "44653eae1407f79dff6f52fcf594ae84cb165ec4" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Fri Jun 13 13:46:40 2008 +0100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:05 2008 +1000" }, "message": "virtio: Use bus_type probe and remove methods\n\nHook up to the probe() and remove() methods in bus_type\nrather than device_driver. The latter has been preferred\nsince 2.6.16.\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "44653eae1407f79dff6f52fcf594ae84cb165ec4", "tree": "9c8f356f6c294678a05dbc983e60b8b5d864b7bc", "parents": [ "674bfc23c585b34c42263d73fb51710d49762a23" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:04 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Jul 25 12:06:04 2008 +1000" }, "message": "virtio: don\u0027t always force a notification when ring is full\n\nWe force notification when the ring is full, even if the host has\nindicated it doesn\u0027t want to know. This seemed like a good idea at\nthe time: if we fill the transmit ring, we should tell the host\nimmediately.\n\nUnfortunately this logic also applies to the receiving ring, which is\nrefilled constantly. We should introduce real notification thesholds\nto replace this logic. Meanwhile, removing the logic altogether breaks\nthe heuristics which KVM uses, so we use a hack: only notify if there are\noutgoing parts of the new buffer.\n\nHere are the number of exits with lguest\u0027s crappy network implementation:\nBefore:\n\tnetwork xmit 7859051 recv 236420\nAfter:\n\tnetwork xmit 7858610 recv 118136\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "b92dea67cc66970cda6b5b11895d08e35b4618e7", "tree": "556e842834c7316f57f3f27fea6b8622d74fde8e", "parents": [ "0269c5c6d9a9de22715ecda589730547435cd3e8" ], "author": { "name": "Mark McLoughlin", "email": "markmc@redhat.com", "time": "Sun Jun 15 23:20:50 2008 +1000" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jun 15 13:46:16 2008 -0700" }, "message": "virtio: Complete feature negotation before updating status\n\nlguest (in rusty\u0027s use-tun-ringfd patch) assumes that the\nguest has updated its feature bits before setting its status\nto VIRTIO_CONFIG_S_DRIVER_OK.\n\nThat\u0027s pretty reasonable, so let\u0027s make it so.\n\nSigned-off-by: Mark McLoughlin \u003cmarkmc@redhat.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b4f68be6c5d507afdcd74f5be3df0b1209cda503", "tree": "85c0771058ff08c5dab5eedbf3395959dbafc878", "parents": [ "7757f09c70af87887dfc195e6d6ddd54f5cc7c39" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:45 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:46 2008 +1000" }, "message": "virtio: force callback on empty.\n\nvirtio allows drivers to suppress callbacks (ie. interrupts) for\nefficiency (no locking, it\u0027s just an optimization).\n\nThere\u0027s a similar mechanism for the host to suppress notifications\ncoming from the guest: in that case, we ignore the suppression if the\nring is completely full.\n\nIt turns out that life is simpler if the host similarly ignores\ncallback suppression when the ring is completely empty: the network\ndriver wants to free up old packets in a timely manner, and otherwise\nhas to use a timer to poll.\n\nWe have to remove the code which ignores interrupts when the driver\nhas disabled them (again, it had no locking and hence was unreliable\nanyway).\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "52a3a05f3ab82655ffa4c9bf6835565c98a3c2e5", "tree": "f4257904896d265059d4a9f330e8bf5d3181c6f9", "parents": [ "f7f510ec195781c857ab76366a3e1c59e1caae42" ], "author": { "name": "Christian Borntraeger", "email": "borntraeger@de.ibm.com", "time": "Mon May 26 11:29:27 2008 +0200" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:45 2008 +1000" }, "message": "virtio_net: another race with virtio_net and enable_cb\n\nHello Rusty,\n\nseems that we still have a problem with virtio_net and the enable_cb callback.\nDuring a long running network stress tests with virtio and got the following\noops:\n\n------------[ cut here ]------------\nkernel BUG at drivers/virtio/virtio_ring.c:230!\nillegal operation: 0001 [#1] SMP\nModules linked in:\nCPU: 0 Not tainted 2.6.26-rc2-kvm-00436-gc94c08b-dirty #34\nProcess netserver (pid: 2582, task: 000000000fbc4c68, ksp: 000000000f42b990)\nKrnl PSW : 0704c00180000000 00000000002d0ec8 (vring_enable_cb+0x1c/0x60)\n R:0 T:1 IO:1 EX:1 Key:0 M:1 W:0 P:0 AS:3 CC:0 PM:0 EA:3\nKrnl GPRS: 0000000000000000 0000000000000000 000000000ef3d000 0000000010009800\n 0000000000000000 0000000000419ce0 0000000000000080 000000000000007b\n 000000000adb5538 000000000ef40900 000000000ef40000 000000000ef40920\n 0000000000000000 0000000000000005 000000000029c1b0 000000000fea7d18\nKrnl Code: 00000000002d0ebc: a7110001 tmll %r1,1\n 00000000002d0ec0: a7740004 brc 7,2d0ec8\n 00000000002d0ec4: a7f40001 brc 15,2d0ec6\n \u003e00000000002d0ec8: a517fffe nill %r1,65534\n 00000000002d0ecc: 40103000 sth %r1,0(%r3)\n 00000000002d0ed0: 07f0 bcr 15,%r0\n 00000000002d0ed2: e31020380004 lg %r1,56(%r2)\n 00000000002d0ed8: a7480000 lhi %r4,0\nCall Trace:\n([\u003c000000000029c0fc\u003e] virtnet_poll+0x290/0x3b8)\n [\u003c0000000000333fb8\u003e] net_rx_action+0x9c/0x1b8\n [\u003c00000000001394bc\u003e] __do_softirq+0x74/0x108\n [\u003c000000000010d16a\u003e] do_softirq+0x92/0xac\n [\u003c0000000000139826\u003e] irq_exit+0x72/0xc8\n [\u003c000000000010a7b6\u003e] do_extint+0xe2/0x104\n [\u003c0000000000110508\u003e] ext_no_vtime+0x16/0x1a\nLast Breaking-Event-Address:\n [\u003c00000000002d0ec4\u003e] vring_enable_cb+0x18/0x60\n\nI looked into the virtio_net code for some time and I think the following\nscenario happened. Please look at virtnet_poll:\n[...]\n /* Out of packets? */\n if (received \u003c budget) {\n netif_rx_complete(vi-\u003edev, napi);\n if (unlikely(!vi-\u003ervq-\u003evq_ops-\u003eenable_cb(vi-\u003ervq))\n \u0026\u0026 napi_schedule_prep(napi)) {\n vi-\u003ervq-\u003evq_ops-\u003edisable_cb(vi-\u003ervq);\n __netif_rx_schedule(vi-\u003edev, napi);\n goto again;\n }\n }\n\nIf an interrupt arrives after netif_rx_complete, a second poll routine can run\non a different cpu. The second check for napi_schedule_prep would prevent any\nharm in the network stack, but we have called enable_cb possibly after the\ndisable_cb in skb_recv_done.\n\nstatic void skb_recv_done(struct virtqueue *rvq)\n{\n struct virtnet_info *vi \u003d rvq-\u003evdev-\u003epriv;\n /* Schedule NAPI, Suppress further interrupts if successful. */\n if (netif_rx_schedule_prep(vi-\u003edev, \u0026vi-\u003enapi)) {\n rvq-\u003evq_ops-\u003edisable_cb(rvq);\n __netif_rx_schedule(vi-\u003edev, \u0026vi-\u003enapi);\n }\n}\n\nThat means that the second poll routine runs with interrupts enabled, which is\nok, since we can handle additional interrupts. The problem is now that the\nsecond poll routine might also call enable_cb, triggering the BUG.\n\nThe only solution I can come up with, is to remove the BUG statement in\nenable_cb - similar to disable_cb. Opinions or better ideas where the oops\ncould come from?\n\nSigned-off-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "b769f579081943f14e0ff03b7b0bd3a11cf14625", "tree": "026d89b6d937af43af4a692bcf547e032f0c58cc", "parents": [ "5610bd1524332fe7d651eb56cc780e32763a2ac3" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:42 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:42 2008 +1000" }, "message": "virtio: set device index in common code.\n\nAnthony Liguori points out that three different transports use the virtio code,\nbut each one keeps its own counter to set the virtio_device\u0027s index field. In\ntheory (though not in current practice) this means that names could be\nduplicated, and that risk grows as more transports are created.\n\nSo we move the selection of the unique virtio_device.index into the common code\nin virtio.c, which has the side-benefit of removing duplicate code.\n\nThe only complexity is that lguest and S/390 use the index to uniquely identify\nthe device in case of catastrophic failure before register_virtio_device() is\ncalled: now we use the offset within the descriptor page as a unique identifier\nfor the printks.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Carsten Otte \u003ccotte@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nCc: Chris Lalancette \u003cclalance@redhat.com\u003e\nCc: Anthony Liguori \u003canthony@codemonkey.ws\u003e\n" }, { "commit": "5610bd1524332fe7d651eb56cc780e32763a2ac3", "tree": "551e8ea4cbbf087f3fe23f10eebb075466369228", "parents": [ "2ad3cfbac58d0a6c6e65aafd9e0e757ca3d35292" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:42 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:42 2008 +1000" }, "message": "virtio: virtio_pci should not set bus_id.\n\nThe common virtio code sets the bus_id, overriding anything virtio_pci\nsets anyway.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Carsten Otte \u003ccotte@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nCc: Chris Lalancette \u003cclalance@redhat.com\u003e\nCc: Anthony Liguori \u003canthony@codemonkey.ws\u003e\n" }, { "commit": "2ad3cfbac58d0a6c6e65aafd9e0e757ca3d35292", "tree": "fd17937a5873c078487a02adc5dbdc8a175b0796", "parents": [ "ac9d463afb1ca2434335351f3b7d9e4c8f8470e9" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:41 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 30 15:09:42 2008 +1000" }, "message": "virtio: bus_id for devices should contain \u0027virtio\u0027\n\nChris Lalancette \u003cclalance@redhat.com\u003e points out that virtio.c sets all device\nnames to \u00270\u0027, \u00271\u0027, etc, which looks silly in /proc/interrupts. We change this\nfrom \u0027%d\u0027 to \u0027virtio%d\u0027.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Carsten Otte \u003ccotte@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nCc: Chris Lalancette \u003cclalance@redhat.com\u003e\nCc: Anthony Liguori \u003canthony@codemonkey.ws\u003e\n" }, { "commit": "c45a6816c19dee67b8f725e6646d428901a6dc24", "tree": "096e3263fd14e140685bcc3082394ff15f5aeddb", "parents": [ "72e61eb40b55dd57031ec5971e810649f82b0259" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:50 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:50 2008 +1000" }, "message": "virtio: explicit advertisement of driver features\n\nA recent proposed feature addition to the virtio block driver revealed\nsome flaws in the API: in particular, we assume that feature\nnegotiation is complete once a driver\u0027s probe function returns.\n\nThere is nothing in the API to require this, however, and even I\ndidn\u0027t notice when it was violated.\n\nSo instead, we require the driver to specify what features it supports\nin a table, we can then move the feature negotiation into the virtio\ncore. The intersection of device and driver features are presented in\na new \u0027features\u0027 bitmap in the struct virtio_device.\n\nNote that this highlights the difference between Linux unsigned-long\nbitmaps where each unsigned long is in native endian, and a\nstraight-forward little-endian array of bytes.\n\nDrivers can still remove feature bits in their probe routine if they\nreally have to.\n\nAPI changes:\n- dev-\u003econfig-\u003efeature() no longer gets and acks a feature.\n- drivers should advertise their features in the \u0027feature_table\u0027 field\n- use virtio_has_feature() for extra sanity when checking feature bits\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "72e61eb40b55dd57031ec5971e810649f82b0259", "tree": "66a836c7799b21156d4fc87f42e5817d7d95535b", "parents": [ "5539ae9613587e4a4eec42d420b8bdd9ff552a65" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:49 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:50 2008 +1000" }, "message": "virtio: change config to guest endian.\n\nA recent proposed feature addition to the virtio block driver revealed\nsome flaws in the API, in particular how easy it is to break big\nendian machines.\n\nThe virtio config space was originally chosen to be little-endian,\nbecause we thought the config might be part of the PCI config space\nfor virtio_pci. It\u0027s actually a separate mmio region, so that\nargument holds little water; as only x86 is currently using the virtio\nmechanism, we can change this (but must do so now, before the\nimpending s390 merge).\n\nAPI changes:\n- __virtio_config_val() just becomes a striaght vdev-\u003econfig_get() call.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "597d56e4b51fc3385e097e52d6e92bf596ff21ec", "tree": "7e5db5e9491ac0c24c1f2bea979dc9d957d7b537", "parents": [ "655aa31f028c4498e8896576571ee1ea68dd26e0" ], "author": { "name": "Harvey Harrison", "email": "harvey.harrison@gmail.com", "time": "Mon Mar 31 17:53:55 2008 -0700" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:44 2008 +1000" }, "message": "virtio: fix sparse return void-valued expression warnings\n\ndrivers/virtio/virtio_pci.c:148:2: warning: returning void-valued expression\ndrivers/virtio/virtio_pci.c:155:2: warning: returning void-valued expression\n\nSigned-off-by: Harvey Harrison \u003charvey.harrison@gmail.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "5ef827526fc01820a7a80827802e9fad3f34f937", "tree": "c41345c6de6ad5b4b4d7071aa46a9e27a773247c", "parents": [ "886c35fbcf6fb2eee15687efc2d64d99b6ad9a4a" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:43 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri May 02 21:50:43 2008 +1000" }, "message": "virtio: ignore corrupted virtqueues rather than spinning.\n\nA corrupt virtqueue (caused by the other end screwing up) can have\nstrange results such as a driver spinning: just bail when we try to\nget a buffer from a known-broken queue.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "2557a933b795c1988c721ebb871cd735128bb9cb", "tree": "96581bd5acac44bbe0399aa445197509d9ae60ff", "parents": [ "4cac04dd63fa3b202ee313ed1afbbd135ab887ee" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Apr 07 14:30:28 2008 +1000" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Apr 07 13:14:22 2008 -0700" }, "message": "virtio: remove overzealous BUG_ON.\n\nThe \u0027disable_cb\u0027 callback is designed as an optimization to tell the host\nwe don\u0027t need callbacks now. As it is not reliable, the debug check is\noverzealous: it can happen on two CPUs at the same time. Document this.\n\nEven if it were reliable, the virtio_net driver doesn\u0027t disable\ncallbacks on transmit so the START_USE/END_USE debugging reentrance\nprotection can be easily tripped even on UP.\n\nThanks to Balaji Rao for the bug report and testing.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCC: Balaji Rao \u003cbalajirrao@gmail.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "97968358ab98c2d5fc5c3071ba70bd6a5faecbb8", "tree": "09201cba8cdff97f11f14aa3ea0517942ccb2948", "parents": [ "b2ddb9019ea13fb7b62d8e45adcc468376af0de7" ], "author": { "name": "Al Viro", "email": "viro@ftp.linux.org.uk", "time": "Sat Mar 29 03:09:48 2008 +0000" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Mar 30 14:20:23 2008 -0700" }, "message": "virtio_pci iomem annotations\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "bd6c26900bae19bd51abedfc9a8a281afbba3a06", "tree": "cb2058d113d81fa4b168b8675f000f8aecbf8b26", "parents": [ "3085354de635179d70c240e6d942bcbd1d93056c" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Wed Mar 19 20:35:04 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Fri Mar 28 11:05:51 2008 +1100" }, "message": "virtio_pci: unregister virtio device at device remove\n\nMake sure to call unregister_virtio_device() when a virtio device is removed.\nOtherwise, virtio_pci.ko cannot be rmmod\u0027d.\n\nThis was spotted by Marcelo Tosatti.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "4265f161b6bb7b31163671329b1142b9023bf4e3", "tree": "1494768aaf38407ca8f1336f6ae80301b27135bb", "parents": [ "da74e89d40995600b3b07ac500084920247687ce" ], "author": { "name": "Christian Borntraeger", "email": "borntraeger@de.ibm.com", "time": "Fri Mar 14 14:17:05 2008 +0100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 17 22:58:21 2008 +1100" }, "message": "virtio: fix race in enable_cb\n\nThere is a race in virtio_net, dealing with disabling/enabling the callback.\nI saw the following oops:\n\nkernel BUG at /space/kvm/drivers/virtio/virtio_ring.c:218!\nillegal operation: 0001 [#1] SMP\nModules linked in: sunrpc dm_mod\nCPU: 2 Not tainted 2.6.25-rc1zlive-host-10623-gd358142-dirty #99\nProcess swapper (pid: 0, task: 000000000f85a610, ksp: 000000000f873c60)\nKrnl PSW : 0404300180000000 00000000002b81a6 (vring_disable_cb+0x16/0x20)\n R:0 T:1 IO:0 EX:0 Key:0 M:1 W:0 P:0 AS:0 CC:3 PM:0 EA:3\nKrnl GPRS: 0000000000000001 0000000000000001 0000000010005800 0000000000000001\n 000000000f3a0900 000000000f85a610 0000000000000000 0000000000000000\n 0000000000000000 000000000f870000 0000000000000000 0000000000001237\n 000000000f3a0920 000000000010ff74 00000000002846f6 000000000fa0bcd8\nKrnl Code: 00000000002b819a: a7110001 tmll %r1,1\n 00000000002b819e: a7840004 brc 8,2b81a6\n 00000000002b81a2: a7f40001 brc 15,2b81a4\n \u003e00000000002b81a6: a51b0001 oill %r1,1\n 00000000002b81aa: 40102000 sth %r1,0(%r2)\n 00000000002b81ae: 07fe bcr 15,%r14\n 00000000002b81b0: eb7ff0380024 stmg %r7,%r15,56(%r15)\n 00000000002b81b6: a7f13e00 tmll %r15,15872\nCall Trace:\n([\u003c000000000fa0bcd0\u003e] 0xfa0bcd0)\n [\u003c00000000002b8350\u003e] vring_interrupt+0x5c/0x6c\n [\u003c000000000010ab08\u003e] do_extint+0xb8/0xf0\n [\u003c0000000000110716\u003e] ext_no_vtime+0x16/0x1a\n [\u003c0000000000107e72\u003e] cpu_idle+0x1c2/0x1e0\n\nThe problem can be triggered with a high amount of host-\u003eguest traffic.\nI think its the following race:\n\npoll says netif_rx_complete\npoll calls enable_cb\nenable_cb opens the interrupt mask\na new packet comes, an interrupt is triggered----\\\nenable_cb sees that there is more work |\nenable_cb disables the interrupt |\n . V\n . interrupt is delivered\n . skb_recv_done does atomic napi test, ok\n some waiting disable_cb is called-\u003echeck fails-\u003ebang!\n .\npoll would do napi check\npoll would do disable_cb\n\nThe fix is to let enable_cb not disable the interrupt again, but expect the\ncaller to do the cleanup if it returns false. In that case, the interrupt is\nonly disabled, if the napi test_set_bit was successful.\n\nSigned-off-by: Christian Borntraeger \u003cborntraeger@de.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e (cleaned up doco)\n" }, { "commit": "bdc1681cdf1ab6a65fa935a2b3f8fc63b20c54ea", "tree": "17910bdca2cee81b0d4a3ee877aec82db0017a09", "parents": [ "c483934670d31e064e18967bb679c1079b54ea72" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 17 22:58:15 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 17 22:58:19 2008 +1100" }, "message": "virtio: handle \u003e 2 billion page balloon targets\n\nIf the host asks for a huge target towards_target() can overflow, and\nwe up oops as we try to release more pages than we have. The simple\nfix is to use a 64-bit value.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "27ebe308af7c14aed0711e25817c6a1235bd16ff", "tree": "dafd3caaa1591a60a7b79f7027c00bcae30f4a6e", "parents": [ "a978b30af3bab0dd9af9350eeda25e76123fa28e" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Sun Mar 02 16:37:48 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Mar 17 22:58:13 2008 +1100" }, "message": "virtio: Use spin_lock_irqsave/restore for virtio-pci\n\nvirtio-pci acquires its spin lock in an interrupt context so it\u0027s necessary\nto use spin_lock_irqsave/restore variants. This patch fixes guest SMP when\nusing virtio devices in KVM.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "6659a0f0bb7481d0f94ca3f203a4e1e406a9694d", "tree": "6803f9ad7ceb8556e8a07a8ba2e2d1a61e7283f1", "parents": [ "bd1939de9061dbc5cac44ffb4425aaf4c9b894f1" ], "author": { "name": "Johann Felix Soden", "email": "johfel@users.sourceforge.net", "time": "Wed Feb 06 01:40:22 2008 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@woody.linux-foundation.org", "time": "Wed Feb 06 10:41:21 2008 -0800" }, "message": "virtio: add missing #include \u003clinux/delay.h\u003e\n\nInclude linux/delay.h to fix compiler error:\n\ndrivers/virtio/virtio_balloon.c: In function \u0027fill_balloon\u0027:\ndrivers/virtio/virtio_balloon.c:98: error: implicit declaration of function \u0027msleep\u0027\n\nSigned-off-by: Johann Felix Soden \u003cjohfel@users.sourceforge.net\u003e\nCc: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6b35e40767c6c1ac783330109ae8e0c09ea6bc82", "tree": "e820d770894cb945c3070fa6739b0149b1bfa2f9", "parents": [ "55a7c066041e7850948d29ed813f62821a9ec046" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:12 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:13 2008 +1100" }, "message": "virtio: balloon driver\n\nAfter discussions with Anthony Liguori, it seems that the virtio\nballoon can be made even simpler. Here\u0027s my attempt.\n\nThe device configuration tells the driver how much memory it should\ntake from the guest (ie. balloon size). The guest feeds the page\nnumbers it has taken via one virtqueue.\n\nA second virtqueue feeds the page numbers the driver wants back: if\nthe device has the VIRTIO_BALLOON_F_MUST_TELL_HOST bit, then this\nqueue is compulsory, otherwise it\u0027s advisory (and the guest can simply\nfault the pages back in).\n\nThis driver can be enhanced later to deflate the balloon via a\nshrinker, oom callback or we could even go for a complete set of\nin-guest regulators.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "55a7c066041e7850948d29ed813f62821a9ec046", "tree": "ea4ef20c8014d5222957cef83d4fc62b75c58fd6", "parents": [ "3343660d8c62c6b00b2f15324ef3fcb6be207bfa" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Mon Jan 28 09:59:59 2008 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:12 2008 +1100" }, "message": "virtio: Use PCI revision field to indicate virtio PCI ABI version\n\nAs Avi pointed out, as we continue to massage the virtio PCI ABI, we can make\nthings a little more friendly to users by utilizing the PCI revision field to\nindicate which version of the ABI we\u0027re using. This is a hard ABI version\nand incrementing it will cause the guest driver to break.\n\nThis is the necessary changes to virtio_pci to support this.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "3343660d8c62c6b00b2f15324ef3fcb6be207bfa", "tree": "06d86446da65fc1814edad944e43aeb62f092422", "parents": [ "d50ed907dc3db5bf2dd0a05b4e199a65793a3788" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Mon Nov 12 21:30:26 2007 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:11 2008 +1100" }, "message": "virtio: PCI device\n\nThis is a PCI device that implements a transport for virtio. It allows virtio\ndevices to be used by QEMU based VMMs like KVM or Xen.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "c6fd47011b4bdebad3f1513bac75fe4895e332ee", "tree": "12224cdb2594be26e02291b142ba6e1e2c1c8122", "parents": [ "15f9c8903cbdb02aee0f1bcf86a97c2e238b9a3d" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:05 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:06 2008 +1100" }, "message": "virtio: Allow virtio to be modular and used by modules\n\nThis is needed for the virtio PCI device to be compiled as a module.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "15f9c8903cbdb02aee0f1bcf86a97c2e238b9a3d", "tree": "a215ccf3f745e52ae67aed1ac822baa57326b9b6", "parents": [ "0ad07ec1fd2f3e90dfb086b04c07a83db40e9172" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:05 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:05 2008 +1100" }, "message": "virtio: Use the sg_phys convenience function.\n\nSimple cleanup.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "81a8deab1ce3816c6a89e3429e234e7d3686da94", "tree": "d62e7dcc09dddd0edc6d507e7b219c16e66d3b02", "parents": [ "6e5aa7efb27aec7e55b6463fa2c8db594c4226fa" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:04 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:04 2008 +1100" }, "message": "virtio: handle interrupts after callbacks turned off\n\nAnthony Liguori found double interrupt suppression in the virtio_net\ndriver, triggered by two skb_recv_done\u0027s in a row. This is because\nvirtio_ring\u0027s interrupt suppression is a best-effort optimization: it\ncontains no synchronization so the host can miss it and still send\ninterrupts.\n\nBut it\u0027s certainly nicer for virtio users if calling disable_cb\nactually disables callbacks, so we check for the race in the interrupt\nroutine.\n\nNote: SMP guests might require syncronization here, but since\ndisable_cb is actually called from interrupt context, there has to be\nsome form of synchronization before the next same interrupt handler is\ncalled (Linux guarantees that the same device\u0027s irq handler will never\nrun simultanously on multiple CPUs).\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "6e5aa7efb27aec7e55b6463fa2c8db594c4226fa", "tree": "060a955e711ac224136157a5410e88dcdab965af", "parents": [ "b3369c1fb410fddeb38a404316c861395f6d6ae8" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:03 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:03 2008 +1100" }, "message": "virtio: reset function\n\nA reset function solves three problems:\n\n1) It allows us to renegotiate features, eg. if we want to upgrade a\n guest driver without rebooting the guest.\n\n2) It gives us a clean way of shutting down virtqueues: after a reset,\n we know that the buffers won\u0027t be used by the host, and\n\n3) It helps the guest recover from messed-up drivers.\n\nSo we remove the -\u003eshutdown hook, and the only way we now remove\nfeature bits is via reset.\n\nWe leave it to the driver to do the reset before it deletes queues:\nthe balloon driver, for example, needs to chat to the host in its\nremove function.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "426e3e0af5d2473e67d4256fc1340b7faebd1cc7", "tree": "29912e63889e45307f41ef08116ecd895b17e21c", "parents": [ "3309daaad724dd08eb598bf9c12b7bb9daddd706" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:49:59 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:50:00 2008 +1100" }, "message": "virtio: clarify NO_NOTIFY flag usage\n\nThe other side (host) can set the NO_NOTIFY flag as an optimization,\nto say \"no need to kick me when you add things\". Make it clear that\nthis is advisory only; especially that we should always notify when\nthe ring is full.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "18445c4d501b9ab4336f66ef46b092661ddaf336", "tree": "9d23185f207d912942890cf047d1d3200806b401", "parents": [ "a586d4f6016f7139d8c26df0e6927131168d3b5b" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:49:57 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:49:58 2008 +1100" }, "message": "virtio: explicit enable_cb/disable_cb rather than callback return.\n\nIt seems that virtio_net wants to disable callbacks (interrupts) before\ncalling netif_rx_schedule(), so we can\u0027t use the return value to do so.\n\nRename \"restart\" to \"cb_enable\" and introduce \"cb_disable\" hook: callback\nnow returns void, rather than a boolean.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "a586d4f6016f7139d8c26df0e6927131168d3b5b", "tree": "1c47e1a6b6b8fb18baa42f32980f29c4ae9cbbdc", "parents": [ "f35d9d8aae08940b7fdd1bb8110619da2ece6b28" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:49:56 2008 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Feb 04 23:49:57 2008 +1100" }, "message": "virtio: simplify config mechanism.\n\nPreviously we used a type/len pair within the config space, but this\nseems overkill. We now simply define a structure which represents the\nlayout in the config space: the config space can now only be extended\nat the end.\n\nThe main driver-visible changes:\n1) We indicate what fields are present with an explicit feature bit.\n2) Virtqueues are explicitly numbered, and not in the config space.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "74b2553f1d13e60fb27063204bd5b6908a6f8494", "tree": "cd35e82d16cf190ccd95362478a598314de639ce", "parents": [ "d1c856e0f1a4c946c6329cff126548ef4288735f" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Nov 19 11:20:42 2007 -0500" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Nov 19 11:20:42 2007 +1100" }, "message": "virtio: fix module/device unloading\n\nThe virtio code never hooked through the -\u003eremove callback. Although\nnoone supports device removal at the moment, this code is already\nneeded for module unloading.\n\nThis of course also revealed bugs in virtio_blk, virtio_net and lguest\nunloading paths.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "42b36cc0ce717deeb10030141a43dede763a3ebe", "tree": "b2dc48b4f16c5dc59461ad24b027d631edda1da4", "parents": [ "1200e646ae238afc536be70257290eb33fb6e364" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Nov 12 13:39:18 2007 +1100" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Nov 12 13:59:40 2007 +1100" }, "message": "virtio: Force use of power-of-two for descriptor ring sizes\n\nThe virtio descriptor rings of size N-1 were nicely set up to be\naligned to an N-byte boundary. But as Anthony Liguori points out, the\nfree-running indices used by virtio require that the sizes be a power\nof 2, otherwise we get problems on wrap (demonstrated with lguest).\n\nSo we replace the clever \"2^n-1\" scheme with a simple \"align to page\nboundary\" scheme: this means that all virtio rings take at least two\npages, but it\u0027s safer than guessing cache alignment.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "1bc4953ed44454c7f53d0b609445d1534981ee75", "tree": "05f67206d6978261ad2569d3a0030f0299331191", "parents": [ "4d125de3a5d130054df2285e542c1491d214d3e8" ], "author": { "name": "Anthony Liguori", "email": "aliguori@us.ibm.com", "time": "Wed Nov 07 15:49:24 2007 -0600" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Nov 12 13:59:09 2007 +1100" }, "message": "virtio: Fix used_idx wrap-around\n\nThe more_used() function compares the vq-\u003evring.used-\u003eidx with last_used_idx.\nSince vq-\u003evring.used-\u003eidx is a 16-bit integer, and last_used_idx is an\nunsigned int, this results in unpredictable behavior when vq-\u003evring.used-\u003eidx\nwraps around.\n\nThis patch corrects this by changing last_used_idx to the correct type.\n\nSigned-off-by: Anthony Liguori \u003caliguori@us.ibm.com\u003e\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" }, { "commit": "0a8a69dd77ddbd4513b21363021ecde7e1025502", "tree": "ed6d8f0756835390b4c0d9a172422f2e42a65523", "parents": [ "b01d9f2863349b0e041b90c3c86a998ee0fed2b0" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Oct 22 11:03:40 2007 +1000" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Oct 23 15:49:55 2007 +1000" }, "message": "Virtio helper routines for a descriptor ringbuffer implementation\n\nThese helper routines supply most of the virtqueue_ops for hypervisors\nwhich want to use a ring for virtio. Unlike the previous lguest\nimplementation:\n\n1) The rings are variable sized (2^n-1 elements).\n2) They have an unfortunate limit of 65535 bytes per sg element.\n3) The page numbers are always 64 bit (PAE anyone?)\n4) They no longer place used[] on a separate page, just a separate\n cacheline.\n5) We do a modulo on a variable. We could be tricky if we cared.\n6) Interrupts and notifies are suppressed using flags within the rings.\n\nUsers need only get the ring pages and provide a notify hook (KVM\nwants the guest to allocate the rings, lguest does it sanely).\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\nCc: Dor Laor \u003cdor.laor@qumranet.com\u003e\n" }, { "commit": "b01d9f2863349b0e041b90c3c86a998ee0fed2b0", "tree": "8789a06631a0ee502c58388d12c230eb4a2059b2", "parents": [ "31610434bc3523c0b01a10917a1185096a03c4c8" ], "author": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Mon Oct 22 11:03:39 2007 +1000" }, "committer": { "name": "Rusty Russell", "email": "rusty@rustcorp.com.au", "time": "Tue Oct 23 15:49:55 2007 +1000" }, "message": "Module autoprobing support for virtio drivers.\n\nThis adds the logic to convert the virtio ids into module aliases, and\nincludes a modalias entry in sysfs and the env var to make probing work.\n\nSigned-off-by: Rusty Russell \u003crusty@rustcorp.com.au\u003e\n" } ], "next": "ec3d41c4db4c21164332826ea8d812f94f2f6886" }