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

A couple of tweaks #1544

Merged
merged 2 commits into from
Oct 27, 2022
Merged

A couple of tweaks #1544

merged 2 commits into from
Oct 27, 2022

Conversation

NiLuJe
Copy link
Member

@NiLuJe NiLuJe commented Oct 27, 2022

  • ffi/posix: Add fflush. Necessary for fsync shenanigans ;).
  • fb/mxcfb: Don't be cute with update markers, just wraparound at UINT32_MAX. I don't think anything really cares about discontinuous markers, but, better be safe than sorry... (The original intent was to stay low to keep our markers lower than the stock framework on Kindle, but given that you can't find any better kernels than lab126's, it's unlikely to cause any trouble in Kindle-land. Plus, outside of fast mode, which is only used sporadically (read: virtual keyboard), markers are collected differently (via the new wait_for_any ioctl) by the stock framework on Bellatrix anyway, and given that the new Basic is on the Bellatrix platform, lab126 seems to be pretty done with i.MX SoCs ;)).

This change is Reviewable

On the off-chance some wonky kernels out there have a
special aversion to discontinuous markers...
@poire-z
Copy link
Contributor

poire-z commented Oct 27, 2022

(Should I bring this one with my bump for xtext and other libs ?)

@Frenzie
Copy link
Member

Frenzie commented Oct 27, 2022

I don't see why not — it should be 100% harmless. ^_^

@NiLuJe
Copy link
Member Author

NiLuJe commented Oct 27, 2022

Nah, I'll do it later tonight ;).

(I have other unpushed changes behind).

@NiLuJe
Copy link
Member Author

NiLuJe commented Oct 27, 2022

Actually, no, it's good, the rest (if any) will be in the KB PR, so this can go in ;).

@NiLuJe NiLuJe merged commit 21f4b97 into koreader:master Oct 27, 2022
@NiLuJe NiLuJe changed the title ffi: Add fflush to posix_h A couple of tweaks Oct 27, 2022
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

Successfully merging this pull request may close these issues.

3 participants