From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:48664) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1STCoq-0003yq-Ha for qemu-devel@nongnu.org; Sat, 12 May 2012 09:57:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1STCoo-0002BW-Jf for qemu-devel@nongnu.org; Sat, 12 May 2012 09:57:12 -0400 Received: from cantor2.suse.de ([195.135.220.15]:37996 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1STCoo-0002BS-A8 for qemu-devel@nongnu.org; Sat, 12 May 2012 09:57:10 -0400 Message-ID: <4FAE6C2E.4090700@suse.de> Date: Sat, 12 May 2012 15:57:02 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <4FAE4FE7.1040905@suse.de> <4FAE59BF.4020908@suse.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 4/4, master+QEMU 1.1] sun4u: implement interrupt clearing registers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Blue Swirl Cc: Anthony Liguori , qemu-devel@nongnu.org, Artyom Tarasenko Am 12.05.2012 15:08, schrieb Blue Swirl: > On Sat, May 12, 2012 at 12:38 PM, Andreas F=C3=A4rber wrote: >> Am 12.05.2012 14:32, schrieb Blue Swirl: >>> On Sat, May 12, 2012 at 11:56 AM, Andreas F=C3=A4rber wrote: >>>> Am 12.05.2012 11:15, schrieb Artyom Tarasenko: >>>>> Implement registers for clearing OBIO and PCI interrupts >>>>> >>>>> Signed-off-by: Artyom Tarasenko >>>> >>>> Implementing new registers is a feature, not a 1.1 bugfix... >>>> Many of us would like to get patches committed and have to wait. >>> >>> Those 12 trivial lines in Sparc64 specific code won't destabilize >>> anything. Two wrongs do not make one right, but several non-bug fix >>> patches have been committed since start of hard freeze. Including >>> yours. >> >> Which do you mean? You have not applied nor reviewed my pending sparc >> patches (and I don't expect you to apply them). I've only supplied >> bugfix and cleanup patches for 1.1 that I'm aware of. >=20 > For example > f5df5baf11a32ae6a669ac945625d1c3e4deb76d cpu: Update documentation and = comment That one was truely a bug fix. :) > aabfd88d5e1ec0878aa70076c3de1859614671f4 configure: Reindent VirtFS che= ck This was a prerequisite for a bug fix that required adding code with proper indentation. > Both are nice, trivial patches, most certainly they don't implement > any registers and the release will be better with those included, but > they are not bug fixes. Right, they are cleanups, see above. They do not add random new code. They might remove dead code. But if the new criteria is "trivial lines that don't destabilize anything" then I have a 74-patch series for you to commit on the list (which I'd rather not, respecting the Freeze). Artyom apparently didn't understand that 1.1 equals master, judging from $subject, so he is likely unaware of our release process. Just saying, the same standards should apply to everyone, and that would include qemu-ga as well. I do welcome Artyom's sparc64 progress btw. :-) Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg