<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/fs/btrfs, branch v2018.09</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/fs/btrfs?h=v2018.09</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/fs/btrfs?h=v2018.09'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2018-07-20T19:37:44Z</updated>
<entry>
<title>fs: btrfs: Fix wrong comparison in logical to physical mapping</title>
<updated>2018-07-20T19:37:44Z</updated>
<author>
<name>Marek Behún</name>
<email>marek.behun@nic.cz</email>
</author>
<published>2018-07-04T18:23:01Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f8c173b6a0b7695089d5ec48cb5b320863c91fdf'/>
<id>urn:sha1:f8c173b6a0b7695089d5ec48cb5b320863c91fdf</id>
<content type='text'>
The comparison
  logical &gt; item-&gt;logical + item-&gt;length
in btrfs_map_logical_to_physical is wrong and should be instead
  logical &gt;= item-&gt;logical + item-&gt;length
For example, if
  item-&gt;logical = 4096
  item-&gt;length = 4096
and we are looking for logical = 8192, it is not part of item (item is
[4096, 8191]). But the comparison is false and we think we have found
the correct item, although we should be searing in the right subtree.

This fixes some bugs I encountered.

Signed-off-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Do not fail when all root_backups are empty</title>
<updated>2018-06-18T18:43:12Z</updated>
<author>
<name>Yevgeny Popovych</name>
<email>yevgenyp@pointgrab.com</email>
</author>
<published>2018-06-11T11:14:33Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=d146a7b9c6200c7a84a15208f0c06e0ab60890dd'/>
<id>urn:sha1:d146a7b9c6200c7a84a15208f0c06e0ab60890dd</id>
<content type='text'>
This is the case when reading freshly created filesystem.
The error message is like the following:
    btrfs_read_superblock: No valid root_backup found!

Since the data from super_roots/root_backups is not actually used -
decided to rework btrfs_newest_root_backup() into
btrfs_check_super_roots() that will only check if super_roots
array is valid and correctly handle empty scenario.

As a result:
* btrfs_read_superblock() now only checks if super_roots array is valid;
  the case when it is empty is considered OK.
* removed root_backup pointer from btrfs_info,
  which would be NULL in case of empty super_roots.
* btrfs_read_superblock() verifies number of devices from the superblock
  itself, not newest root_backup.

Signed-off-by: Yevgeny Popovych &lt;yevgenyp@pointgrab.com&gt;
Cc: Marek Behun &lt;marek.behun@nic.cz&gt;
Cc: Sergey Struzh &lt;sergeys@pointgrab.com&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Fix not all CHUNK_ITEMs being read from CHUNK_TREE</title>
<updated>2018-06-13T11:49:12Z</updated>
<author>
<name>Yevgeny Popovych</name>
<email>yevgenyp@pointgrab.com</email>
</author>
<published>2018-06-05T10:11:01Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f559180176a8650a78edb5be73935f51cd6a0064'/>
<id>urn:sha1:f559180176a8650a78edb5be73935f51cd6a0064</id>
<content type='text'>
This causes errors when translating logical addresses to physical:
  btrfs_map_logical_to_physical: Cannot map logical address &lt;addr&gt; to physical
  btrfs_file_read: Error reading extent

The behavior of btrfs_map_logical_to_physical() is to stop traversing
CHUNK_TREE when it encounters first non-CHUNK_ITEM, which makes
only some portion of CHUNK_ITEMs being read.
Change it to skip over non-chunk items.

Signed-off-by: Yevgeny Popovych &lt;yevgenyp@pointgrab.com&gt;
Cc: Marek Behun &lt;marek.behun@nic.cz&gt;
Cc: Sergey Struzh &lt;sergeys@pointgrab.com&gt;
Reviewed-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>SPDX: Convert all of our single license tags to Linux Kernel style</title>
<updated>2018-05-07T13:34:12Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2018-05-06T21:58:06Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=83d290c56fab2d38cd1ab4c4cc7099559c1d5046'/>
<id>urn:sha1:83d290c56fab2d38cd1ab4c4cc7099559c1d5046</id>
<content type='text'>
When U-Boot started using SPDX tags we were among the early adopters and
there weren't a lot of other examples to borrow from.  So we picked the
area of the file that usually had a full license text and replaced it
with an appropriate SPDX-License-Identifier: entry.  Since then, the
Linux Kernel has adopted SPDX tags and they place it as the very first
line in a file (except where shebangs are used, then it's second line)
and with slightly different comment styles than us.

In part due to community overlap, in part due to better tag visibility
and in part for other minor reasons, switch over to that style.

This commit changes all instances where we have a single declared
license in the tag as both the before and after are identical in tag
contents.  There's also a few places where I found we did not have a tag
and have introduced one.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Remove unused debug code left from development</title>
<updated>2018-04-06T21:04:33Z</updated>
<author>
<name>Marek Behún</name>
<email>marek.behun@nic.cz</email>
</author>
<published>2018-03-19T11:02:11Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=22fc7b6cd6e66911c58848d03eb0f1ddee024137'/>
<id>urn:sha1:22fc7b6cd6e66911c58848d03eb0f1ddee024137</id>
<content type='text'>
Signed-off-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Fix printf format character warning</title>
<updated>2018-01-29T17:50:08Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2018-01-27T22:23:21Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f39bfec230828a843bb53d0213709d47db1574df'/>
<id>urn:sha1:f39bfec230828a843bb53d0213709d47db1574df</id>
<content type='text'>
When printing a size_t value we need to use %zu for portability between
32bit and 64bit targets.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
Reviewed-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Fix unaligned memory accesses</title>
<updated>2018-01-28T17:27:12Z</updated>
<author>
<name>Alberto Sánchez Molero</name>
<email>alsamolero@gmail.com</email>
</author>
<published>2018-01-20T07:17:57Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=2021f083ed634f8233054a6299d95666a933434e'/>
<id>urn:sha1:2021f083ed634f8233054a6299d95666a933434e</id>
<content type='text'>
Loading files stored with lzo compression from a btrfs filesystem was
producing unaligned memory accesses, which were causing a data abort
and a reset on an Orange Pi Zero.

The change in hash.c is not triggered by any error but follows the
same pattern. Please confirm.

Fixed according to doc/README.unaligned-memory-access.txt

Signed-off-by: Alberto Sánchez Molero &lt;alsamolero@gmail.com&gt;
Tested-by: Robert Nelson &lt;robertcnelson@gmail.com&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Remove a foreign language note</title>
<updated>2017-10-16T13:42:51Z</updated>
<author>
<name>Marek Behún</name>
<email>marek.behun@nic.cz</email>
</author>
<published>2017-10-06T13:04:59Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=733d963e3b3e39fd49a9090b3f29ad0dafd2277d'/>
<id>urn:sha1:733d963e3b3e39fd49a9090b3f29ad0dafd2277d</id>
<content type='text'>
I accidentaly left a foreign language note in the code from development.

Signed-off-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>fs: btrfs: Fix usage of uninitialized variables</title>
<updated>2017-10-16T13:42:51Z</updated>
<author>
<name>Marek Behún</name>
<email>marek.behun@nic.cz</email>
</author>
<published>2017-10-06T13:04:57Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ecab881c7fcd7484420b6aa15b0f748458640191'/>
<id>urn:sha1:ecab881c7fcd7484420b6aa15b0f748458640191</id>
<content type='text'>
The variable res should be initialized to 0 in these functions,
because if the searched key is not found, the variable is used
uninitialized.

Reported-by: Coverity (CID: 167335)
Reported-by: Coverity (CID: 167336)
Reported-by: Coverity (CID: 167337)
Signed-off-by: Marek Behun &lt;marek.behun@nic.cz&gt;
</content>
</entry>
<entry>
<title>fs/btrfs: Fix warning in btrfs_check_super()</title>
<updated>2017-10-03T12:44:13Z</updated>
<author>
<name>Tom Rini</name>
<email>trini@konsulko.com</email>
</author>
<published>2017-10-03T12:44:13Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6b7fd350e85a97892a2cb066364797f3f3dcb9c9'/>
<id>urn:sha1:6b7fd350e85a97892a2cb066364797f3f3dcb9c9</id>
<content type='text'>
We specifically say that the last arg is u32, so use %lu.

Signed-off-by: Tom Rini &lt;trini@konsulko.com&gt;
</content>
</entry>
</feed>
