public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Hyok S. Choi" <hyok.choi@samsung.com>
To: "'Russell King - ARM Linux'" <linux@arm.linux.org.uk>,
	Greg Ungerer <gerg@snapgear.com>,
	Christoph Hellwig <hch@infradead.org>
Cc: Linux-Kernel List <linux-kernel@vger.kernel.org>,
	uClinux development list <uclinux-dev@uclinux.org>,
	linux-arm-kernel@lists.arm.linux.org.uk
Subject: RE: [ANNOUNCE] 2.6.4-hsc1 patch for MMU-less ARM is available.
Date: Mon, 29 Mar 2004 12:54:24 +0900	[thread overview]
Message-ID: <005201c41541$86d46320$7dc2dba8@dmsst.net> (raw)
In-Reply-To: <20040328200026.A10359@flint.arm.linux.org.uk>

Hello,

RMK> As Christoph said - why do you have a complete copy of the ARM specific
support,
RMK>   including lots of stuff which will probably never be used on MMU-less
CPUs?

I DO agree with you ;-) The stuffs are to be clean-up(removed) in next
coming-ups, we planed, already.

Christoph> Is the code really that different that you need a armnommu arch
instead of merging it into arch/arm?
RMK> I don't understand this "complete copy and modify" thinking that
uclinux people seem to have - it's
RMK> completely against the Linux development methodology.

I think it is just like the arm26 and the m68knommu case. maintenance issue.

a. readability and portability. If you've ever seen armnommu arch files in
2.4 version, it was just like what you've told, (patching
on arch/arm, although it was renamed to arch/armnommu). It was very
obfuscated.
  In fact, arch/arm supports lots of platforms (e.g. sa, xscale, and so on),
and they has many machine specific craps on common codes. In some cases, I
experienced some modification made some side-effects on other platforms.
  Thus, to increase readability and portability, separating armnommu from
arm directory could help this. I know that this
is not a best solution, but we have no other options, I think. In addition,
it was the way what uc people have done, so that they can work more easily.
think about it, to merge all arm (with mmu plus without mmu) codes in
arch/arm can be a right choice, but obviously it makes the codes hard to
read and hard to port. 

b. armnommu has dependencies with nommu support codes which is maintained by
uc people. we should test with their working codes. Much of the arch codes
are for mm. As you said, I think it should be optimized(removed or
re-written) for armnommu. e.g., mmu-less arm7tdmi, which is one of the main
target for armnommu, many of files like arm/kernel/head.S is not work at
all, and the files in arm/mm are not needed.

However, this port is based on RMK codes, if you say this patch must be
merged into rmk patch, I should agree. ;) but I think to be merged into one
directory is not good idea.

And for uclinux people, as linuxdevices.com survey says, there exist many
developers, and they made much efforts, they are linux people also, I think.
Ignoring them can not be a good idea. 

Best regards,
Hyok S. Choi

<EOT>
CHOI, HYOK-SUNG
Engineer (Linux System Software)
S/W Platform Lab, Digital Media R&D Center
Samsung Electronics Co.,Ltd.
tel: +82-31-200-8594  fax: +82-31-200-3427
e-mail: hyok.choi@samsung.com

[compile&run]
main(a){printf(a,34,a="main(a){printf(a,34,a=%c%s%c,34);}",34);}
 
 




       reply	other threads:[~2004-03-29  3:55 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20040328200026.A10359@flint.arm.linux.org.uk>
2004-03-29  3:54 ` Hyok S. Choi [this message]
2004-03-27  9:29 [ANNOUNCE] 2.6.4-hsc1 patch for MMU-less ARM is available Hyok S. Choi
2004-03-27 12:14 ` Christoph Hellwig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='005201c41541$86d46320$7dc2dba8@dmsst.net' \
    --to=hyok.choi@samsung.com \
    --cc=gerg@snapgear.com \
    --cc=hch@infradead.org \
    --cc=linux-arm-kernel@lists.arm.linux.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=uclinux-dev@uclinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox