docs: my_proc_file_ops from :c:func: to :c:data: #214

Signed-off-by: Alexandra Sandulescu <[email protected]>

* use maximum 80 characters per line * use specific c constructs (:c:type``, :c:func``, etc) instead of :code: or elixir links * remove end of line whitespace Signed-off-by: Anda Nicolae <[email protected]> Signed-off-by: Octavian Purdila <[email protected]>

Signed-off-by: Alexandra Sandulescu <[email protected]>

Lab07 presents the API used by the kernel when working with block devices. It also explains the core structure of a generic block device driver. Signed-off-by: Anda Nicolae <[email protected]>

* add removed entry from index.rst * remove unneeded section and bump up the remaining ones * use "Submit" instead of "Transmit" * use "type" instead of "struct" in Sphinx syntax * parse line > 80 Signed-off-by: Anda Nicolae <[email protected]>

Remove reference to deprecated field struct block_device *bi_bdev from struct bio and add instead struct gendisk *bi_disk field. Signed-off-by: Anda Nicolae <[email protected]>

Signed-off-by: Razvan Deaconescu <[email protected]>

* fix grammar issues * use Sphinx syntax when needed to highlight C structures, functions, macros etc. * parse line > 80 Signed-off-by: Anda Nicolae <[email protected]>

One can use this skeleton to start implementing e100 driver assignment. Signed-off-by: Daniel Baluta <[email protected]>

Signed-off-by: Daniel Baluta <[email protected]>

Change network for second interface because nttcp is compiled with hardcoded listening addresss (172.30.0.1). TODO: recompile nttcp to support configurable ip address. Signed-off-by: Daniel Baluta <[email protected]>

Signed-off-by: Razvan Deaconescu <[email protected]>

Signed-off-by: Daniel Baluta <[email protected]>

Signed-off-by: Razvan Deaconescu <[email protected]>

Makefile is to be used for building the test suite executable instead of Makefile.checker.

Typo

Because we are using virtio the device name is vd* not sd*. Also, because it is the fourth disk the name will be /dev/vdd. Signed-off-by: Diana Ungureanu <[email protected]>

Signed-off-by: Daniel Baluta <[email protected]>

Signed-off-by: Cristi Done <[email protected]>

Dyndbg can't filter messages printed by pr_info(). Use pr_debug() instead.

Course presents F5/F6 keys as shortcuts to move between multiple results but they are not implemented (not working). Signed-off-by: Lazar Razvan <[email protected]>

Dyndbg can't filter messages printed by pr_info(). Use pr_debug() instead.

Signed-off-by: Cristi Done <[email protected]>

* CONFIG_FRAME_WARN is set to 1024 and we had a warning that a frame size was larger than this Signed-off-by: Claudiu Ghioc <[email protected]>

Signed-off-by: Cristi Done <[email protected]>

Signed-off-by: Valentin Ghita [email protected]

Signed-off-by: Cristi Done <[email protected]>

"device" and "my_data->buffer" represent the same thing(dev is simulated). Add offset in my_write func, like my_read.

Signed-off-by: Cristi Done <[email protected]>

Signed-off-by: Razvan Deaconescu <[email protected]>

Timers API changed since 4.14: * setup_timer was renamed to timer_setup * timer handlers argument type is now `struct timer_list *`

…0-list

…ment

Signed-off-by: Cristi Done <[email protected]>

…queue to sections Signed-off-by: Sergiu Weisz <[email protected]>

Signed-off-by: Sergiu Weisz <[email protected]>

Signed-off-by: Valenting Ghita <[email protected]>

Signed-off-by: Valentin Ghita <[email protected]>

…labs/file_systems_part_2 teaching: labs: filesystems part 2: Add lab documentation

* if I_NEW flag is not set after iget_locked then we are dealing with a cached inode which can be used right away. Signed-off-by: Claudiu Ghioc <[email protected]>

…_use_cached_inodes teaching: labs: filesystems part 2: minfs: return inode from cache

Signed-off-by: Valentin Ghita [email protected]

…ncioiu/remove-dirname-error-msg tools:labs: Add --no-run-if-empty option to xargs

teaching: labs: device_drivers: Fix typo

tools: labs: kernel_modules: fix dyndbg

…ting colon (:) In filesystems_part2 lab

In filesystems_part2 lab

…it() It's already called as part of the skeleton of filesystems_part2 lab.

(Not) used in filesystems lab in minfs_readir() skeleton.

…ommeting of simple_lookup call In filesystems_part2 lab

Use id 3 for `Create operation` section in filesystems_part2 lab.

Signed-off-by: Sergiu Weisz <[email protected]>

Function bodies are already added in the skeleton of filesystems_part2 lab..

In filesystems_part2 lab.

In minfs driver of the filesystems_part2 lab

In filesystems_part2 lab

As of QEMU 3.0 the vlan parameter has been removed, so I had to change the -net option to -netdev, which has a similar functionality to vlan, ie run the device in a separate network. Signed-off-by: Sergiu Weisz <[email protected]>

Signed-off-by: Sergiu Weisz <[email protected]>

tools: labs: qemu: Makefile: Remove deprecated vlan parameter

…ems_part2 Documentation: teaching: labs: filesystems_part2: Fix typos

teaching: labs: Review lab09: Filesystems (part 2)

Signed-off-by: Sergiu Weisz <[email protected]>

Documentation: teaching: labs: networking: Add networking lab

Signed-off-by: Sergiu Weisz <[email protected]>

It was looking for netcat in a directory that was two levels higher in the hierarchy, but netcat was only one level above Signed-off-by: Sergiu Weisz <[email protected]>

Signed-off-by: Sergiu Weisz <[email protected]>

From my research in the kernel. The last parameter in the accept op signifies whether the socket will be used in the kernel, and the socket in this exercise will be used in kernel, to this should be set to true. Signed-off-by: Sergiu Weisz <[email protected]>

Documentation: teaching: labs: networking: Minor content fixes

…racer Improve README and remove Makefile errors when building.

In networking lab

In networking, 5-udp-sock task

In networking lab

…n packets In networking lab

In networking lab, task 5-udp-sock

tools: labs: Update networking lab (linux-kernel-labs#10)

Signed-off-by: Octavian Purdila <[email protected]>

Update 2020 cs.pub.ro team and fix a few intro slides titles

…-fixes Documentation: teaching: labs: introduction content fixes

Signed-off-by: Valentin Ghita [email protected]

Signed-off-by: Sergiu Weisz <[email protected]>

Signed-off-by: Valentin Ghita [email protected]

…el_modules/content_updates Documentation: teaching: labs: kernel_modules exercises updates

…ses in every lab Signed-off-by: Valentin Ghita [email protected]

Signed-off-by: Valentin Ghita [email protected]

…el_modules/add_missing_kdb_section Documentation: teaching: labs: kernel_modules: add missing KDB section

tools/labs: Remove unnecessary copy from make copy

…labs/add_git_pull_warning Documentation: teaching: labs: Add git pull warning before exercises

Signed-off-by: Razvan Deaconescu <[email protected]>

At some point we renamed Documentation/labs with Documentation/teaching but did not update README.rst symbolic link. Do it now! Signed-off-by: Daniel Baluta <[email protected]>

Signed-off-by: Octavian Purdila <[email protected]>

Only trigger builds on pushes and pull request on master branch. Signed-off-by: Octavian Purdila <[email protected]>

This avoids conflicts for pull request builds. Signed-off-by: Octavian Purdila <[email protected]>

Signed-off-by: Sergiu Weisz <[email protected]>

Github action workflows do not pass secrets to forked repositories which means that we can't publish to linux-kernel-library.github.io. For now publish as an artifact. Signed-off-by: Octavian Purdila <[email protected]>

…el_api/content_fixes Documentation: teaching: labs: kernel_api: Fix indentation for exercises TODOs

tools/labs: Change make clean rule to only remove generated files

Currently the ditaa command is invoked directly and assumes that binfmt_misc support is enabled on the host to execute ditaa via jexec or jarwrapper. However that is not true in all cases (e.g. docker guests does not yet properly support binfmt_misc). This commit adds support for command invocation customization with two environment variables: SPHINX_DITAA_CMD to specify the executable and SPHINX_DITAA_ARG to specify a list of parameters. Signed-off-by: Octavian Purdila <[email protected]>

pip depedencies belong to the host setup environment. We will provide a Dockerfile to deal with host dependencies. Signed-off-by: Octavian Purdila <[email protected]>

…cker Signed-off-by: Octavian Purdila <[email protected]>

…cu/labs/kernel_api/exercises_typos Documentation: teaching: labs: kernel_api: Fix typos in exercises

…st-test

Add support for building documentation via docker

…cu/labs/kernel_api/ex7_list_test tools/labs: Add TODO for exercise 7-list-test

This will enable support for visualizing documentation changes on linux-kernel-library.github.io Signed-off-by: Octavian Purdila <[email protected]>

.github: publish CI artifacts to linux-kernel-library.github.io

Signed-off-by: Octavian Purdila <[email protected]>

.github: remove debug left over

…pact of changes Signed-off-by: Razvan Deaconescu <[email protected]>

…cker Add setup for vmchecker

Signed-off-by: Valentin Ghita [email protected]

…evice_drivers/exercises-updates Documentation: labs: device_drivers: update exercises

Signed-off-by: Valentin Ghita [email protected]

…ntation/add-contribution-page Documentation: Add contributions page

Documentation: teaching: labs: deferred_work: Fix example code

…rocedure Signed-off-by: Valentin Ghita [email protected]

…/labs/device_drivers/update_extra_exercises_testing Documentation: labs: device_drivers: Update extra exercises testing

Signed-off-by: SeongJae Park <[email protected]>

docs/contributing: Fix typos: s/linux-lernel/linux-kernel/

Fix typo at MODULE_AUTHOR

When running as root, the USER variable is not set, so we have to set it explicitly. Signed-off-by: Sergiu Weisz <[email protected]>

The correct return value for the case when interrupt was not from this device or was not handled is IRQ_NONE

Signed-off-by: Octavian Purdila <[email protected]>

…with netconsole Signed-off-by: Razvan Deaconescu <[email protected]>

…n slate image for vmchecker Signed-off-by: Razvan Deaconescu <[email protected]>

…ebsite Signed-off-by: Razvan Deaconescu <[email protected]>

out/run-stdout.vmr and out/run-stderr.vmr Signed-off-by: Razvan Deaconescu <[email protected]>

Signed-off-by: Razvan Deaconescu <[email protected]>

Add the missing CLI in the pseudocode description. Signed-off-by: Octavian Purdila <[email protected]>

Add pseudocode that explains the BTS instruction. Signed-off-by: Octavian Purdila <[email protected]>

Fixes the following warnings: Documentation/teaching/lectures/smp.rst:210: WARNING: Bullet list ends without a blank line; unexpected unindent. Documentation/teaching/lectures/smp.rst:771: WARNING: Bullet list ends without a blank line; unexpected unindent. Documentation/teaching/lectures/smp.rst:777: WARNING: Bullet list ends without a blank line; unexpected unindent. Signed-off-by: Octavian Purdila <[email protected]>

Add reference to LWN ticket spinlock article. Signed-off-by: Octavian Purdila <[email protected]>

Update the RCU diagram so that B to C link is showed as preserved during the removal phase. Signed-off-by: Octavian Purdila <[email protected]>

Signed-off-by: Valentin Ghita [email protected]

Remove the so2_ prefix which has no relevance currently and simplify the example to make it easier to demonstrate in the debugging lecture. Signed-off-by: Octavian Purdila <[email protected]>

Remove a few headers we don't use and the so2 prefix. More importantly remove the schedule_timeout statement as it is not needed for lockdep to detect the issue and add debug prints to show that. Signed-off-by: Octavian Purdila <[email protected]>

Remove the while (1) block as it is not necessary for lockdep to detect the issue. Also, use new APIs to make the code more compact and remove the so2 prefix. Signed-off-by: Octavian Purdila <[email protected]>

Enable kmemleak in the kernel config and update qemu to start with 256MB as otherwise we get an early memory allocation error. Signed-off-by: Octavian Purdila <[email protected]>

This is just a workaround, see the thread [1] on the netdev mailing list for the discussion around this issue. But it allows us to avoid the initial lockdep warning on boot that disables futher lockdep checks. [1] https://www.spinics.net/lists/netdev/msg529595.html Signed-off-by: Octavian Purdila <[email protected]>

Update the SLUB debugging, kmemleak and lockdep checker sections with more information a few examples. Signed-off-by: Octavian Purdila <[email protected]>

Signed-off-by: Octavian Purdila <[email protected]>

As soon as the lock is unlocked in task_info_find_pid, the caller cannot be sure if the pointer that is returned is valid. As long as the returned pointer is used, the lock must be held. In the case of task_info_add_to_list, the lock that must be held is a writer lock, in case of a reader lock other readers could read an inconsistent state of the struct task_info.

Starting from kernel version 4.10, `bio_set_op_attrs` is marked as obsolete. The recommended action is to directly assign the `bio_opf` field of the `struct bio`. Code extract from `/include/linux/blk_types.h` (v4.19): ```c /* obsolete, don't use in new code */ static inline void bio_set_op_attrs(struct bio *bio, unsigned op, unsigned op_flags) { bio->bi_opf = op | op_flags; } ``` Signed-off-by: Horia Ion <[email protected]>

`minfs_write_inode` should fill the disk inode with the aquired `uid` and `gid` from the inode, not the other way round. Please check the `minix` solution below: https://elixir.bootlin.com/linux/latest/source/fs/minix/inode.c#L557

This fixes errors during the "Install native dependencies" step for building documentation. Signed-off-by: Octavian Purdila <[email protected]>

Changed

For the `makedev` macro in newer versions of Glibc (since v2.28) we need to directly include <sys/sysmacros.h>, because that is no longer included by <sys/types.h>. Fix by including the correct header. Signed-off-by: Paul-Stelian Olaru <[email protected]>

Up until this commit only core-image-miminal-* Yocto image files were ignored. Others, such as core-image-sato-* Yocto image files were not. This commit fixes that, will al Yocto image files being ignored. Signed-off-by: Razvan Deaconescu <[email protected]>

Signed-off-by: Sergiu Weisz <[email protected]>

For 5-pitix assignment there is no info about score or the number of test passed (at least a counter). Simple searching for "failed" in output is error prone (and painfully hard for the average Joe). A simple solution is to add a test_ok var to count the number of passed test. Better solutions exist, but keep it simple stupid as fellow student only care about the number of tests passed to know the homework is okay.

- Change TODO numbering to match code template - Fix and improve testing instructions

… ditaa diagrams Signed-off-by: Octavian Purdila <[email protected]>

`$` should be used with normal shell commands, while `#` should be used for root commands. `ls` commands on /dev/* doesn't usually require root access.

Signed-off-by: SeongJae Park <[email protected]>

Some of the documents are written under assumption of this repo is cloned as '~/src/linux' while others assumes '~/so2/linux'. This commit makes all documents to use 'src/', as it is more widely used. $ git grep src/linux Documentation/teaching | wc -l 58 $ git grep so2/linux Documentation/teaching | wc -l 19 Signed-off-by: SeongJae Park <[email protected]>

Signed-off-by: SeongJae Park <[email protected]>

The document says writing to '/proc/hello_kdb_break' increments 'kdb_write_address', but the code just write 1 to 'write_address'. This commit makes the code to work as explained in the document. Signed-off-by: SeongJae Park <[email protected]>

Commit 44b5f01 ("Documentation/teaching: Use 'src/linux' instead of 'so2/linux'") missed fixing directory creation command. This commit fixes it. Signed-off-by: SeongJae Park <[email protected]>

the `inte i;` here clashes with `struct bvec_iter i;` and is not used

Signed-off-by: SeongJae Park <[email protected]>

…tation dyndbg option example for 'format' has unnecessary indentation. This commit removes it. Signed-off-by: SeongJae Park <[email protected]>

In 'Dyndbg Options' section, verbatim (``*``) is applied to all keywords but 'line'. This commit makes it consistently applied. Signed-off-by: SeongJae Park <[email protected]>

In 'Dyndbg Options' section, it explains 'I', 'M', and 'T' flags, which don't exist. According to the explanations, those should be 'l', 'm', and 't'. This commit fixes those. Signed-off-by: SeongJae Park <[email protected]>

Signed-off-by: SeongJae Park <[email protected]>

Seems reader-writer spinlock interfaces intended to be displayed as a bullets list. However, the syntax is wrongly applied. Moreover, it would seems natural to displayed inside the sentence, as those for spinlock are. So, this commit makes the interfaces to be naturally listed in the sentence. Signed-off-by: SeongJae Park <[email protected]>

…bles' section Signed-off-by: SeongJae Park <[email protected]>

So it doesn't render as `my_proc_file_ops`

Commits on Oct 2, 2020

Typo: mincom -> minicom

mido3ds authored and tavip committed Oct 2, 2020

Configuration menu

View commit details

Copy full SHA for a3ed56c

Browse repository at this point

Copy the full SHA

a3ed56c View commit details

Browse the repository at this point in the history

docs: my_proc_file_ops from :c:func: to :c:data: #214

Are you sure you want to change the base?

docs: my_proc_file_ops from :c:func: to :c:data: #214

Commits on Jan 20, 2019

Commits on Feb 9, 2019

Commits on Feb 26, 2019

Commits on Feb 28, 2019

Commits on Mar 4, 2019

Commits on Mar 5, 2019

Commits on Mar 6, 2019

Commits on Mar 11, 2019

Commits on Mar 12, 2019

Commits on Mar 13, 2019

Commits on Mar 14, 2019

Commits on Mar 15, 2019

Commits on Mar 16, 2019

Commits on Mar 17, 2019

Commits on Mar 27, 2019

Commits on Mar 28, 2019

Commits on Mar 30, 2019

Commits on Apr 2, 2019

Commits on Apr 8, 2019

Commits on Apr 10, 2019

Commits on Apr 11, 2019

Commits on Apr 16, 2019

Commits on Apr 17, 2019

Commits on Apr 21, 2019

Commits on Apr 24, 2019

Commits on Apr 25, 2019

Commits on Apr 27, 2019

Commits on May 4, 2019

Commits on May 5, 2019

Commits on May 6, 2019

Commits on Feb 18, 2020

Commits on Feb 19, 2020

Commits on Feb 20, 2020

Commits on Feb 24, 2020

Commits on Feb 25, 2020

Commits on Feb 26, 2020

Commits on Feb 27, 2020

Commits on Feb 28, 2020

Commits on Mar 1, 2020

Commits on Mar 2, 2020

Commits on Mar 3, 2020

Commits on Mar 4, 2020

Commits on Mar 8, 2020

Commits on Mar 10, 2020

Commits on Mar 11, 2020

Commits on Mar 13, 2020

Commits on Mar 14, 2020

Commits on Mar 15, 2020

Commits on Mar 18, 2020

Commits on Mar 19, 2020

Commits on Mar 24, 2020

Commits on Mar 30, 2020

Commits on Apr 1, 2020

Commits on Apr 5, 2020

Commits on Apr 7, 2020

Commits on Apr 21, 2020

Commits on Apr 30, 2020

Commits on May 4, 2020

Commits on May 5, 2020

Commits on May 6, 2020

Commits on May 7, 2020

Commits on May 11, 2020

Commits on May 13, 2020

Commits on Oct 2, 2020

Commits on Oct 7, 2020

Commits on Oct 9, 2020

Commits on Oct 14, 2020

Commits on Oct 15, 2020

Commits on Oct 16, 2020

Commits on Oct 18, 2020

Commits on Oct 19, 2020

Commits on Oct 21, 2020