From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753281AbcEINP4 (ORCPT ); Mon, 9 May 2016 09:15:56 -0400 Received: from zimbra13.linbit.com ([212.69.166.240]:33420 "EHLO zimbra13.linbit.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751384AbcEINPy (ORCPT ); Mon, 9 May 2016 09:15:54 -0400 Date: Mon, 9 May 2016 15:15:48 +0200 From: Lars Ellenberg To: Nicolas Dichtel Cc: davem@davemloft.net, netdev@vger.kernel.org, philipp.reisner@linbit.com, drbd-dev@lists.linbit.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH net-next v3] block/drbd: align properly u64 in nl messages Message-ID: <20160509131547.GX16459@soda.linbit> Mail-Followup-To: Nicolas Dichtel , davem@davemloft.net, netdev@vger.kernel.org, philipp.reisner@linbit.com, drbd-dev@lists.linbit.com, linux-kernel@vger.kernel.org References: <20160503100644.GE16459@soda.linbit> <1462786820-15519-1-git-send-email-nicolas.dichtel@6wind.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1462786820-15519-1-git-send-email-nicolas.dichtel@6wind.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, May 09, 2016 at 11:40:20AM +0200, Nicolas Dichtel wrote: > The attribute 0 is never used in drbd, so let's use it as pad attribute > in netlink messages. This minimizes the patch. > > Note that this patch is only compile-tested. > > Signed-off-by: Nicolas Dichtel > Signed-off-by: Lars Ellenberg > --- > > v2 -> v3: > use 0 as padattr instead of adding new attributes Thanks. > v1 -> v2: > rework the patch to handle all cases > > Maybe prefixing genl_magic_func.h and genl_magic_struct.h by 'drbd_' > could be interesting so that new module won't use it. What is your > opinion? This was supposed to not be DRBD specific. But it might even still need some massaging before it was truly generic. And obviously, it does not meet the taste of genetlink folks, to say the least :( I don't care either way. Lars Ellenberg