Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2013-01-19 Thread Pavel Machek
Hi!

> >>> Ok, plus there's already computing standard called "arc".
> >>>
> >>> https://www.linux-mips.org/wiki/ARC
> >>>
> >>> What about calling the architecture "sarc" or "syarc" ?
> >> I think it would be best use of everybody's time if you could focus on
> >> the technical content of the port rather than the name. It has been like
> >> that for +10 years and I don't intend to make the kernel port name
> >> different from THE arch name itself.
> > Yeah, and it is best use of reviewers time to confuse them with
> > one-letter difference to very popular architecture... and use three
> > letter acronym that was already taken.
> >
> > In this state, I hope your port never gets merged. Yes, I got used to
> > do cd a/ar.
> 
> Although I really don't want to spend time arguing with you but still .. 
> if
> you had a neighbor with same name as your friend your strategy most likely 
> would
> be to force one of them to change names - kind of seems funny and
> stupid.

Yeah, imagine creating "/bin/mr" that would be du equivalent. How many
people would delete their files by mistake? 

> Having said that, despite all the ranting so far, you've not managed to 
> comment on
> a single line of the actual code, which is more funny and stupid. So

Yeah, you tricked me into reviewing your code. No, I will not do more
reviews for you. Yes, I bet I'm not the only one, and I bet that
mistake will happen about once per developer.

Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2013-01-19 Thread Vineet Gupta
On Saturday 19 January 2013 01:16 AM, Pavel Machek wrote:
> On Tue 2012-11-20 19:47:58, Vineet Gupta wrote:
>> On Tuesday 20 November 2012 07:29 PM, Pavel Machek wrote:
>>> On Tue 2012-11-20 14:47:37, Pavel Machek wrote:
 Hi!

> 
> https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
>
> All comments/criticisms are welcome !
 Huh. arch/arc is very similar to arch/arm. So similar that it took me
 10 minutes to figure out. I guess theres no alternate name arc could
 use?
>>> Hmm... "Advanced Risc"... :-(.
>>>
>>> Ok, plus there's already computing standard called "arc".
>>>
>>> https://www.linux-mips.org/wiki/ARC
>>>
>>> What about calling the architecture "sarc" or "syarc" ?
>> I think it would be best use of everybody's time if you could focus on
>> the technical content of the port rather than the name. It has been like
>> that for +10 years and I don't intend to make the kernel port name
>> different from THE arch name itself.
> Yeah, and it is best use of reviewers time to confuse them with
> one-letter difference to very popular architecture... and use three
> letter acronym that was already taken.
>
> In this state, I hope your port never gets merged. Yes, I got used to
> do cd a/ar.
>   Pavel

Although I really don't want to spend time arguing with you but still .. if
you had a neighbor with same name as your friend your strategy most likely would
be to force one of them to change names - kind of seems funny and stupid.

Having said that, despite all the ranting so far, you've not managed to comment 
on
a single line of the actual code, which is more funny and stupid. So please stop
blathering and do some constructive work. I won't go around asking ARC name to 
be
changed because someone's Tab completion doesn't work anymore.

And I'm sorry dude, that alone won't prevent the port from being merged.

-Vineet

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2013-01-19 Thread Pavel Machek
Hi!

> > Yeah, and it is best use of reviewers time to confuse them with
> > one-letter difference to very popular architecture... and use three
> > letter acronym that was already taken.
> > 
> > In this state, I hope your port never gets merged. Yes, I got used to
> > do cd a/ar.
> 
> You already lost that capability with the addition of arm64. As you
> may remember, that one was proposed to be called arch/aarch64 at first,
> which wasn't very popular ;-)

Actually, with arm26, which is older IIRC. But tab still works there,
just will not fill the last /, and you can't mistake arm64 with arm in
filenames etc -- different length.

> A lot of people are familiar with the ARC name, it's been around
> for decades and has sold billions of CPUs under that name, likely
> more than x86 or mips (still dwarfed by ARM of course). They've just

Well, it seems that mips people were aware of the ARC name, it just
meant something different:

https://www.linux-mips.org/wiki/ARC

Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2013-01-18 Thread Arnd Bergmann
On Friday 18 January 2013, Pavel Machek wrote:
> Yeah, and it is best use of reviewers time to confuse them with
> one-letter difference to very popular architecture... and use three
> letter acronym that was already taken.
> 
> In this state, I hope your port never gets merged. Yes, I got used to
> do cd a/ar.

You already lost that capability with the addition of arm64. As you
may remember, that one was proposed to be called arch/aarch64 at first,
which wasn't very popular ;-)

A lot of people are familiar with the ARC name, it's been around
for decades and has sold billions of CPUs under that name, likely
more than x86 or mips (still dwarfed by ARM of course). They've just
kept a low profile so far and never tried to upstream their Linux
port (unlike their gcc port, which was merged in 1997).

I'm sure there are a lot of bad things one can say about Synopsys,
but they are doing the right thing here, and calling their CPU
architecture by a different name is not going to be helpful
to their users.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2013-01-18 Thread Pavel Machek
On Tue 2012-11-20 19:47:58, Vineet Gupta wrote:
> On Tuesday 20 November 2012 07:29 PM, Pavel Machek wrote:
> > On Tue 2012-11-20 14:47:37, Pavel Machek wrote:
> >> Hi!
> >>
> >>> 
> >>> https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
> >>>
> >>> All comments/criticisms are welcome !
> >> Huh. arch/arc is very similar to arch/arm. So similar that it took me
> >> 10 minutes to figure out. I guess theres no alternate name arc could
> >> use?
> > Hmm... "Advanced Risc"... :-(.
> >
> > Ok, plus there's already computing standard called "arc".
> >
> > https://www.linux-mips.org/wiki/ARC
> >
> > What about calling the architecture "sarc" or "syarc" ?
> 
> I think it would be best use of everybody's time if you could focus on
> the technical content of the port rather than the name. It has been like
> that for +10 years and I don't intend to make the kernel port name
> different from THE arch name itself.

Yeah, and it is best use of reviewers time to confuse them with
one-letter difference to very popular architecture... and use three
letter acronym that was already taken.

In this state, I hope your port never gets merged. Yes, I got used to
do cd a/ar.
Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-20 Thread Vineet Gupta
On Tuesday 20 November 2012 07:29 PM, Pavel Machek wrote:
> On Tue 2012-11-20 14:47:37, Pavel Machek wrote:
>> Hi!
>>
>>> https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
>>>
>>> All comments/criticisms are welcome !
>> Huh. arch/arc is very similar to arch/arm. So similar that it took me
>> 10 minutes to figure out. I guess theres no alternate name arc could
>> use?
> Hmm... "Advanced Risc"... :-(.
>
> Ok, plus there's already computing standard called "arc".
>
> https://www.linux-mips.org/wiki/ARC
>
> What about calling the architecture "sarc" or "syarc" ?

I think it would be best use of everybody's time if you could focus on
the technical content of the port rather than the name. It has been like
that for +10 years and I don't intend to make the kernel port name
different from THE arch name itself.




>
>   Pavel

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-20 Thread Pavel Machek
On Tue 2012-11-20 14:47:37, Pavel Machek wrote:
> Hi!
> 
> > https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
> > 
> > All comments/criticisms are welcome !
> 
> Huh. arch/arc is very similar to arch/arm. So similar that it took me
> 10 minutes to figure out. I guess theres no alternate name arc could
> use?

Hmm... "Advanced Risc"... :-(.

Ok, plus there's already computing standard called "arc".

https://www.linux-mips.org/wiki/ARC

What about calling the architecture "sarc" or "syarc" ?

Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-20 Thread Vineet Gupta
On Tuesday 20 November 2012 07:17 PM, Pavel Machek wrote:
> Hi!
>
>> https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
>>
>> All comments/criticisms are welcome !
> Huh. arch/arc is very similar to arch/arm. So similar that it took me
> 10 minutes to figure out. I guess theres no alternate name arc could
> use?

Thanks for the review comment - I'll pass this message to our marketing
folks !

>
>   Pavel

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-20 Thread Pavel Machek
Hi!

> https://raw.github.com/vineetgarc/publish/master/ELCE-2012-ARC-Linux.pdf
> 
> All comments/criticisms are welcome !

Huh. arch/arc is very similar to arch/arm. So similar that it took me
10 minutes to figure out. I guess theres no alternate name arc could
use?

Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-08 Thread Vineet Gupta
Hi Arnd,

Thanks for your valuable comments I will rework the port.

P.S. Sorry for top-posting. 

Thanks,
Vineet

On Wednesday 07 November 2012, Vineet Gupta wrote:
> This patchset based off-of 3.7-rc3, introduces the Linux kernel port to
> ARC700 processor family (750D and 770D) from Synopsys.
>
> ARC700 is highly configurable and power efficient 32-bit RISC core with MMU.
> It is embedded in SoCs deployed in TV Set Top boxes, Digital Media Players,
> all the way to Network-on-Chips.

Hi Vineet,

I'm largely quite happy with how the series has turned out, having looked
at earlier versions of the ARC kernel a couple of years ago. I've commented
already on individual patches, but overall, I see two main issues that
need to be resolved in order to merge the port:

* You have to use the generic syscall interface, and that means not using
  any of the legacy system calls that have since been replaced by newer
  versions of the same.

* You are missing dynamic hardware detection. Rather than building a kernel
  with everything known about the hardware at compile time, new ports these
  days are normally able to run on all kinds of hardware and detect the
  differences by looking at configuration registers (e.g. PCI), asking
  firmware (Open Firmware, ACPI, ...) or by interpreting a device tree
  that is passed by the boot loader (most embedded systems).
  I assume that device tree is the right solution for you because a lot of
  the hardware you use is likely shared with ARM, PowerPC or MIPS based
  SoCs and they are all using (or getting migrated to) DT now instead of
  static platform devices,

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-07 Thread Gilad Ben-Yossef
On Wed, Nov 7, 2012 at 11:47 AM, Vineet Gupta
 wrote:
> Hi,
>
> This patchset based off-of 3.7-rc3, introduces the Linux kernel port to
> ARC700 processor family (750D and 770D) from Synopsys.


We've been using the port for 6 month now at Ezchip running on top of
FPGA in 4 way SMP configuration (support for which will no doubt
follow later) and have subjected it to various stress tests (hackbench
and such) and it is working up quite well over all.

I realize some code style or organization changes might be in order
but at the same time wanted to attest the nature of its good and
behavior performance on real hardware; so for what it's worth:

Tested-by: Gilad Ben-Yossef 

Cheers,
Gilad

-- 
Gilad Ben-Yossef
Chief Coffee Drinker
gi...@benyossef.com
Israel Cell: +972-52-8260388
US Cell: +1-973-8260388
http://benyossef.com

"If you take a class in large-scale robotics, can you end up in a
situation where the homework eats your dog?"
 -- Jean-Baptiste Queru
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [RFC Patch v1 00/31] Synopsys ARC Linux kernel Port

2012-11-07 Thread Arnd Bergmann
On Wednesday 07 November 2012, Vineet Gupta wrote:
> This patchset based off-of 3.7-rc3, introduces the Linux kernel port to
> ARC700 processor family (750D and 770D) from Synopsys.
> 
> ARC700 is highly configurable and power efficient 32-bit RISC core with MMU.
> It is embedded in SoCs deployed in TV Set Top boxes, Digital Media Players,
> all the way to Network-on-Chips.

Hi Vineet,

I'm largely quite happy with how the series has turned out, having looked
at earlier versions of the ARC kernel a couple of years ago. I've commented
already on individual patches, but overall, I see two main issues that
need to be resolved in order to merge the port:

* You have to use the generic syscall interface, and that means not using
  any of the legacy system calls that have since been replaced by newer
  versions of the same.

* You are missing dynamic hardware detection. Rather than building a kernel
  with everything known about the hardware at compile time, new ports these
  days are normally able to run on all kinds of hardware and detect the
  differences by looking at configuration registers (e.g. PCI), asking
  firmware (Open Firmware, ACPI, ...) or by interpreting a device tree
  that is passed by the boot loader (most embedded systems).
  I assume that device tree is the right solution for you because a lot of
  the hardware you use is likely shared with ARM, PowerPC or MIPS based
  SoCs and they are all using (or getting migrated to) DT now instead of
  static platform devices,

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/