Skip to content

A couple of tweaks#1544

Merged
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master
Oct 27, 2022
Merged

A couple of tweaks#1544
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master

Conversation

@NiLuJe

@NiLuJe NiLuJe commented Oct 27, 2022

Copy link
Copy Markdown
Member
  • 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

poire-z commented Oct 27, 2022

Copy link
Copy Markdown
Contributor

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

@Frenzie

Frenzie commented Oct 27, 2022

Copy link
Copy Markdown
Member

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

@NiLuJe

NiLuJe commented Oct 27, 2022

Copy link
Copy Markdown
Member Author

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

(I have other unpushed changes behind).

@NiLuJe

NiLuJe commented Oct 27, 2022

Copy link
Copy Markdown
Member Author

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