[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v6 5/5] x86/mem_sharing: style cleanup


  • To: Tamas K Lengyel <tamas@xxxxxxxxxxxxx>
  • From: Jan Beulich <JBeulich@xxxxxxxx>
  • Date: Thu, 18 Jul 2019 13:37:01 +0000
  • Accept-language: en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1;spf=pass smtp.mailfrom=suse.com;dmarc=pass action=none header.from=suse.com;dkim=pass header.d=suse.com;arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=D6OeLqUEMnKmAtqPybt9VT42ApnkeHXZycGSku74TZs=; b=F+7mS9YLecyzaW7tBL+nvlY4QpWMK4hEkRK8GI9EvyaFK0SPu2EAbiZHrSyNSBNfemQpHDH6FPwgQjk/Iuv3NsU7Bu14XOc1Inj6PZQKf7wIjXHsJNNalZIBTiH10snR6ODEOtslA//tD+zmXiW8nh9gDTC5C4DE9CLcFDpxtfEgN9T1XAEtzVzWR0ljAPjMNhX8L+fsNgZlXMCKHuzAbRiFbZnq9z8lcKtVDJ+N8JZX9W59U0lB0m2D+fFgpqjZLEJlgTsZR4pnLflciFNHjz3dA/NfrqAXnHoTZq7pR/qSTPnuiL+hVxcNH7Yvd8pjXFepXBQj8Y+Pfc3Aa/xXHg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ho21qak6J5QzPDMp3vDuwzw2D9IEmuvvakhz19cyYkzBPPRzNgV7x0OFGu7nDvILVVMkpPxAwo2cpWNZGqkk1Dgl3z8X+NN8C6dj/Vy1H+1PZNRck8HetHfD+UifcPK9A80l9gxs3mVFr26T2yjqIesckUgtPWhVrx5dtTcgbY5MBvam5Y3v8T4KBicTgEjQ7Ehvrdhj5qAJFpm0d6NDrpfr/e2rt6N8vRDnyOD5eV5e9BiNTH2r8lwI1YTP69kQHWVeGh6gjiJidLgiUfvMquGhJhXN1j9N8UdQmEmbzgUHx157R4tGtZ3baVu9sO2ZHauo6ySg6V5qoAL1e/rrhg==
  • Authentication-results: spf=none (sender IP is ) smtp.mailfrom=JBeulich@xxxxxxxx;
  • Cc: "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Thu, 18 Jul 2019 13:37:24 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHVPNaxkDvYm35RQ027yAvXjVGEzqbQNNsAgAAjJviAAAPJgIAAAPIjgAAFVAA=
  • Thread-topic: [Xen-devel] [PATCH v6 5/5] x86/mem_sharing: style cleanup

On 18.07.2019 15:16, Tamas K Lengyel wrote:
> On Thu, Jul 18, 2019 at 7:14 AM Jan Beulich <JBeulich@xxxxxxxx> wrote:
>>
>> On 18.07.2019 14:59, Tamas K Lengyel wrote:
>>> On Thu, Jul 18, 2019 at 4:56 AM Jan Beulich <JBeulich@xxxxxxxx> wrote:
>>>>
>>>> On 17.07.2019 21:33, Tamas K Lengyel wrote:
>>>>> @@ -136,8 +137,8 @@ static inline bool _page_lock(struct page_info *page)
>>>>>                 cpu_relax();
>>>>>             nx = x + (1 | PGT_locked);
>>>>>             if ( !(x & PGT_validated) ||
>>>>> -             !(x & PGT_count_mask) ||
>>>>> -             !(nx & PGT_count_mask) )
>>>>> +                !(x & PGT_count_mask) ||
>>>>> +                !(nx & PGT_count_mask) )
>>>>>                 return false;
>>>>>         } while ( cmpxchg(&page->u.inuse.type_info, x, nx) != x );
>>>>
>>>> Aren't you screwing up indentation here? It looks wrong both in my
>>>> mail client's view and on the list archives, whereas. Furthermore
>>>> this is code you've introduced earlier in the series, so it should
>>>> be got right there, not here.
>>>
>>> The style was auto-applied with astyle using the bsd format. In the
>>> previous patch there were no style-changes applied because it was a
>>> copy-paste job from the other code location. I rather keep
>>> code-copying and style fixes separate.
>>
>> But you're actively breaking Xen style here (and below).
> 
> I don't see any mention of style restrictions regarding this in
> CODING_STYLE. If there is, I would prefer changing that so we can
> automate style checks which IMHO are the biggest waste of everyone's
> time to do manually.

./CODING_STYLE fails to mention many aspects of what we do everywhere.
Almost any attempt of updating it has failed for me in the past, often
due to entire lack of responses on patches (in other cases also because
of people disagreeing). Despite you being the maintainer of the file I
strongly think you shouldn't actively break style that's in line with
large swathes of code elsewhere.

Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.