From mboxrd@z Thu Jan 1 00:00:00 1970 X-Msuck: nntp://news.gmane.org/gmane.linux.lib.musl.general/8637 Path: news.gmane.org!not-for-mail From: Rich Felker Newsgroups: gmane.linux.lib.musl.general Subject: Re: [PATCH 1/5] Pull a couple common AWK functions for CFI scripts into separate file Date: Thu, 8 Oct 2015 17:37:21 -0400 Message-ID: <20151008213721.GA8645@brightrain.aerifal.cx> References: <1443785556-27473-1-git-send-email-alexinbeijing@gmail.com> Reply-To: musl@lists.openwall.com NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: ger.gmane.org 1444340259 11422 80.91.229.3 (8 Oct 2015 21:37:39 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Thu, 8 Oct 2015 21:37:39 +0000 (UTC) To: musl@lists.openwall.com Original-X-From: musl-return-8649-gllmg-musl=m.gmane.org@lists.openwall.com Thu Oct 08 23:37:38 2015 Return-path: Envelope-to: gllmg-musl@m.gmane.org Original-Received: from mother.openwall.net ([195.42.179.200]) by plane.gmane.org with smtp (Exim 4.69) (envelope-from ) id 1ZkIsb-0002Pd-8A for gllmg-musl@m.gmane.org; Thu, 08 Oct 2015 23:37:37 +0200 Original-Received: (qmail 29869 invoked by uid 550); 8 Oct 2015 21:37:34 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: Original-Received: (qmail 29845 invoked from network); 8 Oct 2015 21:37:33 -0000 Content-Disposition: inline In-Reply-To: <1443785556-27473-1-git-send-email-alexinbeijing@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) Original-Sender: Rich Felker Xref: news.gmane.org gmane.linux.lib.musl.general:8637 Archived-At: On Fri, Oct 02, 2015 at 01:32:32PM +0200, Alex Dowad wrote: > There is a lot which could be common between i386 and x86_64, but none of it > will be useful for any other arch. These should be useful for all archs, > however. > --- Thanks! I'm commenting on these as a set rather than breaking this down into separate replies. I think you still have errors in your instruction parsing which will lead to incorrect clobbers, just not as bad as before. For example, movl $0,(%esp,%esi) gets marked as clobbering %esi. Using .* in the patterns is probably a bad idea. I think the x86_64 patch is also affected, but I tested it and otherwise it basically works. In any case I'm going ahead and applying patches 1-3, with 2 and 3 merged into one commit. In general, try to merge changes that are conceptually related and where putting them together in one patch doesn't obscure what was changed; for example, merging 4 with 2-3 would obscure changes IMO since there would be pattern changes and operand order changes in the same lines. I've also edited commit messages to match capitalization used in shortlog and to make git log fit in 80 columns (generally messages should be wrapped at 70-72 chars, like email). Could you follow up with fixes for patches 4 and 5? I'd like to try to get these into the next release (soon). Rich