Skip to content

Enable BCACHE on Raspberry Pi build#9550

Merged
igorpecovnik merged 2 commits into
armbian:mainfrom
aulanov:enable-bcache
Mar 17, 2026
Merged

Enable BCACHE on Raspberry Pi build#9550
igorpecovnik merged 2 commits into
armbian:mainfrom
aulanov:enable-bcache

Conversation

@aulanov
Copy link
Copy Markdown
Contributor

@aulanov aulanov commented Mar 17, 2026

Documentation summary for feature / change

This PR enables CONFIG_BCACHE on bcm2711 builds.
CONFIG_BCACHE is enabled on most other boards. There are no reasons it should be enabled on other boards but not Raspberries.

How Has This Been Tested?

I've tested this by compiling the kernel and making sure bcache works on Raspberry Pi 5.

Checklist:

Please delete options that are not relevant.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

Summary by CodeRabbit

  • New Features
    • Added BCACHE filesystem cache backend as a loadable kernel module. This change makes BCACHE available in both current and edge kernel configurations, allowing it to be loaded dynamically at runtime where supported. Users may see the module available for enabling or loading via standard kernel module management tools.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f9702883-f204-4b52-b88e-d532a5c1cc4c

📥 Commits

Reviewing files that changed from the base of the PR and between e91b191 and 8148ea4.

📒 Files selected for processing (2)
  • config/kernel/linux-bcm2711-current.config
  • config/kernel/linux-bcm2711-edge.config
🚧 Files skipped from review as they are similar to previous changes (1)
  • config/kernel/linux-bcm2711-current.config

📝 Walkthrough

Walkthrough

Added the kernel configuration symbol CONFIG_BCACHE set to module ("m") in two BCM2711 kernel config files, enabling BCACHE as a loadable kernel module in those defconfigs.

Changes

Cohort / File(s) Summary
Kernel configurations
config/kernel/linux-bcm2711-current.config, config/kernel/linux-bcm2711-edge.config
Added CONFIG_BCACHE=m to enable BCACHE as a loadable kernel module in both BCM2711 kernel configuration files.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I found a config, tidy and neat,
A little module hopping on its feet.
BCACHE tucked in as "m" with cheer,
Ready to cache, springing near! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately and specifically describes the main change: enabling BCACHE support for the Raspberry Pi (bcm2711) kernel build. It is concise, clear, and directly related to the changeset modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Tip

You can customize the high-level summary generated by CodeRabbit.

Configure the reviews.high_level_summary_instructions setting to provide custom instructions for generating the high-level summary.

@github-actions github-actions Bot added the 05 Milestone: Second quarter release label Mar 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Hey @aulanov! 👋

Thanks for submitting your first pull request to the Armbian project — we're excited to have you contributing! 🧡
Your effort doesn’t just improve Armbian — it benefits the entire community of users and developers.

If you'd like to stay informed about project updates or collaborate more closely with the team,
you can optionally share some personal contact preferences at armbian.com/update-data.
This helps us keep in touch without relying solely on GitHub notifications.

Also, don’t forget to ⭐ star the repo if you haven’t already — and welcome aboard! 🚀

@github-actions github-actions Bot added size/small PR with less then 50 lines Needs review Seeking for review Hardware Hardware related like kernel, U-Boot, ... labels Mar 17, 2026
@EvilOlaf
Copy link
Copy Markdown
Member

Can you check against edge and legacy config too?

@aulanov
Copy link
Copy Markdown
Contributor Author

aulanov commented Mar 17, 2026

Can you check against edge and legacy config too?

Legacy already has this enabled. I've now updated edge config as well.

Copy link
Copy Markdown
Member

@igorpecovnik igorpecovnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can enable that, no problem.

@igorpecovnik igorpecovnik merged commit 0d9a9cd into armbian:main Mar 17, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

05 Milestone: Second quarter release Hardware Hardware related like kernel, U-Boot, ... Needs review Seeking for review size/small PR with less then 50 lines

Development

Successfully merging this pull request may close these issues.

3 participants