CI: Add patch check by using build kernel #250
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
.github/workflows/build-kernel.yml
Outdated
| @@ -0,0 +1,58 @@ | |||
| name: build-kernel | |||
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| @@ -0,0 +1,58 @@ | |||
| name: build-kernel | |||
|
|
|||
| on: | |||
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| name: build-kernel | ||
|
|
||
| on: | ||
| push: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
|
|
||
| on: | ||
| push: | ||
| pull_request: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| on: | ||
| push: | ||
| pull_request: | ||
| workflow_dispatch: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| pull_request: | ||
| workflow_dispatch: | ||
|
|
||
| env: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| workflow_dispatch: | ||
|
|
||
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
|
|
||
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig | ||
| KBUILD_BUILD_HOST: deepin-kernel-builder |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig | ||
| KBUILD_BUILD_HOST: deepin-kernel-builder | ||
| email: [email protected] |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| KBUILD_BUILD_USER: deepin-kernel-sig | ||
| KBUILD_BUILD_HOST: deepin-kernel-builder | ||
| email: [email protected] | ||
| KDEB_COMPRESS: xz |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| KBUILD_BUILD_HOST: deepin-kernel-builder | ||
| email: [email protected] | ||
| KDEB_COMPRESS: xz | ||
| KDEB_CHANGELOG_DIST: unstable |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| email: [email protected] | ||
| KDEB_COMPRESS: xz | ||
| KDEB_CHANGELOG_DIST: unstable | ||
| OUT_DIR: out |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| KDEB_CHANGELOG_DIST: unstable | ||
| OUT_DIR: out | ||
|
|
||
| permissions: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| OUT_DIR: out | ||
|
|
||
| permissions: | ||
| pull-requests: read |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| permissions: | ||
| pull-requests: read | ||
|
|
||
| jobs: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| pull-requests: read | ||
|
|
||
| jobs: | ||
| build-kernel: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
|
|
||
| jobs: | ||
| build-kernel: | ||
| runs-on: ubuntu-latest |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| jobs: | ||
| build-kernel: | ||
| runs-on: ubuntu-latest | ||
| steps: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| build-kernel: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: "Update APT sources" |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: "Update APT sources" | ||
| run: | |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| steps: | ||
| - name: "Update APT sources" | ||
| run: | | ||
| sudo apt update |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| run: | | ||
| sudo apt update | ||
|
|
||
| - name: Free Disk Space (Ubuntu) |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| sudo apt update | ||
|
|
||
| - name: Free Disk Space (Ubuntu) | ||
| uses: jlumbroso/free-disk-space@main |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
|
|
||
| - name: Free Disk Space (Ubuntu) | ||
| uses: jlumbroso/free-disk-space@main | ||
| with: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| - name: Free Disk Space (Ubuntu) | ||
| uses: jlumbroso/free-disk-space@main | ||
| with: | ||
| # this might remove tools that are actually needed, |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| uses: jlumbroso/free-disk-space@main | ||
| with: | ||
| # this might remove tools that are actually needed, | ||
| # if set to "true" but frees about 6 GB |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| with: | ||
| # this might remove tools that are actually needed, | ||
| # if set to "true" but frees about 6 GB | ||
| tool-cache: true |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
.github/workflows/build-kernel.yml
Outdated
| # this might remove tools that are actually needed, | ||
| # if set to "true" but frees about 6 GB | ||
| tool-cache: true | ||
| # all of these default to true, but feel free to set to |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
opsiff
force-pushed
the
opsiff-patch-build-kernel-ci
branch
from
a6e708f to
3c02189
Compare
| @@ -0,0 +1,30 @@ | |||
| name: build kernel | |||
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| @@ -0,0 +1,30 @@ | |||
| name: build kernel | |||
| on: | |||
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| @@ -0,0 +1,30 @@ | |||
| name: build kernel | |||
| on: | |||
| push: | |||
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| name: build kernel | ||
| on: | ||
| push: | ||
| pull_request: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| on: | ||
| push: | ||
| pull_request: | ||
| workflow_dispatch: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| pull_request: | ||
| workflow_dispatch: | ||
|
|
||
| env: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| workflow_dispatch: | ||
|
|
||
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
deepin pr auto review关键摘要:
是否建议立即修改: |
|
|
||
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig | ||
| KBUILD_BUILD_HOST: deepin-kernel-builder |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| env: | ||
| KBUILD_BUILD_USER: deepin-kernel-sig | ||
| KBUILD_BUILD_HOST: deepin-kernel-builder | ||
| email: [email protected] |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| KBUILD_BUILD_HOST: deepin-kernel-builder | ||
| email: [email protected] | ||
|
|
||
| permissions: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| email: [email protected] | ||
|
|
||
| permissions: | ||
| pull-requests: read |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| permissions: | ||
| pull-requests: read | ||
|
|
||
| jobs: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| pull-requests: read | ||
|
|
||
| jobs: | ||
| build-kernel: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
|
|
||
| jobs: | ||
| build-kernel: | ||
| runs-on: [self-hosted, linux] |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| jobs: | ||
| build-kernel: | ||
| runs-on: [self-hosted, linux] | ||
| steps: |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| build-kernel: | ||
| runs-on: [self-hosted, linux] | ||
| steps: | ||
| - uses: actions/checkout@v3 |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| runs-on: [self-hosted, linux] | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - name: "Install Deps" |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - name: "Install Deps" | ||
| run: | |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| - uses: actions/checkout@v3 | ||
| - name: "Install Deps" | ||
| run: | | ||
| git config --global user.email $email |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| - name: "Install Deps" | ||
| run: | | ||
| git config --global user.email $email | ||
| git config --global user.name $KBUILD_BUILD_USER |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| git config --global user.email $email | ||
| git config --global user.name $KBUILD_BUILD_USER | ||
|
|
||
| - name: "Compile kernel" |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| git config --global user.name $KBUILD_BUILD_USER | ||
|
|
||
| - name: "Compile kernel" | ||
| run: | |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
|
|
||
| - name: "Compile kernel" | ||
| run: | | ||
| # .config |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| - name: "Compile kernel" | ||
| run: | | ||
| # .config | ||
| make deepin_x86_desktop_defconfig |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
| run: | | ||
| # .config | ||
| make deepin_x86_desktop_defconfig | ||
| make -j$(nproc) |
There was a problem hiding this comment.
WARNING: It's generally not useful to have the filename in the file
commit d92a7580392ad4681b1d4f9275d00b95375ebe01 upstream. Only export struct fb_info.fix.smem_start if that is required by the user and the memory does not come from vmalloc(). Setting struct fb_info.fix.smem_start breaks systems where DMA memory is backed by vmalloc address space. An example error is shown below. [ 3.536043] ------------[ cut here ]------------ [ 3.540716] virt_to_phys used for non-linear address: 000000007fc4f540 (0xffff800086001000) [ 3.552628] WARNING: CPU: 4 PID: 61 at arch/arm64/mm/physaddr.c:12 __virt_to_phys+0x68/0x98 [ 3.565455] Modules linked in: [ 3.568525] CPU: 4 PID: 61 Comm: kworker/u12:5 Not tainted 6.6.23-06226-g4986cc3e1b75-dirty deepin-community#250 [ 3.577310] Hardware name: NXP i.MX95 19X19 board (DT) [ 3.582452] Workqueue: events_unbound deferred_probe_work_func [ 3.588291] pstate: 60400009 (nZCv daif +PAN -UAO -TCO -DIT -SSBS BTYPE=--) [ 3.595233] pc : __virt_to_phys+0x68/0x98 [ 3.599246] lr : __virt_to_phys+0x68/0x98 [ 3.603276] sp : ffff800083603990 [ 3.677939] Call trace: [ 3.680393] __virt_to_phys+0x68/0x98 [ 3.684067] drm_fbdev_dma_helper_fb_probe+0x138/0x238 [ 3.689214] __drm_fb_helper_initial_config_and_unlock+0x2b0/0x4c0 [ 3.695385] drm_fb_helper_initial_config+0x4c/0x68 [ 3.700264] drm_fbdev_dma_client_hotplug+0x8c/0xe0 [ 3.705161] drm_client_register+0x60/0xb0 [ 3.709269] drm_fbdev_dma_setup+0x94/0x148 Additionally, DMA memory is assumed to by contiguous in physical address space, which is not guaranteed by vmalloc(). Resolve this by checking the module flag drm_leak_fbdev_smem when DRM allocated the instance of struct fb_info. Fbdev-dma then only sets smem_start only if required (via FBINFO_HIDE_SMEM_START). Also guarantee that the framebuffer is not located in vmalloc address space. Signed-off-by: Thomas Zimmermann <[email protected]> Reported-by: Peng Fan (OSS) <[email protected]> Closes: https://lore.kernel.org/dri-devel/[email protected]/ Reported-by: Geert Uytterhoeven <[email protected]> Closes: https://lore.kernel.org/dri-devel/CAMuHMdX3N0szUvt1VTbroa2zrT1Nye_VzPb5qqCZ7z5gSm7HGw@mail.gmail.com/ Fixes: a51c766 ("drm/fb-helper: Consolidate CONFIG_DRM_FBDEV_LEAK_PHYS_SMEM") Tested-by: Geert Uytterhoeven <[email protected]> Reviewed-by: Daniel Vetter <[email protected]> Cc: <[email protected]> # v6.4+ Link: https://patchwork.freedesktop.org/patch/msgid/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
Modify a GitHub action workflow to check pull requests by build kernel