Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JUMBO FRAME SUPPORT #137

Closed
DCNESS1213 opened this issue Mar 5, 2024 · 10 comments
Closed

JUMBO FRAME SUPPORT #137

DCNESS1213 opened this issue Mar 5, 2024 · 10 comments

Comments

@DCNESS1213
Copy link

can this be used to support jumbo frame up to 9600.

@Killeroo
Copy link
Owner

Depending on your network setup then yes. You can send a ping that is over 9600 bytes using PowerPing but ultimately it is up to your network adapter if that will be sent as a jumbo frame, PowerPing cannot affect how the data is actually transmitted.

@DCNESS1213
Copy link
Author

im try to use this syntax but not able to send the ping due to an error. Am i doing it right or there is another way to set the Frame size.

c:>powerping 8.8.8.8 -s 1500 -ti 6 -sym 1 -c 1000

image

@Killeroo
Copy link
Owner

Ah yes this does seem to be a PowerPing bug, it's been reported before (#119). I will look to get this fixed soon, thanks for reporting.

Killeroo added a commit that referenced this issue Jun 2, 2024
…bytes (#119 & #137)

Fixed receive buffer not scaling with size of packet being sent
@Killeroo
Copy link
Owner

Killeroo commented Jun 5, 2024

I've fixed this bug and you should be able to send a ping of over 1500 bytes in the latest release (you can download it here).

Just any FYI, in testing I found that 8.8.8.8 won't reply to any pings bigger than 1000 bytes :) But I did verify that the full 1500 bytes are getting sent on the wire.

@DCNESS1213
Copy link
Author

Thanks sir i tried this just now.

Just sending ping to my router(Cisco 8200)

image

@Killeroo
Copy link
Owner

Killeroo commented Jun 7, 2024

Found an issue with parsing the returned packet, thanks for reporting this. Will update the release later today 👍

@DCNESS1213
Copy link
Author

thanks sir, i appreciate the work you put in to this

@Killeroo
Copy link
Owner

No worries, I've updated the latest release now. Try it out, that should fix the exception you were seeing.

@DCNESS1213
Copy link
Author

Thanks will try it now. looks great

@DCNESS1213
Copy link
Author

everything in order. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants