-
Notifications
You must be signed in to change notification settings - Fork 798
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
ionice: ioprio_set failed: Invalid argument #498
Comments
Hi any updates? |
Sorry for the delay. Without an strace it is difficult to say what exactly (what's unimplemented) you are running into. Could you collect one and share it out? |
Certainly. Do you have instructions? |
strace -o strace_file_name -ff failing_command And then share out the strace file. |
user@PC:~/cm12$ strace -o strace_file_name -ff "schedtool -B -n 1 -e ionice -n 1 make -C |
Sorry I believe the "-ff" argument must come before "-o". After you run the below please share strace_file_name.* with us. strace -ff -o strace_file_name failing_command |
execve("/usr/bin/schedtool", ["schedtool", "-B", "-n", "1", "-e", "ionice", "-n", "1", "make", "-C", "-j4"], [/* 17 vars /]) = 0 |
Thanks, this looks like it's failing due to the ioprio_set system call not being implemented. |
Hopefully is it an easy fix? |
@MrObvious - I have opened a bug to track this, but unfortunately it won't make the cut for the Anniversary update. But, as far as Insider ring is considered, it shouldn't matter much because they should (hopefully) start flighting post RS1 bits soon. |
This error is extremely important to fix. If bash on Windows 10 can not build rom android os, this is a very bad thing |
Any updates on when this will be pushed out to the fast ring? |
I have not noticed any updates to fix this error. @MrObvious |
Unfortunately this syscall isn't going to make the Windows 10 Anniversary Update. However, this is on our backlog and we'll definitely be looking into it. |
I hope to have this fix update coming soon. @benhillis |
Use make instead of mka |
As a workaround: Here's a replacement 'ionice' that's a no-op. It'd be interesting if someone could give it a try: |
If without ionce is not build rom android @aseering how do you feel ?? |
@thanhphat11 -- Unfortunately, I don't understand your question... Could you re-phrase it? |
Currently ionce error and I can not make it brunch rom android build tool can not check when I rom android brunch. Does your ionce can solve this problem ?? @aseering |
Brunch isn't needed to build an android ROM On Aug 11, 2016 5:53 AM, "Tsần Thành Phát" notifications@github.com wrote:
|
Ah, thanks for re-phrasing. I hope my ionice will solve that problem. Could you try it and see if it works? ionice doesn't work in WSL; I can't fix that. But I think the Android ROM build doesn't really need ionice. I think it just uses ionice for better performance. So my script just hides the error and disables ionice. I think that will work. But the only way to know for sure is to try it. |
Hi, i tried your ionice workaround, and also the lunch/breakfast and then make combo, but either way i just get the output of what it builds (Platform, Target and so on) and afterwards nothing, CPU usage is at ~15% on all cores but no output anymore on the bash, if i cancel, i get loads of python errors: File "build/tools/findleaves.py", line 100, in
File "build/tools/findleaves.py", line 94, in main |
I have the exact same issue as @SGCMarkus when compiling CyanogenMod 13 for Oneplus 3 |
The issue is still occurring on the latest build 14901. |
Update : This issue is still NOT solved in build 14926 |
I think this is failing in findleaves.py because of some circular structure of symbolic links. findleaves is called from build/core/main.mk. It just might instead be a MAX_PATH issue. |
I thought no one cared to fix this problem |
This issue has been forgotten @ALL |
I've opened an internal bug to track this. |
Anyone managed fix to this? |
Any update on this? I've heard that the creators update will include updates to the Linux subsystem, will it have this syscall? |
@matyasf Please read thread above: We won't have time to implement the ioprio_set system call for Creators Update, but we have an internal bug tracking this issue and it's on the backlog for a future release. |
@bitcrazed - Thanks for the update! |
This issue is still occurring on build # 16184 (latest build in fast ring). |
It seems to work perfectly using But still getting error with brunch. |
Is your CPU in use when you're in this state? If so, what process is using CPU? If not, could you try running |
Hi there! Is there any ETA on this issue? It's still up on build 16232. |
Taking a look at this now. |
Some basic support for this system call is making its way to an Insider build. |
This is fixed in Fall Creators Update and any build > 16273. |
Doing an Android compile, I cannot go much further on the brunch stage. Let me know if this is enough detail, or you need more.
For reference: http://stackoverflow.com/questions/37444998/error-ionice-ioprio-set-failed-invalid-argument-while-trying-to-build-custom
The text was updated successfully, but these errors were encountered: