** Tags added: cpe-onsite
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after reboot
Status in linux package in Ubuntu:
Triaged
S
I reviewed this once again:
1. persistent /dev/bcache - not possible or needed due to how kernel
enumeration works
2. our userspace handles /dev/by-dname/ properly so we just need
CACHED_UUID given to us during coldplug in some way to properly create
/dev/bcache/by-uuid/ symlinks and consequent
So, the kernel fix could look like this:
* instead of just failing with "device already registered"
* the kernel would emit another uevent with CACHED_UUID which would be
processed another time to set up a symlink
It seems like this approach with "synthetic uevents" may be used for
other purpose
We do run udevd at the initramfs stage in init-top and stop it in init-
bottom:
➜ scripts git:(ubuntu/xenial-updates) ✗ grep -RiP udev
init-bottom/udev:# Stop udevd, we'll miss a few events while we run init, but
we catch up
init-bottom/udev:udevadm control --exit
init-top/udev:SYSTEMD_LOG_LEVE
Let's assume there is a systemd fault here too, thus marking systemd as
affected. I will try to play around with this, but may get to this by
December 14th the earliest.
** Also affects: systemd (Ubuntu)
Importance: Undecided
Status: New
--
You received this bug notification because yo
After booting with break=bottom kernel parameter I can see that symlinks
are in fact present at the initramfs stage.
They are gone afterwards post pivot_root & systemd start (the bcache dir
isn't even present while it was there)
ubuntu@maas-xenial4:~$ ls /dev/disk/
by-dname by-id by-label by-p
Just in case also checked with this rule that creates links at /dev
/dev/disk/by-uuid /dev/bcache/by-uuid https://paste.ubuntu.com/26081756/
They are all there originally (see the screenshot).
But not there afterwards. So it's not just a problem with a different
path.
https://paste.ubuntu.com/26
** Bug watch added: Debian Bug tracker #844775
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=844775
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uu
Ryan,
As discussed in IRC, I tweaked locations for testing in #11 and added
all paths in #14.
Looks like after `mount -n -o move /dev ${rootmnt}/dev` ${rootmnt}/dev
has the proper content (see the bcache rule as well)
https://paste.ubuntu.com/26082611/
== before ==
lrwxrwxrwx113
/de
Looks like masking systemd-udevd removes this behavior. Need to figure
out where it actually clears everything out.
ubuntu@maas-xenial4:~$ sudo systemctl mask systemd-udevd
11:32 PM Created symlink from /etc/systemd/system/systemd-udevd.service to
/dev/null.
# reboot
ubuntu@maas-xenial4:~$ ls /
Untested patch. Hoping to convey the change that's needed.
- decouple emitting a cached_dev CHANGE uevent which includes dev.uuid and
dev.label
from bch_cached_dev_run() which only happens when a bcacheX device is bound
to the
actual backing block device (bcache0 -> vdb)
- update bch_c
The attachment "bcache_always_emit_backing_dev_change_uevent.diff" seems
to be a patch. If it isn't, please remove the "patch" flag from the
attachment, remove the "patch" tag, and if you are a member of the
~ubuntu-reviewers, unsubscribe the team.
[This is an automated message performed by a Lau
** Tags removed: kernel-da-key
** Tags added: kernel-key
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after reboot
Status in linux
@Ryan
I tried to build an Artful test kernel with your patch. However, there
were compilation errors:
ubuntu-artful/drivers/md/bcache/super.c:868:9: error: 'env' undeclared (first
use in this function); did you mean '_end'?
kfree(env[1]);
^~~
_end
Did you have any issues c
It looks like env[] was declared in bch_cached_dev_emit_change(), but
not in bch_cached_dev_run().
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links
Revised patch, should fix error with kfree on env
** Patch added: "bcache_always_emit_backing_dev_change_uevent_v2.diff"
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1729145/+attachment/5018329/+files/bcache_always_emit_backing_dev_change_uevent_v2.diff
--
You received this bug notif
I built an Artful test kernel with the path provided by Ryan. The test
kernel can be downloaded from:
http://kernel.ubuntu.com/~jsalisbury/lp1729145/
Can those that can reproduce this bug give this test kernel a try?
--
You received this bug notification because you are a member of Kernel
Pack
I attached an update to your patch, Ryan.
I added a subject, commit log and couple of missing semi colons /
parenthesis.
Can you look it over and see if you approve of the log and subject?
I'll see if it applies to the other Ubuntu releases and builds on them
as well.
** Patch added: "0001-bcach
In the final patch we submit for SRU, it will also include your Signed-
off-by, I just forgot to add it in the attached one.
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
I built Xenial, Zesty and Bionic test kernels with your patch. They can
also be downloaded from:
http://kernel.ubuntu.com/~jsalisbury/lp1729145/
Can those that can reproduce this bug give this test kernel a try?
Zesty required upstream commit 4b758df21 as a prereq. Zesty never got it,
since
Thanks for testing and the patch, Ryan. I can submit and SRU request.
We should also send this upstream for inclusion in the mainline kernel.
Is that something you think you can do? If not, I can do it for you.
--
You received this bug notification because you are a member of Kernel
Packages,
Submitted SRU requests:
https://lists.ubuntu.com/archives/kernel-team/2017-December/088680.html
https://lists.ubuntu.com/archives/kernel-team/2017-December/088682.html
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https:
This bug was fixed in the package linux - 4.4.0-119.143
---
linux (4.4.0-119.143) xenial; urgency=medium
* linux: 4.4.0-119.143 -proposed tracker (LP: #1760327)
* Dell XPS 13 9360 bluetooth scan can not detect any device (LP: #1759821)
- Revert "Bluetooth: btusb: fix QCA Rome
This bug was fixed in the package linux - 4.15.0-15.16
---
linux (4.15.0-15.16) bionic; urgency=medium
* linux: 4.15.0-15.16 -proposed tracker (LP: #1761177)
* FFe: Enable configuring resume offset via sysfs (LP: #1760106)
- PM / hibernate: Make passing hibernate offsets more
This bug was fixed in the package linux - 4.13.0-38.43
---
linux (4.13.0-38.43) artful; urgency=medium
* linux: 4.13.0-38.43 -proposed tracker (LP: #1755762)
* Servers going OOM after updating kernel from 4.10 to 4.13 (LP: #1748408)
- i40e: Fix memory leak related filter prog
** Changed in: linux (Ubuntu Bionic)
Status: In Progress => Fix Committed
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after
This bug is awaiting verification that the kernel in -proposed solves
the problem. Please test the kernel and update this bug with the
results. If the problem is solved, change the tag 'verification-needed-
bionic' to 'verification-done-bionic'. If the problem still exists,
change the tag 'verifica
** Tags removed: verification-needed-bionic
** Tags added: kernel-fixup-verification-needed-bionic
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links
This bug is awaiting verification that the kernel in -proposed solves
the problem. Please test the kernel and update this bug with the
results. If the problem is solved, change the tag 'verification-needed-
bionic' to 'verification-done-bionic'. If the problem still exists,
change the tag 'verifica
This bug was erroneously marked for verification in bionic; verification
is not required and verification-needed-bionic is being removed.
** Tags removed: verification-needed-bionic
** Tags added: verification-done-bionic
--
You received this bug notification because you are a member of Kernel
Joseph forwarded this upstream. LKML mailing list post
https://lkml.org/lkml/2018/3/1/812 .
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not c
This bug is awaiting verification that the kernel in -proposed solves
the problem. Please test the kernel and update this bug with the
results. If the problem is solved, change the tag 'verification-needed-
artful' to 'verification-done-artful'. If the problem still exists,
change the tag 'verifica
This bug is awaiting verification that the kernel in -proposed solves
the problem. Please test the kernel and update this bug with the
results. If the problem is solved, change the tag 'verification-needed-
xenial' to 'verification-done-xenial'. If the problem still exists,
change the tag 'verifica
Artful verified:
ubuntu@ubuntu:~$ lsb_release -rd
Description:Ubuntu 17.10
Release:17.10
ubuntu@ubuntu:~$ uname -a
Linux ubuntu 4.13.0-38-generic #43-Ubuntu SMP Wed Mar 14 15:20:44 UTC 2018
x86_64 x86_64 x86_64 GNU/Linux
ubuntu@ubuntu:~$ apt-cache policy linux-image-virtual
linux-imag
Xenial verified.
root@ubuntu:~# lsb_release -rd
Description:Ubuntu 16.04.4 LTS
Release:16.04
root@ubuntu:~# uname -a
Linux ubuntu 4.4.0-117-generic #141-Ubuntu SMP Tue Mar 13 11:58:07 UTC 2018
x86_64 x86_64 x86_64 GNU/Linux
root@ubuntu:~# apt-cache policy linux-image-virtual
linux-ima
** Changed in: linux (Ubuntu Xenial)
Status: In Progress => Fix Committed
** Changed in: linux (Ubuntu Artful)
Status: In Progress => Fix Committed
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bug
** Description changed:
1. $ lsb_release -rd
Description: Ubuntu 17.10
Release: 17.10
2. $ apt-cache policy linux-image-`uname -r`
linux-image-4.13.0-16-generic:
- Installed: 4.13.0-16.19
- Candidate: 4.13.0-16.19
- Version table:
- *** 4.13.0-16.19 500
- 500 http
@jsalisbury Did you get a chance to forward the patch upstream? We've a
userspace fix in the bcache-tools package for now; I've sent that
upstream here:
https://github.com/koverstreet/bcache-tools/pull/1
Kent was asking about the kernel fix, to which I pointed to the patch
here.
--
You receive
@raharper, I got side tracked and it got pushed down on my list. I can
still send it upstream if you want. It's currently in Ubuntu as a SAUCE
patch.
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpa
** Changed in: linux (Ubuntu Zesty)
Status: In Progress => Won't Fix
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after rebo
** Merge proposal linked:
https://code.launchpad.net/~raharper/curtin/+git/curtin/+merge/336871
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links
Anyone interested in this bug, please take a look at:
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1861941
Good summary of the issue:
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1861941/comments/27
https://bugs.launchpad.net/ubuntu/+source/systemd/+bug/1861941/comments/47
ht
For older releases such as bionic we have one event:
UDEV [1304.304437] change /devices/virtual/block/bcache0 (block)
.ID_FS_TYPE_NEW=
ACTION=change
CACHED_LABEL=
CACHED_UUID=0192e0a7-9d8e-4771-9bb1-d730a2f66a9d
DEVLINKS=/dev/bcache/by-uuid/0192e0a7-9d8e-4771-9bb1-d730a2f66a9d
DEVNAME=/dev/bcac
Ignore my above 2 comments, posted these to the wrong bug.
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after reboot
Status in lin
Perhaps we need some udev plumbing expertise to resolve this.
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after reboot
Status in
** Changed in: linux (Ubuntu Xenial)
Status: Triaged => In Progress
** Changed in: linux (Ubuntu Xenial)
Assignee: (unassigned) => Joseph Salisbury (jsalisbury)
** Changed in: linux (Ubuntu Zesty)
Status: Triaged => In Progress
** Changed in: linux (Ubuntu Zesty)
Assignee
Once the kernel is fixed, are there any changes that are required to
systemd/udev?
** Changed in: systemd (Ubuntu Bionic)
Status: New => Incomplete
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launc
** Changed in: systemd (Ubuntu Xenial)
Status: New => Won't Fix
** Changed in: systemd (Ubuntu Zesty)
Status: New => Won't Fix
** Changed in: systemd (Ubuntu Artful)
Status: New => Won't Fix
** Changed in: systemd (Ubuntu Bionic)
Status: Incomplete => Won't Fix
--
Y
To recreate:
Add a second disk (any size will do) and split the disk into 4 equal
partitions. Then
make-bcache -C /dev/vdb1
make-bcache -B /dev/vdb2
make-bcache -B /dev/vdb3
make-bcache -B /dev/vdb4
Note, you can see the proper symlinks created at this time:
ls -al /dev/bcache/by-uuid/
Reboot
Did this issue start happening after an update/upgrade? Was there a
prior kernel version where you were not having this particular problem?
Would it be possible for you to test the latest upstream kernel? Refer
to https://wiki.ubuntu.com/KernelMainlineBuilds . Please test the latest
v4.14 kernel[
This fails on Xenial -> Artful. I've not yet tested Trusty, but I
suspect it's just a latent bug in the bcache driver.
I'll grab the upstream kernel and provide results.
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
ht
Mainline kernel does not help; it fails the same way and also has a bug
that's fixed in the Ubuntu kernel:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1667078
Here's the details from the mainline test:
ubuntu@a-unsolar-melonie:~$ dpkg --list | grep linux-image
ii linux-image-4.13.0-16-
** Changed in: linux (Ubuntu)
Status: Confirmed => Triaged
** Changed in: linux (Ubuntu Artful)
Status: Confirmed => Triaged
** Also affects: linux (Ubuntu Bionic)
Importance: Medium
Status: Triaged
** Also affects: linux (Ubuntu Xenial)
Importance: Undecided
St
Just to clarify on UUIDs - there are 3 types of UUIDs in question:
1. cache set UUIDs (upstream docs mention that multiple cache devices per cache
set are not supported but a configuration with M:N as in
: has been taken into account)
2. cache device superblock UUIDs;
3. backing device superbloc
There is a scenario where a real rootfs is located on a bcache device,
however, for that we need to register a bcache device at the initrd
stage which already happens now. Then we'd locate a file system on it
and do pivot_root and so on.
The bcache naming, I believe, is not guaranteed at this poin
** Tags added: cscc
--
You received this bug notification because you are a member of Kernel
Packages, which is subscribed to linux in Ubuntu.
https://bugs.launchpad.net/bugs/1729145
Title:
/dev/bcache/by-uuid links not created after reboot
Status in linux package in Ubuntu:
Fix Released
St
So, /dev/bcache/by-uuid is not getting created.
That's the same kernel bug I filed.
And, if they were, I think they'd get moved properly.
init-bottom/udev script does the following:
# Stop udevd, we'll miss a few events while we run init, but we catch up
udevadm control --exit
# move the /dev
Investigating why /dev doesn't have the correct links even if they're
present led me to look at
/usr/share/initramfs-tools/scripts/init-bottom/udev (provided via the udev
dpkg from systemd)
In there, the mount -n -o move is used to migrate the /dev from initramfs
to the rootfs/dev
This bug:
https
It looks like there is some ordering issues:
This is a grep through /run/udev/links ; these are checked by udev-dev
# find . -name 'b250*'
./\x2fdisk\x2fby-uuid\x2f0a270acb-56b8-4498-8bad-b3bb149fe869/b250:1
./\x2fdisk\x2fby-uuid\x2f92b0868d-7e56-4956-8e55-2c90ebee4a72/b250:0
./\x2fbcache\x2fby-u
Dec 01 05:07:25.679368 ubuntu systemd-udevd[474]: LINK
'disk/by-uuid/0a270acb-56b8-4498-8bad-b3bb149fe869'
/lib/udev/rules.d/60-persistent-storage.rules:79
Dec 01 05:07:25.683008 ubuntu systemd-udevd[474]: WARK:
dev_old_name=/dev/bcache/by-uuid/92d882d8-38cd-4537-847b-6f9c40ba67b4
dev_old_path=/dev
Hi Joseph,
Sorry, I didn't give that a compile either; I just wanted to show what the
change could look like;
Let me see if I can get that to at least compile.
On Mon, Dec 4, 2017 at 11:18 AM, Joseph Salisbury <
joseph.salisb...@canonical.com> wrote:
> It looks like env[] was declared in bch_ca
Looks like those two kfree's in dev_run can be dropped since that was an
exit after kmalloc'ing env entries which are now only done in
bch_cached_dev_emit_change()
which is only called by dev_run after it knows that the device is not
yet running.
On Mon, Dec 4, 2017 at 11:35 AM, Ryan Harper
wr
Thanks! I'll give it a try today.
On Tue, Dec 5, 2017 at 3:43 PM, Joseph Salisbury <
joseph.salisb...@canonical.com> wrote:
> I built an Artful test kernel with the path provided by Ryan. The test
> kernel can be downloaded from:
>
> http://kernel.ubuntu.com/~jsalisbury/lp1729145/
>
> Can those
Woot!
I think that did the trick.
ubuntu@ubuntu:~$ cat /etc/cloud/build.info
build_name: server
serial: 20171122
ubuntu@ubuntu:~$ lsb_release -rd
Description: Ubuntu 17.10
Release: 17.10
ubuntu@ubuntu:~$ set -o vi
ubuntu@ubuntu:~$ dpkg --list | grep linux-image
ii linux-image-4.13.0-17-generic
Thanks for doing the cleanup; Patch looks good and I approve.
Signed-off-by: Ryan Harper
On Tue, Dec 5, 2017 at 4:59 PM, Joseph Salisbury <
joseph.salisb...@canonical.com> wrote:
> In the final patch we submit for SRU, it will also include your Signed-
> off-by, I just forgot to add it in the
Here's the Zesty test; all looks good.
ubuntu@ubuntu:~$ cat /etc/cloud/build.info
build_name: server
serial: 20171207
ubuntu@ubuntu:~$ uname -a
Linux ubuntu 4.10.0-40-generic #44~lp1729145 SMP Wed Dec 6 16:21:45 UTC
2017 x86_64 x86_64 x86_64 GNU/Linux
ubuntu@ubuntu:~$ tree /dev/bcache
/dev/bcache
Tested the xenial update. I had one boot where the links didn't get
created, but I cannot recreate that issue now.
On Thu, Dec 7, 2017 at 9:56 AM, Ryan Harper
wrote:
> Here's the Zesty test; all looks good.
>
> ubuntu@ubuntu:~$ cat /etc/cloud/build.info
> build_name: server
> serial: 20171207
>
Confirmed bionic works as expected.
I suspect you can send that upstream with my SoB faster than I can.
Definitely interested in seeing if they'll take something like that.
On Thu, Dec 7, 2017 at 1:21 PM, Joseph Salisbury <
joseph.salisb...@canonical.com> wrote:
> Thanks for testing and the patc
Is this also fixed in bionic yet?
On Tue, Apr 3, 2018 at 9:10 AM, Launchpad Bug Tracker
<1729...@bugs.launchpad.net> wrote:
> This bug was fixed in the package linux - 4.13.0-38.43
>
> ---
> linux (4.13.0-38.43) artful; urgency=medium
>
> * linux: 4.13.0-38.43 -proposed tracker (LP:
Not sure if this was supposed to also be fixed in bionic-proposed yet,
but I assumed so and
it's not working there; I'll test -proposed in artful and xenial next.
ubuntu@ubuntu:~$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=18.04
DISTRIB_CODENAME=bionic
DISTRIB_DESCRIPTION="Ubuntu Bion
On Fri, Feb 16, 2018 at 12:14 PM, Joseph Salisbury <
joseph.salisb...@canonical.com> wrote:
> @raharper, I got side tracked and it got pushed down on my list. I can
> still send it upstream if you want. It's currently in Ubuntu as a SAUCE
> patch.
>
That would be great!
Thanks,
>
> --
> You
On Tue, Dec 12, 2017 at 5:52 AM, Dimitri John Ledkov wrote:
> Once the kernel is fixed, are there any changes that are required to
> systemd/udev?
>
No changes needed.
>
> ** Changed in: systemd (Ubuntu Bionic)
>Status: New => Incomplete
>
> --
> You received this bug notification beca
72 matches
Mail list logo