From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [PATCH 18/41] regset: new method and helpers for it Date: Tue, 30 Jun 2020 14:25:08 +0100 Message-ID: <20200630132508.GF2786714@ZenIV.linux.org.uk> References: <20200629182349.GA2786714@ZenIV.linux.org.uk> <20200629182628.529995-1-viro@ZenIV.linux.org.uk> <20200629182628.529995-18-viro@ZenIV.linux.org.uk> <20200629203028.GB2786714@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43576 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730381AbgF3NZO (ORCPT ); Tue, 30 Jun 2020 09:25:14 -0400 Content-Disposition: inline In-Reply-To: <20200629203028.GB2786714@ZenIV.linux.org.uk> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Linus Torvalds Cc: linux-arch , Linux Kernel Mailing List , David Miller , Tony Luck , Will Deacon On Mon, Jun 29, 2020 at 09:30:28PM +0100, Al Viro wrote: > > (b) or just call it something better to begin with. Maybe just > > "get_regset" instead? > > > > Frankly, other field names there are also nasty - 'set' is not > fun to grep for, but there are 'n' and 'size' as well. There's > also 'bias' and 'align' (both completely unused)... How about ->regset_get()? I've force-pushed a variant with that change + removal of leftover unused variables (caught by Intel build bot) + braino fix in sparc conversion (spotted when looking through the patchset again); same branch names. Not sure if it's worth a repost of the entire patchbomb...