My first patch to the mailist, but being held

My first patch to the mailist, but being held

[Wan ZongShun]

Hmm, this patch have been in Russell's list a few days, now I want to
'CC' to the list,
but being held.

--------

Your mail to 'linux-arm-kernel' with the subject

   Re: [PATCH v2] ARM: Add common files of platform definition and
cpu     implementing

Is being held until the list moderator can review it for approval.

The reason it is being held:

   Message body is too big: 55015 bytes with a limit of 40 KB

Either the message will get posted to the list, or you will receive
notification of the moderator's decision.  If you would like to cancel
this posting, please visit the following URL:

-- 
Wan z.s

My first patch to the mailist, but being held

[Eric Miao]

On Tue, Sep 1, 2009 at 9:16 PM, Wan ZongShun<mcuos.com@gmail.com> wrote:
> Hmm, this patch have been in Russell's list a few days, now I want to
> 'CC' to the list,
> but being held.
>
> --------
>
> Your mail to 'linux-arm-kernel' with the subject
>
> ? Re: [PATCH v2] ARM: Add common files of platform definition and
> cpu ? ? implementing
>
> Is being held until the list moderator can review it for approval.
>
> The reason it is being held:
>
> ? Message body is too big: 55015 bytes with a limit of 40 KB
>

I think the restriction on attachment size can be loosed a bit, but that
really depends on the real hardware capacity of the server. Anyway,
a patch larger than 40KB doesn't sound like a good one at least.

> Either the message will get posted to the list, or you will receive
> notification of the moderator's decision. ?If you would like to cancel
> this posting, please visit the following URL:
>
> --
> Wan z.s
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>

My first patch to the mailist, but being held

[Wan ZongShun]

2009/9/1 Eric Miao <eric.y.miao@gmail.com>:
> On Tue, Sep 1, 2009 at 9:16 PM, Wan ZongShun<mcuos.com@gmail.com> wrote:
>> Hmm, this patch have been in Russell's list a few days, now I want to
>> 'CC' to the list,
>> but being held.
>>
>> --------
>>
>> Your mail to 'linux-arm-kernel' with the subject
>>
>> ? Re: [PATCH v2] ARM: Add common files of platform definition and
>> cpu ? ? implementing
>>
>> Is being held until the list moderator can review it for approval.
>>
>> The reason it is being held:
>>
>> ? Message body is too big: 55015 bytes with a limit of 40 KB
>>
>
> I think the restriction on attachment size can be loosed a bit, but that
> really depends on the real hardware capacity of the server. Anyway,
> a patch larger than 40KB doesn't sound like a good one at least.

Yes, I think so too, due to I never meet this type 'held' in LAKML and
LKML, so I am amazed at this 'held' in the list.

>> Either the message will get posted to the list, or you will receive
>> notification of the moderator's decision. ?If you would like to cancel
>> this posting, please visit the following URL:
>>
>> --
>> Wan z.s
>>
>> _______________________________________________
>> linux-arm-kernel mailing list
>> linux-arm-kernel at lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>>
>



-- 
Wan z.s

My first patch to the mailist, but being held

[David Woodhouse]

On Tue, 2009-09-01 at 22:05 +0800, Wan ZongShun wrote:
> 
> Yes, I think so too, due to I never meet this type 'held' in LAKML and
> LKML, so I am amazed at this 'held' in the list.

Not all of the mailman settings were imported from the old list to the
new one. If the moderators want to increase the size limit, they are
free to do that -- the hardware can certainly cope.

But Eric is right that patches above 40KiB are fairly suspect anyway --
why is it so large?

-- 
dwmw2

My first patch to the mailist, but being held

[Wan ZongShun]

Dear David,

2009/9/1 David Woodhouse <dwmw2@infradead.org>:
> On Tue, 2009-09-01 at 22:05 +0800, Wan ZongShun wrote:
>>
>> Yes, I think so too, due to I never meet this type 'held' in LAKML and
>> LKML, so I am amazed at this 'held' in the list.
>
> Not all of the mailman settings were imported from the old list to the
> new one. If the moderators want to increase the size limit, they are
> free to do that -- the hardware can certainly cope.
>
> But Eric is right that patches above 40KiB are fairly suspect anyway --
> why is it so large?

The patch is one of the series patches of changing my w90x900 platform
architecture, originally, I submitted nine patches to mend my
platform, Russell has reviewed them and
hoped me to re-arrange the patches, so I Integrated the nine patches
into three ones.

I just ignore the size of patch issue.

> --
> dwmw2
>
>



-- 
Wan z.s

My first patch to the mailist, but being held

[Eric Miao]

On Tue, Sep 1, 2009 at 10:25 PM, Wan ZongShun<mcuos.com@gmail.com> wrote:
> Dear David,
>
> 2009/9/1 David Woodhouse <dwmw2@infradead.org>:
>> On Tue, 2009-09-01 at 22:05 +0800, Wan ZongShun wrote:
>>>
>>> Yes, I think so too, due to I never meet this type 'held' in LAKML and
>>> LKML, so I am amazed at this 'held' in the list.
>>
>> Not all of the mailman settings were imported from the old list to the
>> new one. If the moderators want to increase the size limit, they are
>> free to do that -- the hardware can certainly cope.
>>
>> But Eric is right that patches above 40KiB are fairly suspect anyway --
>> why is it so large?
>
> The patch is one of the series patches of changing my w90x900 platform
> architecture, originally, I submitted nine patches to mend my
> platform, Russell has reviewed them and
> hoped me to re-arrange the patches, so I Integrated the nine patches
> into three ones.
>
> I just ignore the size of patch issue.
>

You can try 'git format-patch -M' to shrink the size of the patch significantly
if renaming is involved.

My first patch to the mailist, but being held

[Wan ZongShun]

Dear Eric,


2009/9/1 Eric Miao <eric.y.miao@gmail.com>:
> On Tue, Sep 1, 2009 at 10:25 PM, Wan ZongShun<mcuos.com@gmail.com> wrote:
>> Dear David,
>>
>> 2009/9/1 David Woodhouse <dwmw2@infradead.org>:
>>> On Tue, 2009-09-01 at 22:05 +0800, Wan ZongShun wrote:
>>>>
>>>> Yes, I think so too, due to I never meet this type 'held' in LAKML and
>>>> LKML, so I am amazed at this 'held' in the list.
>>>
>>> Not all of the mailman settings were imported from the old list to the
>>> new one. If the moderators want to increase the size limit, they are
>>> free to do that -- the hardware can certainly cope.
>>>
>>> But Eric is right that patches above 40KiB are fairly suspect anyway --
>>> why is it so large?
>>
>> The patch is one of the series patches of changing my w90x900 platform
>> architecture, originally, I submitted nine patches to mend my
>> platform, Russell has reviewed them and
>> hoped me to re-arrange the patches, so I Integrated the nine patches
>> into three ones.
>>
>> I just ignore the size of patch issue.
>>
>
> You can try 'git format-patch -M' to shrink the size of the patch significantly
> if renaming is involved.
>

I have done this, thanks.



-- 
Wan z.s

My first patch to the mailist, but being held

[Pavel Machek]

Hi!

> > Yes, I think so too, due to I never meet this type 'held' in LAKML and
> > LKML, so I am amazed at this 'held' in the list.
> 
> Not all of the mailman settings were imported from the old list to the
> new one. If the moderators want to increase the size limit, they are
> free to do that -- the hardware can certainly cope.

Could we do that?

> But Eric is right that patches above 40KiB are fairly suspect anyway --
> why is it so large?

Not all patches are for inclusion, some are for comments, and some
drivers are fairly big (dream camera). lkml has limit 400KB iirc, so
40KB is very low...

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html