From mboxrd@z Thu Jan 1 00:00:00 1970 From: Corneliu ZUZU Subject: Re: [PATCH 1/7] arm: move arch/arm/hvm.c to arch/arm/hvm/hvm.c Date: Tue, 9 Feb 2016 21:19:29 +0200 Message-ID: <56BA3BC1.70007@bitdefender.com> References: <1454950682-9459-1-git-send-email-czuzu@bitdefender.com> <1454950682-9459-2-git-send-email-czuzu@bitdefender.com> <56B9CD43.5060009@bitdefender.com> <56B9E1A602000078000CFFF5@prv-mh.provo.novell.com> <56B9DC4E.5090304@bitdefender.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============8323624949958370411==" Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Tamas K Lengyel Cc: Kevin Tian , Keir Fraser , Ian Campbell , Razvan Cojocaru , Stefano Stabellini , Jun Nakajima , Andrew Cooper , Xen-devel , Stefano Stabellini , Jan Beulich List-Id: xen-devel@lists.xenproject.org This is a multi-part message in MIME format. --===============8323624949958370411== Content-Type: multipart/alternative; boundary="------------090908070908090001020101" This is a multi-part message in MIME format. --------------090908070908090001020101 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit On 2/9/2016 7:40 PM, Tamas K Lengyel wrote: > > > On Tue, Feb 9, 2016 at 5:32 AM, Corneliu ZUZU > wrote: > > There are already directories w/ just one/a few files in them, > even small (e.g. common/gcov/gcov.c). > IMHO no harm is done if a file is put in its proper directory even > before it grows too large. > This way one can find the file more easily, future additions are > more visibly encouraged to > also be separated in that directory when it makes sense, symmetry > between arch directories remains > intact (=> easier to compare between code for different > arches/find equivalent files between them). > > But I am not a Xen maintainer, I'm only a contributor so I can > only suggest :). > If ARM maintainers (e.g. Tamas) feel the same, I will move the > file back. > > > I don't really have a strong opinion about this either way, so if the > others prefer it staying as it is then there is no point moving it. > > Tamas > Ok, will do, thanks for the feedback. Corneliu. --------------090908070908090001020101 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 7bit
On 2/9/2016 7:40 PM, Tamas K Lengyel wrote:


On Tue, Feb 9, 2016 at 5:32 AM, Corneliu ZUZU <czuzu@bitdefender.com> wrote:
There are already directories w/ just one/a few files in them, even small (e.g. common/gcov/gcov.c).
IMHO no harm is done if a file is put in its proper directory even before it grows too large.
This way one can find the file more easily, future additions are more visibly encouraged to
also be separated in that directory when it makes sense, symmetry between arch directories remains
intact (=> easier to compare between code for different arches/find equivalent files between them).

But I am not a Xen maintainer, I'm only a contributor so I can only suggest :).
If ARM maintainers (e.g. Tamas) feel the same, I will move the file back.

I don't really have a strong opinion about this either way, so if the others prefer it staying as it is then there is no point moving it.

Tamas


Ok, will do, thanks for the feedback.

Corneliu.
--------------090908070908090001020101-- --===============8323624949958370411== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============8323624949958370411==--