vprintf, kprintfの修正 #28

74rina · 2025-12-13T10:44:48Z

commonのvprintfで、特権に依存するputcharを直接呼ばないように変更しました（引数のputcで、kputchar(カーネル)またはputchar(ユーザー)を指定する形です）。

ユーザーのprintf → vprintf( putchar{ syscall } )
カーネルのkprintf → vprintf( kputchar{ sbi_call } )

という構成にしました。

正しい実装か分からず、レビューお願いいたします😭

no2ca

putchar/kputcharがどこで設定されているか分かりやすくて良いと思いました👍

Latte72R · 2025-12-15T11:30:53Z

@74rina
vprintf はstdcの関数なので中間処理をする関数を作ったほうがstdc互換を保てていいかも

int vprintf(const char *fmt, va_list vargs) {
    return _vprintf(putchar, fmt, vargs);
}

int _vprintf(putc_fn_t putc, const char *fmt, va_list vargs) {
    ...
}

lemolatoon · 2025-12-15T11:39:46Z

common.c

void __common_putc(char c);

int vprintf(putc_fn_t putc, const char *fmt, va_list vargs) {
  ...
  __common_putc(...);
  return count;
}

kernel.c

void kputchar(char ch) {...}
void __common_putc(char ch) __attribute__((alias("kputchar")));

usys.c

void putchar(char ch) {...}
void __common_putc(char ch) __attribute__((alias("putchar")));

とする手もあり、どっちがきれいかは諸説あり

Latte72R

@74rina

Latte72R · 2025-12-28T21:00:01Z

kernel/kernel.c

 void kputchar(char ch) { sbi_call(ch, 0, 0, 0, 0, 0, 0, 1); }
+void __common_putc(char ch) __attribute__((alias("kputchar")));

 void putchar(char ch) { kputchar(ch); }


ここの putchar は必要ないです．

Latte72R · 2025-12-28T21:03:19Z

common/common.c

 #include "common.h"

-int vprintf(const char *fmt, va_list vargs) {
+void __common_putc(char c);


関数の宣言はデフォルトで extern なので必要はないけど，extern void __common_putc(char c); のほうが分かりやすいかも．

Latte72R · 2025-12-28T21:19:58Z

common/common.c

-int vprintf(const char *fmt, va_list vargs) {
+void __common_putc(char c);
+
+int vprintf(putc_fn_t __common_putc, const char *fmt, va_list vargs) {


putc_fn_t __common_putc を受け取る必要はないです．

Latte72R · 2025-12-28T21:21:58Z

common/common.h


 int printf(const char *fmt, ...);
-int vprintf(const char *fmt, va_list ap);
+int vprintf(putc_fn_t putc, const char *fmt, va_list vargs);


putc_fn_t __common_putc は必要ないです．（上に同じく）

Latte72R · 2025-12-28T21:22:57Z

common/common_types.h

 typedef uint32_t paddr_t;
 typedef uint32_t vaddr_t;
 typedef uint8_t bool;
+typedef void (*putc_fn_t)(char);


putc_fn_t __common_putc を廃止するので，これも必要ないです．

Latte72R · 2025-12-28T21:30:14Z

kernel/kernel.c

  va_list vargs;
  va_start(vargs, fmt);
-  int ret = kvprintf(fmt, vargs);
+  int ret = vprintf(kputchar, fmt, vargs);


ここも変更してください．

Latte72R · 2025-12-28T21:31:21Z

user/usys.c

 void sys_concat_first_file(void) { syscall(SYS_CAT_FIRST_FILE, 0, 0, 0); }

+
+int vprintf(void (*putc)(char), const char *fmt, va_list vargs);


common.h で宣言しているため不要です．

Latte72R · 2025-12-28T21:32:09Z

common/common.c

  return count;
 }

-int printf(const char *fmt, ...) {


この printf を消す必要はありません．

Latte72R · 2025-12-28T21:33:47Z

user/usys.c


+
+int vprintf(void (*putc)(char), const char *fmt, va_list vargs);
+int printf(const char *fmt, ...) {


printf は common.h に置いてください．

Latte72R · 2025-12-28T21:33:57Z

user/usys.h

 int create_file(const char *name, const uint8_t *data, uint32_t size);
 void sys_list_root_dir(void);
 void sys_concat_first_file(void);
+int printf(const char *fmt, ...);


printf は common.h に置いてください．

74rina added 2 commits December 13, 2025 19:27

vprintf,kprintfの修正

8870526

kprintf内でいきなりputcharではなくkputcharを呼ぶ

38b85ab

lemolatoon requested review from Latte72R, lemolatoon and no2ca December 13, 2025 23:49

no2ca approved these changes Dec 14, 2025

View reviewed changes

lemolatoon approved these changes Dec 14, 2025

View reviewed changes

74rina and others added 4 commits December 26, 2025 19:57

Merge branch 'main' into feature/vprintf

00adaec

common内に__common_putcを追加

71e66f7

alias追加

a5420c2

include

f18847c

Latte72R requested changes Dec 28, 2025

View reviewed changes

Latte72R and others added 11 commits January 11, 2026 10:59

fopen, fclose, fgetc, fputcを実装

f174ac4

コードの見た目を修正しました。

37bf1bb

shutdownを実装

59e2907

Update Makefile

07b427e

fopen, fclose, fgetc, fputcを実装

5215ac9

update

bb339f9

fix

77590ca

ユーザー側からもファイル操作を行えるように修正

6e88103

vprintf,kprintfの修正

daf3f75

printfはcommonに移動

49392cb

Merge branch 'main' into feature/vprintf

77dcaca

Latte72R self-requested a review January 11, 2026 02:36

Latte72R approved these changes Jan 11, 2026

View reviewed changes

Latte72R merged commit 521af6c into main Jan 11, 2026

		void sys_concat_first_file(void) { syscall(SYS_CAT_FIRST_FILE, 0, 0, 0); }


		int vprintf(void (putc)(char), const char fmt, va_list vargs);



		int vprintf(void (putc)(char), const char fmt, va_list vargs);
		int printf(const char *fmt, ...) {

vprintf, kprintfの修正 #28

vprintf, kprintfの修正 #28

Conversation

74rina commented Dec 13, 2025

Uh oh!

no2ca left a comment

Choose a reason for hiding this comment

Uh oh!

Latte72R commented Dec 15, 2025

Uh oh!

lemolatoon commented Dec 15, 2025

Uh oh!

Latte72R left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Latte72R Dec 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Latte72R Dec 28, 2025 •

edited

Loading