From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=56183 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q85ui-0007GA-Mw for qemu-devel@nongnu.org; Fri, 08 Apr 2011 03:15:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q85uh-0003a8-P3 for qemu-devel@nongnu.org; Fri, 08 Apr 2011 03:15:28 -0400 Received: from mx1.redhat.com ([209.132.183.28]:19504) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Q85uh-0003Zt-GW for qemu-devel@nongnu.org; Fri, 08 Apr 2011 03:15:27 -0400 From: Amit Shah Date: Fri, 8 Apr 2011 12:45:14 +0530 Message-Id: Subject: [Qemu-devel] [PATCH 0/5] atapi: Implement 'media' subcommand for GESN List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu list Cc: Kevin Wolf , Juan Quintela , Stefan Hajnoczi , Markus Armbruster , Amit Shah , Paolo Bonzini The GET_EVENT_STATUS_NOTIFICATION ATAPI command is listed as a mandatory command in the spec but we don't really implement it any of its sub-commands. The commit message for the last commit explains why implementing just the media subcommand is helpful and how it goes a long way in getting guests to behave as expected. The difference from the RFC series sent earlier is: - Split into more patches - Add tray open/close notification (from Markus) There certainly is much more work to be done for the other commands and also for state change handling (tray open / close / new media) overall for the block layer, but this is a good first step in being spec-compliant and at the same time making guests work. Please apply! Amit Shah (5): atapi: Allow GET_EVENT_STATUS_NOTIFICATION after media change ide: Move GET_EVENT_STATUS_NOTIFICATION command handling to its own function atapi: GESN: Spin off No Event Available handling into own function atapi: GESN: Add enums for commonly-used field types atapi: Implement 'media' subcommand of GET_EVENT_STATUS_NOTIFICATION command hw/ide/core.c | 172 +++++++++++++++++++++++++++++++++++++++++++++++----- hw/ide/internal.h | 6 ++ 2 files changed, 161 insertions(+), 17 deletions(-) -- 1.7.4.2