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

Supporting pagespeed on ARM devices? #1213

Open
pappkopp opened this issue Jun 12, 2016 · 14 comments
Open

Supporting pagespeed on ARM devices? #1213

pappkopp opened this issue Jun 12, 2016 · 14 comments

Comments

@pappkopp
Copy link

@pappkopp pappkopp commented Jun 12, 2016

Is there any further plans on adding support for arm devices with ngx_pagespeed?

I think that theres s a growing demand with more and more providers selling and hosting servers based on the arm architecture..

@hillsp
Copy link
Contributor

@hillsp hillsp commented Jun 13, 2016

We don't currently have plans to support ARM mostly because I don't think we've especially been peppered with requests for it.

We have a lot of library dependencies and I imagine at least some of them don't support ARM, which makes this a lot more painful.

@ruimnetob
Copy link

@ruimnetob ruimnetob commented May 21, 2017

Will there be a mod_pagespeed supported for armhf? I am running a webserver on my Pi and would be great to have pagespeed there.
with IoT and these ARM System on chip there will be more and more people requesting this

@nachoparker
Copy link

@nachoparker nachoparker commented Oct 24, 2017

+1 here, I am interested in trying this with NextCloudPi

@granvillepaut
Copy link

@granvillepaut granvillepaut commented Aug 28, 2018

I too am running a webserver on a Pi-like board and would be really interested in having this module.

@georges-2000
Copy link

@georges-2000 georges-2000 commented Oct 9, 2018

+1 im interested in this module

@iishtiaque
Copy link

@iishtiaque iishtiaque commented Oct 25, 2018

+1 It would be a really nice thing to have

@thegr33nhornet
Copy link

@thegr33nhornet thegr33nhornet commented Dec 24, 2018

+1 ARM based servers are becoming much more mainstream.

@FalcoGer
Copy link

@FalcoGer FalcoGer commented Feb 6, 2019

I to would like this to work on armv7 and armv8 (64bit) for a raspberry running apache.

@L422Y
Copy link

@L422Y L422Y commented Oct 29, 2019

+1 really would be nice

@petter5
Copy link

@petter5 petter5 commented Dec 11, 2019

+1 would love to have pagespeed for the pi4 - Raspbian - tried the Automated Install build from source, but no sucess,,,

@iishtiaque
Copy link

@iishtiaque iishtiaque commented Dec 20, 2019

@pilfs
Copy link

@pilfs pilfs commented May 10, 2020

I have ngx_pagespeed running on aarch64 and armv7l.

You can find the PSOL binaries and instructions here:

https://gitlab.com/gusco/ngx_pagespeed_arm

I believe we are about to see ARM grow in the server space this year,
thought it would be a shame for ngx_pagespeed to miss out.

@jmarantz
Copy link
Contributor

@jmarantz jmarantz commented May 10, 2020

@jsrz
Copy link

@jsrz jsrz commented Jun 23, 2020

I tried @pilfs instructions and can verify that they do work. I tried it out on an AWS m6g instance.

That said, a few other things to note:

  • The ngx_pagespeed build script downloads pre-built binaries for PSOL. Thus, in order for that script to work on Arm machines, we will need pre-built binaries released for aarch64 (arm64) and armv7l.

  • The ngx_pagespeed script does offer a switch (-s) which tells it to build PSOL rather than download the pre-built binary. This script is broken. I ran it on both Arm and x86 instances. If this script weren't broken, then you would not necessarily need the pre-built PSOL binaries.

  • I was poking around at the patches @pilfs posted. I see a bunch of stuff around atomic operation intrinsics. There are legacy __sync, and newer __atomic intrinsics. I'd guess that there is room to improve performance, especially since newer Arm 8.2 added newer/improved atomic instructions. I don't claim to know what should be done, but it feels like there's room for lots of improvement.

All of that said, I think pilfs changes should be merged to get the Arm support in.

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

Successfully merging a pull request may close this issue.

None yet
You can’t perform that action at this time.