Is these Queue setting right?

Hay,
Please i need you to see if my queue settings good or bad.
My network
Internet > Router Mikrotik > Switch/hub > Clients
My internet line 1M download /256k upload
I wamt the speed to divide on active users and i reduce P2P programss to 128kDown/64kUp

My setting:

INTERFACE QUEUE

0 ll ethernet-default
1 internal ethernet-default

queue type> print
0 name="default" kind=pfifo pfifo-limit=50

1 name="ethernet-default" kind=pfifo pfifo-limit=50

2 name="wireless-default" kind=sfq sfq-perturb=5 sfq-allot=1514

3 name="synchronous-default" kind=red red-limit=60 red-min-threshold=10
red-max-threshold=50 red-burst=20 red-avg-packet=1000

4 name="hotspot-default" kind=sfq sfq-perturb=5 sfq-allot=1514

5 name="pcq-download" kind=pcq pcq-rate=0 pcq-limit=50
pcq-classifier=dst-address pcq-total-limit=2000

6 name="pcq-upload" kind=pcq pcq-rate=0 pcq-limit=50
pcq-classifier=src-address pcq-total-limit=2000

7 name="default-small" kind=pfifo pfifo-limit=10


queue simple> print
Flags: X - disabled, I - invalid, D - dynamic
0 D name="" dst-address=0.0.0.0/0 interface=internal parent=none
direction=both priority=8 queue=hotspot-default/hotspot-default
limit-at=0/0 max-limit=0/0 total-queue=default-small

1 name="P2PLinit" dst-address=0.0.0.0/0 interface=all parent=none
direction=both priority=8 queue=default-small/default-small
limit-at=64000/128000 max-limit=64000/128000 total-queue=default-small
p2p=all-p2p


queue tree> print
Flags: X - disabled, I - invalid
0 name="Download" parent=internal packet-mark="" limit-at=0 queue=default
priority=8 max-limit=1024000 burst-limit=0 burst-threshold=0
burst-time=0s

1 name="queue2" parent=Download packet-mark=users limit-at=0
queue=pcq-download priority=8 max-limit=0 burst-limit=0
burst-threshold=0 burst-time=0s

2 name="Upload" parent=ll packet-mark="" limit-at=0 queue=default priority=8
max-limit=256000 burst-limit=0 burst-threshold=0 burst-time=0s

3 name="queue4" parent=Upload packet-mark=users limit-at=0 queue=pcq-upload
priority=8 max-limit=0 burst-limit=0 burst-threshold=0 burst-time=0s

PCQ settings are wrong and you can’t use simple queues and queue tree together.

Can you help me and explain to me what i should remove and what i have to keep
greatful for any help

http://wiki.mikrotik.com/wiki/PCQ_Examples

Pick either Queue Tree or Simple Queues and stick with just one.

Thank you first for your help.
Now i remove pcq_download and pcq_upload and remove all rules in queue tree and keep only the rules in queue simple

Like that

INTERFACE QUEUE

0 ll ethernet-default
1 internal ethernet-default

print
0 name="default" kind=pfifo pfifo-limit=50

1 name="ethernet-default" kind=pfifo pfifo-limit=50

2 name="wireless-default" kind=sfq sfq-perturb=5 sfq-allot=1514

3 name="synchronous-default" kind=red red-limit=60 red-min-threshold=10
red-max-threshold=50 red-burst=20 red-avg-packet=1000

4 name="hotspot-default" kind=sfq sfq-perturb=5 sfq-allot=1514

5 name="default-small" kind=pfifo pfifo-limit=10

queue simple> print
Flags: X - disabled, I - invalid, D - dynamic
0 D name="" dst-address=0.0.0.0/0 interface=internal parent=none
direction=both priority=8 queue=hotspot-default/hotspot-default
limit-at=0/0 max-limit=0/0 total-queue=default-small

1 name="P2PLinit" dst-address=0.0.0.0/0 interface=all parent=none
direction=both priority=8 queue=default-small/default-small
limit-at=64000/128000 max-limit=64000/128000 total-queue=default-small
p2p=all-p2p

Is these now perfect stting?
About the rule P2P sould i keep it first rule in queue simple or keep it as its now?

Also,
In documentation
http://www.mikrotik.com/testdocs/ros/2.9/root/queue.php
They use queue tree and queue simple in the same time
is there any confuse or mistake here or i dont understand it very well?

You need to add your PCQ’s back and under the simple queue use PCQ for the queue type

Thank you for your help i will try now and will reply with the result

Now everything seems fine and good.
My final setting is

/ queue type
set default name=“default” kind=pfifo pfifo-limit=50
set ethernet-default name=“ethernet-default” kind=pfifo pfifo-limit=50
set wireless-default name=“wireless-default” kind=sfq sfq-perturb=5 sfq-allot=1514
set synchronous-default name=“synchronous-default” kind=red red-limit=60 red-min-threshold=10
red-max-threshold=50 red-burst=20 red-avg-packet=1000
set hotspot-default name=“hotspot-default” kind=sfq sfq-perturb=5 sfq-allot=1514
add name=“pcq-download” kind=pcq pcq-rate=0 pcq-limit=50 pcq-classifier=dst-address pcq-total-limit=2000
add name=“pcq-upload” kind=pcq pcq-rate=0 pcq-limit=50 pcq-classifier=src-address pcq-total-limit=2000
add name=“default-small” kind=pfifo pfifo-limit=10


queue simple> print
Flags: X - disabled, I - invalid, D - dynamic
0 D name=“” target-addresses=10.0.0.1/32 dst-address=0.0.0.0/0 interface=internal parent=none
direction=both priority=8 queue=pcq-upload/pcq-download limit-at=0/0 max-limit=0/0
total-queue=default-small

1 name=“Limit-Local” target-addresses=10.0.0.0/24 dst-address=0.0.0.0/0 interface=internal parent=none
direction=both priority=8 queue=pcq-upload/pcq-download limit-at=0/0 max-limit=256000/1048000
total-queue=default-small

2 X name=“P2PLinit” dst-address=0.0.0.0/0 interface=all parent=none direction=both priority=8
queue=default-small/default-small limit-at=64000/128000 max-limit=64000/128000
total-queue=default-small p2p=all-p2p


queue tree> print
Flags: X - disabled, I - invalid
0 name=“Download” parent=internal packet-mark=“” limit-at=0 queue=default priority=8 max-limit=1024000
burst-limit=0 burst-threshold=0 burst-time=0s

1 name=“queue2” parent=Download packet-mark=users limit-at=0 queue=default priority=8 max-limit=0
burst-limit=0 burst-threshold=0 burst-time=0s

2 name=“Upload” parent=ll packet-mark=“” limit-at=0 queue=default priority=8 max-limit=256000 burst-limit=0
burst-threshold=0 burst-time=0s

3 name=“queue4” parent=Upload packet-mark=users limit-at=0 queue=default priority=8 max-limit=0
burst-limit=0 burst-threshold=0 burst-time=0s

Is these perfect setting and good?

Also
Now if some one try download file and its already in cache is these limition will active or not?
Becuase cache content i want it to be unlimited